-
Notifications
You must be signed in to change notification settings - Fork 555
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
Capitalisation in examples #83
Comments
@robcresswell thank you for opening the issue! We just merged a PR about capitalization in the commit type. I agree with your idea, but think that this should not care about it, because the changelogs tools may do it on the fly while generating it. We may change #82 to be more generic maybe. wdyt about it? |
Yeah, as noted in #24, I think the type and the message itself should be capitalized so easier readability. But I can't tell if there's any advantage/disadvantage for the tooling around this. |
Ah that's sensible reasoning, to not be opinionated about casing at this level of the spec and leave it up to parsers. Thanks for the feedback; seems like something I should open in the tools themselves 😃 |
It's right to discuss about it, and we may enforce it to be capitalized if the community want it and it's reasonable ;) |
Is there an official parser? |
I understand casing should not be an issue for parser, but still capital after the Since the Conventional Commits page is a reference for people looking to write tidy commits, using capitalization in given examples would be better, I think. |
@damianopetrungaro may I draw your attention on this topic again? Considering the three authority sources I referred to in my previous post, un-capitalized examples seem to me... pretty unconventional! Maybe re-open this issue? If it would be accepted and it could help, I would do a pull request but I'm unsure about the project structure; all files in |
The examples you linked to recommend the capitalisation of commit messages/commit subjects, they don't recommend what you seem to be suggesting, i.e. the capitalisation of some other word that appears later in the sentence. Conventional commits "naturally" decrease readability because their defaulting to lowercase goes against English orthography, which the recommendation by these often cited older sources is obviously based on. Capitalising whatever comes after the colon would only make matters worse because it'd break orthographic convention a second or even third time within one short sentence. |
Sorry, should perhaps have tagged you, @tayflo. ^ |
@keikoro Sorry, I forgot to answer. For me the tag before the colon is a tag, i.e. a label for a category, and hence not a part of the sentence, which starts right after the colon, usually with a verb describing the action taken. The colon is here used as a semantic separator, not as it would be in a sentence following proper English grammar. To me it equals something like Hence, it makes more sense to capitalize the character right after the colon. And it also makes sense to me not to put a dot at the end because it is also a kind of title (in addition to saving another character). |
This is minor I suppose, but shouldn't the examples use capitals after the
:
? Given that text is parsed into aCHANGELOG
and each commit title is supposed to be a standalone sentence, would it make sense to capitalise the first letter of each explanation?For example, instead of:
use:
Then the generated changelogs would use proper grammar, I think?
The text was updated successfully, but these errors were encountered: