mirror of
https://github.com/dashpay/dash.git
synced 2024-12-25 12:02:48 +01:00
Merge bitcoin/bitcoin#23728: Use the imperative mood in example subject line
efde11161599deb6d98fb5773c3225d589bb14c2 Use the imperative mood in example subject line (Tobin Harding) Pull request description: The section `Committing Patches` within `CONTIBUTING.md` contains an example commit subject line that violates rule seven of the linked guide to writing commit logs (Chris Beams famous blog post). We should practice what we preach, especially in examples :) Use the imperative mood in example commit message subject line. ACKs for top commit: jarolrod: ACK efde11161599deb6d98fb5773c3225d589bb14c2 shaavan: ACK efde11161599deb6d98fb5773c3225d589bb14c2 Tree-SHA512: f607951020c544a65df2fcb45f40f10d44bc761298d866ef3d3742c63ed21eed4e7a798c361f8449d05781c04133b60f0b78d4402a8235064953a817bb24c7d0
This commit is contained in:
parent
b1d3020568
commit
039357da7b
@ -40,7 +40,7 @@ fixes or code moves with actual code changes.
|
|||||||
|
|
||||||
Commit messages should be verbose by default consisting of a short subject line
|
Commit messages should be verbose by default consisting of a short subject line
|
||||||
(50 chars max), a blank line and detailed explanatory text as separate
|
(50 chars max), a blank line and detailed explanatory text as separate
|
||||||
paragraph(s), unless the title alone is self-explanatory (like "Corrected typo
|
paragraph(s), unless the title alone is self-explanatory (like "Correct typo
|
||||||
in init.cpp") in which case a single title line is sufficient. Commit messages should be
|
in init.cpp") in which case a single title line is sufficient. Commit messages should be
|
||||||
helpful to people reading your code in the future, so explain the reasoning for
|
helpful to people reading your code in the future, so explain the reasoning for
|
||||||
your decisions. Further explanation [here](https://chris.beams.io/posts/git-commit/).
|
your decisions. Further explanation [here](https://chris.beams.io/posts/git-commit/).
|
||||||
|
Loading…
Reference in New Issue
Block a user