Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

📝 Documentation: More prominently feature contribution guide #661

Closed
2 tasks done
rakleed opened this issue Nov 29, 2024 · 6 comments · Fixed by #665
Closed
2 tasks done

📝 Documentation: More prominently feature contribution guide #661

rakleed opened this issue Nov 29, 2024 · 6 comments · Fixed by #665
Labels
area: documentation Improvements or additions to docs good first issue Good for newcomers, please hop on! status: accepting prs Please, send a pull request to resolve this!

Comments

@rakleed
Copy link
Contributor

rakleed commented Nov 29, 2024

Bug Report Checklist

Overview

It would be great to have a contribution guide that includes information on how to test changes and build the plugin. Additionally, it would be helpful to have details about the minimum required Node version and other prerequisites.

Additional Info

No response

@rakleed rakleed added the area: documentation Improvements or additions to docs label Nov 29, 2024
@JoshuaKGoldberg
Copy link
Owner

contribution guide

  1. https://github.com/JoshuaKGoldberg/eslint-plugin-package-json/blob/main/.github/CONTRIBUTING.md
  2. https://github.com/JoshuaKGoldberg/eslint-plugin-package-json/blob/main/.github/DEVELOPMENT.md

test changes

https://github.com/JoshuaKGoldberg/eslint-plugin-package-json/blob/main/.github/DEVELOPMENT.md#testing

build the plugin

https://github.com/JoshuaKGoldberg/eslint-plugin-package-json/blob/main/.github/DEVELOPMENT.md#building

Minimum required Node version

"engines": {
"node": ">=18"
},

I'm inferring you haven't read through the docs very deeply 🙂. Once you've read through them, if you have any specific questions on things not covered, please do post back or file a new issue.

@JoshuaKGoldberg JoshuaKGoldberg closed this as not planned Won't fix, can't repro, duplicate, stale Nov 29, 2024
@JoshuaKGoldberg JoshuaKGoldberg added the invalid This doesn't seem right label Nov 29, 2024
@rakleed
Copy link
Contributor Author

rakleed commented Nov 29, 2024

@JoshuaKGoldberg, for the first time, I discovered the contribution and development guides in the .github directory. I’m not sure, but maybe it would be better to place them in the root directory.

And it would be great if you could include a reference to the contribution/development guide in the README file to increase its discoverability because it's where I actually search for it.

@JoshuaKGoldberg
Copy link
Owner

I can see why you'd ask for that, but it's a GitHub standard to allow them in .github. From https://docs.github.com/en/communities/setting-up-your-project-for-healthy-contributions/setting-guidelines-for-repository-contributors (emphasis mine):

To help your project contributors do good work, you can add a file with contribution guidelines to your project repository's root, docs, or .github folder. When someone opens a pull request or creates an issue, they will see a link to that file. The link to the contributing guidelines also appears on your repository's contribute page. For an example of a contribute page, see github/docs/contribute.

I think the discoverability issue might be something to take up with the GitHub website itself. I agree that it'd be nice to have more prominent "hey go read this!". 😄

@JoshuaKGoldberg JoshuaKGoldberg added status: in discussion Not yet ready for implementation or a pull request and removed invalid This doesn't seem right labels Nov 29, 2024
@JoshuaKGoldberg JoshuaKGoldberg changed the title 📝 Documentation: add contribution guide 📝 Documentation: More prominently feature contribution guide Nov 29, 2024
@JoshuaKGoldberg
Copy link
Owner

That's a good point. And, in fact, I do this in my other repos - this one just isn't matched to the template!

Example from https://github.com/JoshuaKGoldberg/create-typescript-app?tab=readme-ov-file#development:

## Development

See [`.github/CONTRIBUTING.md`](./.github/CONTRIBUTING.md), then [`.github/DEVELOPMENT.md`](./.github/DEVELOPMENT.md).
Thanks! 💖

Accepting PRs to add the equivalent to this project's README.md. Thanks for filing, good catch!

@JoshuaKGoldberg JoshuaKGoldberg added good first issue Good for newcomers, please hop on! status: accepting prs Please, send a pull request to resolve this! and removed status: in discussion Not yet ready for implementation or a pull request labels Nov 29, 2024
Copy link

🎉 This is included in version v0.16.0 🎉

The release is available on:

Cheers! 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: documentation Improvements or additions to docs good first issue Good for newcomers, please hop on! status: accepting prs Please, send a pull request to resolve this!
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants