Commit messages, also known as CL (changelist) descriptions, should be formatted per https://tip.golang.org/doc/contribute.html#commit_messages . For example,
net/http: frob the quux before blarfing [longer description here in the body] Fixes #nnnn
Notably, for the subject (the first line of description):
For the body (the rest of the description):
Signed-off-by
lines. Don't add them. Our Gerrit server & GitHub bots enforce CLA compliance instead.If it's not a complete fix and more is coming, use:
Updates #nnnn
instead of Fixes
. Don't use the other GitHub-supported verbs.
For non-“go” repos (“crypto”, “tools”, “net”, etc), the subject is still the name of the package, but you need to fully-qualify the issue number with the GitHub org/repo syntax:
cipher/rot13: add new super secure cipher Fixes golang/go#1234
Notably, the first line subject should not contain the x/crypto/
prefix. We only do that for the issue tracker.
If you‘re using GitHub Pull Requests, your commit message is constructed by GerritBot based on your PR’s title & description. See https://github.com/golang/go/wiki/GerritBot#how-does-gerritbot-determine-the-final-commit-message
If somebody asks you to modify your commit message, you'll need to modify your PR.