-
Notifications
You must be signed in to change notification settings - Fork 68
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
docs: update CONTRIBUTING.md
and PR template
#217
Conversation
7da8287
to
0530120
Compare
Overall I think the PR template needs to be updated to include the commands to run the CI check script once we merge #208 so that its front and center in case people ignore the contributing docs. Further, that PR template should have a small example of how we want the PR description to look so people can easily work off of that. This should greatly clarify our expectations and decrease turn around time during reviews |
f1aedfa
to
8dbd581
Compare
c890dd7
to
5f8a6ad
Compare
@Dustin-Ray @JayWhite2357 Please review again. |
CONTRIBUTING.md
CONTRIBUTING.md
and PR template
5f8a6ad
to
141f451
Compare
6620d28
to
1f6ac54
Compare
1f6ac54
to
cb27184
Compare
cb27184
to
0fff4b0
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I really like this a lot thanks, hopefully our contributors will find this useful as well
🎉 This PR is included in version 0.28.5 🎉 The release is available on GitHub release Your semantic-release bot 📦🚀 |
Rationale for this change
CONTRIBUTING.md
is seriously out of date now that we are an open source project. Furthermore the PR template needs an example so that we can set adequate expectations for contribution.What changes are included in this PR?
cargo test
commandGetting Started
with Rust installation and repo forking guideCONTRIBUTING.md
and add an exampleAre these changes tested?
N/A since there are no code changes.