prepare-commit-msg: add 50 column marker
ClosedPublic
Actions

Authored by emaste on Sun, Jun 9, 5:27 PM.

Details

Reviewers

imp
lwhsu

Commits

rG87e5b17c149e: prepare-commit-msg: add 50 column marker

Summary

A common convention suggests a 50 column commit subject line (the first
line of the commit message). Add a 50 column marker to the commit
message template

Diff Detail

Lint

Lint Skipped

Unit

Tests Skipped

Event Timeline

emaste requested review of this revision.Sun, Jun 9, 5:27 PM

emaste created this revision.

How about adding a second comment line says that the 50 is for subject and 72 is for body?

This revision is now accepted and ready to land.Sun, Jun 9, 10:13 PM

How about adding a second comment line says that the 50 is for subject and 72 is for body?

I started writing some extra text but then it seems we should also comment on the blank 2nd line and it got to be a tutorial rather than a quick reference. Perhaps we could add a link to the section in the committer's guide. For now I'll commit this bit.

Closed by commit rG87e5b17c149e: prepare-commit-msg: add 50 column marker (authored by emaste). · Explain WhyMon, Jun 10, 2:49 PM

This revision was automatically updated to reflect the committed changes.

emaste added a commit: rG87e5b17c149e: prepare-commit-msg: add 50 column marker.