| ▲ | akersten 3 hours ago | |||||||
The author's example of a conventional commit is not correct anyway IMO, which is maybe why they think the "fix" part is redundant: > fix: prevent foo from bar'ing The whole idea of conventional commit is: > fix: [problem] so the correct conventional commit would be: > fix: foo bar'ing which is succinct and perfectly fine. | ||||||||
| ▲ | 0x457 2 hours ago | parent | next [-] | |||||||
Whole idea of CC is to write commits in away that is easy to generate change logs. If you utilize CC correctly or not, at best you get: commit log that harder to read, changelog that is hard to read and still requires you to write highlights (guess minor and path releases are fine). | ||||||||
| ▲ | jacobsenscott an hour ago | parent | prev | next [-] | |||||||
None of this is as good as "free array memory before it goes out of scope". This is better than `fix: memory leak` - which is what most people would do. It's also better than `fix: free array memory...` because `fix:` is redundant when you have a good message. I get people want to build automation around this stuff, but just do that in a footer of a commit message where humans don't need to see it. | ||||||||
| ▲ | chrismorgan 3 hours ago | parent | prev | next [-] | |||||||
What you describe doesn’t match <https://www.conventionalcommits.org/en/v1.0.0/>’s examples, or any practice I’ve ever seen. > fix: prevent racing of requests Though the example in the actual specification, “fix: array parsing issue when multiple spaces were contained in string”, is more inconclusive (and frankly doesn’t really make sense as a description). | ||||||||
| ||||||||
| ▲ | SebastianKra 3 hours ago | parent | prev [-] | |||||||
yep. I'm on the fence about types generally, but "fix:" saves/standardizes a bunch of phrases like "fix an issue where", "prevent" or having to invert the message by describing the solution instead. | ||||||||
| ||||||||