See how a minor change to your commit message style can make a difference.
Tip
Have a look at git-conventional-commits , a CLI util to ensure these conventions, determine version and generate changelogs
<type>(<optional scope>): <description> empty separator line <optional body> empty separator line <optional footer>
Merge branch '<branch name>'
Follows default git merge message
Revert "<reverted commit subject line>"
Follows default git revert message
chore: init
- API or UI relevant changes
feat
Commits, that add or remove a new feature to the API or UIfix
Commits, that fix a API or UI bug of a precededfeat
commit
refactor
Commits, that rewrite/restructure your code, however do not change any API or UI behaviourperf
Commits are specialrefactor
commits, that improve performance
style
Commits, that do not affect the meaning (white-space, formatting, missing semi-colons, etc)test
Commits, that add missing tests or correcting existing testsdocs
Commits, that affect documentation onlybuild
Commits, that affect build components like build tool, ci pipeline, dependencies, project version, ...ops
Commits, that affect operational components like infrastructure, deployment, backup, recovery, ...chore
Miscellaneous commits e.g. modifying.gitignore
The scope
provides additional contextual information.
- Is an optional part of the format
- Allowed Scopes depends on the specific project
- Don't use issue identifiers as scopes
Breaking changes should be indicated by an !
before the :
in the subject line e.g. feat(api)!: remove status endpoint
- Is an optional part of the format
- Breaking changes must be described in the commit footer section
The description
contains a concise description of the change.
- Is a mandatory part of the format
- Use the imperative, present tense: "change" not "changed" nor "changes"
- Think of
This commit will...
orThis commit should...
- Think of
- Don't capitalize the first letter
- No dot (
.
) at the end
The body
should include the motivation for the change and contrast this with previous behavior.
- Is an optional part of the format
- Use the imperative, present tense: "change" not "changed" nor "changes"
- This is the place to mention issue identifiers and their relations
The footer
should contain any information about Breaking Changes and is also the place to reference Issues that this commit refers to.
- Is an optional part of the format
- optionally reference an issue by its id.
- Breaking Changes should start with the word
BREAKING CHANGE:
followed by space or two newlines. The rest of the commit message is then used for this.
- If your next release contains commit with...
- breaking changes incremented the major version
- API relevant changes (
feat
orfix
) incremented the minor version
- Else increment the patch version
-
feat: add email notifications on new direct messages
-
feat(shopping cart): add the amazing button
-
feat!: remove ticket list endpoint refers to JIRA-1337 BREAKING CHANGE: ticket enpoints no longer supports list all entites.
-
fix(shopping-cart): prevent order an empty shopping cart
-
fix(api): fix wrong calculation of request body checksum
-
fix: add missing parameter to service call The error occurred because of <reasons>.
-
perf: decrease memory footprint for determine uniqe visitors by using HyperLogLog
-
build: update dependencies
-
build(release): bump version to 1.0.0
-
refactor: implement fibonacci number calculation as recursion
-
style: remove empty line
Click to expand
- Create a commit-msg hook using git-conventional-commits cli
- create following file in your repository folder
.git/hooks/pre-receive
#!/usr/bin/env bash # Pre-receive hook that will block commits with messges that do not follow regex rule commit_msg_type_regex='feat|fix|refactor|style|test|docs|build' commit_msg_scope_regex='.{1,20}' commit_msg_description_regex='.{1,100}' commit_msg_regex="^(${commit_msg_type_regex})(\(${commit_msg_scope_regex}\))?: (${commit_msg_description_regex})\$" merge_msg_regex="^Merge branch '.+'\$" zero_commit="0000000000000000000000000000000000000000" # Do not traverse over commits that are already in the repository excludeExisting="--not --all" error="" while read oldrev newrev refname; do # branch or tag get deleted if [ "$newrev" = "$zero_commit" ]; then continue fi # Check for new branch or tag if [ "$oldrev" = "$zero_commit" ]; then rev_span=`git rev-list $newrev $excludeExisting` else rev_span=`git rev-list $oldrev..$newrev $excludeExisting` fi for commit in $rev_span; do commit_msg_header=$(git show -s --format=%s $commit) if ! [[ "$commit_msg_header" =~ (${commit_msg_regex})|(${merge_msg_regex}) ]]; then echo "$commit" >&2 echo "ERROR: Invalid commit message format" >&2 echo "$commit_msg_header" >&2 error="true" fi done done if [ -n "$error" ]; then exit 1 fi
- ⚠ make
.git/hooks/pre-receive
executable (unix:chmod +x '.git/hooks/pre-receive'
)
- https://www.conventionalcommits.org/
- https://github.com/angular/angular/blob/master/CONTRIBUTING.md
- http://karma-runner.github.io/1.0/dev/git-commit-msg.html
I strongly recommend to use
feat
orfix
if it changes the API even just a tiny bit. That is the most important information for a commit.I would recommend to use
refactor
because using abbreviation always lead to less readability afterwards. Saving 4 characters will not have a measurable impact on development. However feel free to use another convention in your projects. The only relevant types arefix
andfeat
I know this is an abbreviation, I don't like it however it's part of the https://www.conventionalcommits.org/en/v1.0.0/ definition and I like to stick to standards.I don't see any reason or additional value in splitting refactoring up in to more granular refactoring types. This will only make it mor difficult to choose between all the types.
If it adds value for your project go for it. In my experience
ci
can be better described asops
related commit if you do a deployment relevant change ortest
if you add something to improve testing before deploying orbuild
if your commit the build of your artifacts.