diff options
author | Greg Troxel <gdt@ir.bbn.com> | 2010-11-03 07:20:38 -0400 |
---|---|---|
committer | Greg Troxel <gdt@ir.bbn.com> | 2010-11-03 07:20:38 -0400 |
commit | d7a9779742748f0658be22664349669438e69af1 (patch) | |
tree | dbd2886e08dfd83312ed46028d8cb5d6f579d55f | |
parent | f3bd05011ceb5173c2b01d2d51616fabe4860f57 (diff) |
infrastructure: Clarify commit message format.
Note 54/72 line length rules, and that this is intended to play nice
with "git log --oneline".
-rw-r--r-- | HACKING | 8 |
1 files changed, 6 insertions, 2 deletions
@@ -125,8 +125,8 @@ COMMIT MESSAGES The commit message should provide: -* A suitable one-line summary as the very first line of the message, in the - form: +* A suitable one-line summary followed by a blank line as the very + first line of the message, in the form: topic: high-level, one line summary @@ -143,6 +143,8 @@ to provide a short description of the general intent of the patch, in terms of the problem it solves and how it achieves it, to help reviewers understand. +The one-line summary must be limited to 54 characters, and all other +lines to 72 characters. The reason for such itemised commit messages is to encourage the author to self-review every line of the patch, as well as provide reviewers an index @@ -162,6 +164,8 @@ zebra: Enhance frob FSM to detect loss of frob * frob.c: (frob_change) set/clear DOWN appropriately on state change. * bar.c: (barinate) Check frob for DOWN state. +Note that the commit message format follows git norms, so that "git +log --oneline" will have useful output. HACKING THE BUILD SYSTEM |