Differences
This shows you the differences between two versions of the page.
| Both sides previous revision Previous revision Next revision | Previous revision | ||
| commitmessageformat [2007/06/15 16:13] – AndrewPantyukhin | commitmessageformat [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' | There' | ||
| ==== Overview ==== | ==== Overview ==== | ||
| < | < | ||
| - | - 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 |
| </ | </ | ||
| Line 32: | Line 39: | ||
| * If committer==submitter, | * If committer==submitter, | ||
| * 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: | ||
| < | < | ||
| - | PR: ports/12345 | + | PR: 12345 |
| Approved by: John Doe < | Approved by: John Doe < | ||
| Approved by: phk (maintainer, | Approved by: phk (maintainer, | ||
| Approved by: portmgr (erwin) | Approved by: portmgr (erwin) | ||
| - | Approved by: Dow Jones (maitainer | + | Approved by: Dow Jones (maintainer |
| Submitted by: Jane Doe < | Submitted by: Jane Doe < | ||
| </ | </ | ||
| Line 54: | Line 61: | ||
| < | < | ||
| - | PR: 12345 | ||
| Approved by: maintainer | Approved by: maintainer | ||
| Approved by: maintainer timeout | Approved by: maintainer timeout | ||
| Line 76: | Line 82: | ||
| | Abc doesn' | | Abc doesn' | ||
| | 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: | ||
| + | |||
| + | < | ||
| + | - Update to 1.5 [1] | ||
| + | - Replace deprecated variable: INSTALLS_SHLIB [2] | ||
| + | - Use PORTDOCS to simplify pkg-plist [2] | ||
| + | |||
| + | PR: ports/ | ||
| + | Submitted by: Antoine Brodin < | ||
| + | Approved by: | ||
| + | </ | ||
| ==== Standard text excerpts ==== | ==== Standard text excerpts ==== | ||
| Line 90: | Line 108: | ||
| - Add rc script | - Add rc script | ||
| </ | </ | ||
| + | |||
| + | ---- | ||
| + | |||
| + | {{tag> | ||
| + | {{tag> | ||