commitmessageformat

Differences

This shows you the differences between two versions of the page.

Link to this comparison view

Both sides previous revision Previous revision
Next revision
Previous revision
commitmessageformat [2007/06/15 16:13] – AndrewPantyukhincommitmessageformat [2022/11/27 23:09] (current) – Overview: update with Git in mind. GrahamPerrin
Line 1: Line 1:
 ===== Commit Message Format ===== ===== Commit Message Format =====
 +**This page is not authoritative. It is rather a collection of reasonable suggestions.** Feel free to contribute to this page.
 +
 There's little technical merit to a standardized commit log format, except for much easier grepping. What we should value here is tradition. There's little technical merit to a standardized commit log format, except for much easier grepping. What we should value here is tradition.
 ==== Overview ==== ==== Overview ====
  
 <code> <code>
-- Did that +Summary, ~50 characters, no trailing punctuation 
-- And that+ 
 +This, that. A blank line between the summary line, and this first  
 +piece of free-form text. Hard wrap at 72 characters. 
 + 
 +The other. Again, 72 characters. A blank line between this final  
 +piece of free-form text, and the colon-separated fields. 
  
-Field one: value +First field:  value 
-Field two: value+Second field: value
 </code> </code>
  
Line 32: Line 39:
   * If committer==submitter, the "Submitted by" field may be omitted   * If committer==submitter, the "Submitted by" field may be omitted
   * Don't use pronouns in field values   * Don't use pronouns in field values
-  * Use a spell-chekcer+  * Use a spell-checker
   * Limit characters to US-ASCII   * Limit characters to US-ASCII
   * Use present tense   * Use present tense
Line 43: Line 50:
  
 <code> <code>
-PR:           ports/12345+PR:           12345
 Approved by:  John Doe <john@doe.org> (maintainer) Approved by:  John Doe <john@doe.org> (maintainer)
 Approved by:  phk (maintainer, implicit) Approved by:  phk (maintainer, implicit)
 Approved by:  portmgr (erwin) Approved by:  portmgr (erwin)
-Approved by:  Dow Jones (maitainer timeout, 25 days)+Approved by:  Dow Jones (maintainer timeout, 25 days)
 Submitted by: Jane Doe <jane@doe.org> Submitted by: Jane Doe <jane@doe.org>
 </code> </code>
Line 54: Line 61:
  
 <code> <code>
-PR:           12345 
 Approved by:  maintainer Approved by:  maintainer
 Approved by:  maintainer timeout Approved by:  maintainer timeout
Line 76: Line 82:
 | Abc doesn't work, fix it! | Reported by: jack | | Abc doesn't work, fix it! | Reported by: jack |
 | Here's a patch | Submitted by: jack | | Here's a patch | Submitted by: jack |
 +
 +  * A maintainer sends a PR. You want to commit it with additional changes. The maintainer approves them:
 +
 +<code>
 +- Update to 1.5 [1]
 +- Replace deprecated variable: INSTALLS_SHLIB [2]
 +- Use PORTDOCS to simplify pkg-plist [2]
 +
 +PR:            ports/113760 [1]
 +Submitted by:  Antoine Brodin <brodin@leposte.net> (maintainer) [1]
 +Approved by:   Antoine Brodin <brodin@leposte.net> (maintainer) [2]
 +</code>
  
 ==== Standard text excerpts ==== ==== Standard text excerpts ====
Line 90: Line 108:
 - Add rc script - Add rc script
 </code> </code>
 +
 +----
 +
 +{{tag>HowTo}}
 +{{tag>Stale}}
  
  • commitmessageformat.1181923986.txt.gz
  • Last modified: 2007/06/15 16:13
  • by AndrewPantyukhin