mirror of
https://github.com/conventional-commits/conventionalcommits.org.git
synced 2026-05-23 14:20:55 -05:00
Improve example "Commit message with multi-paragraph body and multiple footers" #122
Reference in New Issue
Block a user
Delete Branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Originally created by @shaman-apprentice on GitHub (Mar 8, 2021).
As a developer I want commit messages to be succinct and readable. As a user of Conventional Commits I want the examples to be clear and inspiring to use the guidelines.
I had to read the given example below Commit message with multi-paragraph body and multiple footers multiple times to understand, that both bodies are one connected sentence.
I think it is more intuitive to write the two bodies as one, which is more readable than splitting one sentence over 3 lines.
What do you think about updating the example, so that each body is a sentence with a separate meaningful concern by itself? Maybe:
@derekmurawsky commented on GitHub (Sep 23, 2021):
I second this, and would take it a step further by possibly incorporating some of the ideas around what makes a good commit message. I have used this blog post by Chis Beams as the basis of my commit messages for a while now and have found it to be an excellent pattern. The key aspects are:
That last point, number 7, is huge to me. If nothing else is incorporated to conventional commits, I would encourage you consider that. Issues trackers can change over time. Commit histories are, generally, forever.
@damianopetrungaro commented on GitHub (Sep 28, 2021):
@shaman-apprentice I agree with you, do you wanna open a PR for that?
@damianopetrungaro commented on GitHub (Sep 28, 2021):
@derekmurawsky At the moment the convention is totally open to provide support to most of the tooling around, I agree with you that those are really good rules for writing a commit message, but still, I'd consider those not mandatory for the current state of the specification.
@derekmurawsky commented on GitHub (Sep 30, 2021):
Thanks for the reply!
To clarify, I meant that we should either reference the other work or improve the example. I did not mean it should be included in the standard itself.
And I don't mean to speak for @shaman-apprentice - But I believe they were referring to the examples as well.
@damianopetrungaro commented on GitHub (Sep 30, 2021):
Then it’s totally ok! If you want you can open a PR and fix that! I’d love
it :)
On Thu, 30 Sep 2021 at 16:06, Derek Murawsky @.***>
wrote:
@shaman-apprentice commented on GitHub (Oct 1, 2021):
I am just writing quickly from train and I don't want to rush. Thanks for your comments! I will gladly create a PR in the next days.