(no title)
t8sr | 3 months ago
Asking people to fit a meaningful description of the change into 50 characters is silly, and it's IMO the reason why so many of them just write "fix bug" and call it a day.
Someone else has posted the Google guide for CL (change list) messages, but let me boost the signal: https://google.github.io/eng-practices/review/developer/cl-d...
This is, I believe, still the best guide out there. When I'm coaching juniors, I recommend this guide, over the opinionated and outdated git "best practices" and I think the results are much better.
matt_kantor|3 months ago
t8sr|3 months ago
I agree on the value of concise writing and dislike word salads, but if you're a junior engineer, then I have maybe 1 hour with you per week and I probably shouldn't spend that time being your English teacher.
Xss3|3 months ago
If a commit is sufficiently complex the long form could be 600 characters and the short form 200.
tabbott|3 months ago
When I do maintainer work that involves skimming a list of dozens of commits for information, it's very helpful to have concise commit titles.
everybodyknows|3 months ago
joedevon|3 months ago
Croak|3 months ago
[deleted]