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
Document how to write commits #2578
Conversation
Review the rendered version of these changes here: https://github.com/tcharding/rust-bitcoin/blob/03-13-commit-tips/CONTRIBUTING.md#preparing-prs |
Pull Request Test Coverage Report for Build 8274539858Details
💛 - Coveralls |
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.
ACK 370a294
CONTRIBUTING.md
Outdated
@@ -118,6 +118,12 @@ NB: reviewers may run more complex test/CI scripts, thus, satisfying all the | |||
requirements above is just a preliminary, but not necessary sufficient step for | |||
getting the PR accepted as a valid candidate PR for the `master` branch. | |||
|
|||
High quality commits help us review and merge you contributions, we attempt to |
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.
In 370a294:
This ,
should be a period, and capital We
.
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.
Done, thanks.
BTW. Instead of/along with writing it in a document, you might want to set up a git commit template and some simple hook to install it. Example from Fedimint: as you see it's slightly different, but we could put anything there really. A link for new contributors, or to a bigger guide, or even a wallet of text verbatim. |
Reviewers often find themselves linking to blog posts to encourage newer devs to improve their commit logs, we can save everyones time by putting the links in the contributing docs, then we can just point devs there.
370a294
to
8bd0394
Compare
Fixed grammar as suggested. |
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.
ACK 8bd0394
This is docs only, could go in with a single ack. |
Reviewers often find themselves linking to blog posts to encourage newer devs to improve their commit logs, we can save everyones time by putting the links in the contributing docs, then we can just point devs there.