Hacker News new | ask | show | jobs
by idlephysicist 896 days ago
I totally agree, I have seen people get so fixated on this in addition to:

- review feedback format

- variable naming

- keeping line lengths to 80 lines

- log message formats (sweet jesus the amount of time wasted debating that alone)

Whenever I see these conventions about commit formatting they generally seem to focus on the first line (notable exceptions being Git and the Linux kernel among others I assume), one line is rarely enough to describe the change – I’m not advocating for writing an essay for a 5 line change though in some cases it might be warranted.

Professionally I rarely see useful commit messages, by useful I mean something that could be read by someone without context and get a general understanding of why a change was made. Frequently I have seen “updates” / “wip” etc. in the master branch. More frequently I see that the quality of commit message decreases with an engineer’s seniority – though like every thing there are exceptions.