Werner Zeh has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/85915?usp=email )
Change subject: Documentation: Fix wrong link to commit message guidelines ......................................................................
Documentation: Fix wrong link to commit message guidelines
In the gerrit guidelines there is an old link pointing to the retired coreboot wiki (https://www.coreboot.org/Git#Commit_messages) when the commit message guidelines are referenced. Indeed this section was never ported over to the new documentation and is missing.
This commit rewrites this guidelines and adds them as a new section based on what was in the wiki and updates the link accordingly.
Change-Id: I1cd2b13da6fe59697d677c7350d73eda5d486544 Signed-off-by: Werner Zeh werner.zeh@siemens.com --- M Documentation/contributing/gerrit_guidelines.md 1 file changed, 43 insertions(+), 1 deletion(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/15/85915/1
diff --git a/Documentation/contributing/gerrit_guidelines.md b/Documentation/contributing/gerrit_guidelines.md index ce6fa6f..2de1a39 100644 --- a/Documentation/contributing/gerrit_guidelines.md +++ b/Documentation/contributing/gerrit_guidelines.md @@ -304,7 +304,7 @@ the files that you’ve changed, and add those people.
* Familiarize yourself with the coreboot [commit message -guidelines](https://www.coreboot.org/Git#Commit_messages), before pushing +guidelines](#commit-message-guidelines), before pushing patches. This will help to keep annoying requests to fix your commit message to a minimum.
@@ -433,8 +433,50 @@ Requests for clarification and suggestions for updates to these guidelines should be sent to the coreboot mailing list at coreboot@coreboot.org.
+Commit message guidelines +------------------------- +Git does not enforce a certain style for commit messages. For all aspects of +Git to work best with, it's important to follow these simple guidelines for +commit messages: + +- The first line of the commit message consists of a short + (less than 65 characters, absolute maximum is 75) summary +- The second line is empty (no whitespace at all) +- The third and any following number of lines contains a longer description + of the commit as is necessary, including relevant background information + and quite possibly rationale for why the issue was solved in this particular + way. These lines should never be longer than 75 characters. +- The next line is empty (no whitespace at all) +- An optional TEST= line which describes the test that was done in order to + validate the patch +- An optional BUG= line to reference to a bug (of [coreboot's bug tracker] or any + other) this patch resolves +- The next line is empty if the optional lines are used +- A Change-Id: line to let gerrit track this logical change +- A signed-off-by: line according to [Signed-off-by policy](#sign-off-procedure) + +Please do not create the Change-Id: and Signed-off-by: entries manually because +it is boring and error-prone. Instead, please install the commit-msg hook and +let the script to it for you. And remember to always use +```Bash +git commit -s +``` +to have git add your Signed-off-by: automatically. + +When you write your commit message, please keep the following two hints in mind: + +* If anyone involved in coreboot reads your comment in a year or so, she or he + shall still be able to understand what your commit is about, without the need + to analyze the code. +* Double-check that you are really committing what you think you are, e.g. by + typing the following in the top-level coreboot directory: +```Bash +git show +``` + [ready changes]: https://review.coreboot.org/q/age:1d+project:coreboot+status:open+is:mergeab... [Developer's Certificate of Origin 1.1]: https://developercertificate.org/ [Creative Commons Attribution-ShareAlike 2.5 License]: https://creativecommons.org/licenses/by-sa/2.5/ [this LKML thread]: https://lkml.org/lkml/2004/5/23/10 [SCO-Linux disputes]: https://en.wikipedia.org/wiki/SCO%E2%80%93Linux_disputes +[coreboot's bug tracker]: https://ticket.coreboot.org/