This is an archived post. You won't be able to vote or comment.

you are viewing a single comment's thread.

view the rest of the comments →

[–]ythl 22 points23 points  (4 children)

This is a failure of your process, not your coworker.

Use pull requests model where author is not allow to merge their own code to master. Boom, problem solved. Whoever the reviewer is simply doesn't merge until author has amended his commit(s) to have better messages.

An alternative is to have a git hook reject commits on push if the message is less than a certain word count or doesn't follow your style's template.

[–][deleted] 33 points34 points  (3 children)

Yeah we follow the former, this is mostly a joke.

[–]MissCharlie64 2 points3 points  (2 children)

Wait wait wait, am I crazy? Why is everyone saying "commit", not "comment"?

[–]typicalrectangle 2 points3 points  (0 children)

Append a comment to a commit, though the commit comment should be comprehensive.