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/16 18:22] – Insert a reminder: this page is not a policy. AndrewPantyukhincommitmessageformat [2022/11/27 23:09] (current) – Overview: update with Git in mind. GrahamPerrin
Line 6: Line 6:
  
 <code> <code>
-- Did that +Summary, ~50 characters, no trailing punctuation
-- And that+
  
-Field one: value +This, that. A blank line between the summary line, and this first  
-Field two: value+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.  
 + 
 +First field:  value 
 +Second field: value
 </code> </code>
  
Line 34: 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 45: 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 56: Line 61:
  
 <code> <code>
-PR:           12345 
 Approved by:  maintainer Approved by:  maintainer
 Approved by:  maintainer timeout Approved by:  maintainer timeout
Line 78: 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 92: Line 108:
 - Add rc script - Add rc script
 </code> </code>
 +
 +----
 +
 +{{tag>HowTo}}
 +{{tag>Stale}}
  
  • commitmessageformat.1182018173.txt.gz
  • Last modified: 2007/06/16 18:22
  • by AndrewPantyukhin