Increase the quality of your commits with pre-commit
pre-commit [1] is a framework for managing and maintaining pre-commit hooks for git. By running hooks before any commit, many small pitfalls could be avoided before being pushed and will spare reviewers time and energy.
Such hooks could for example check that commit messages follow a specific format or that the code pass a lint test for a specific type of file.
The hooks you use is specified in a .pre-commit-config.yaml file that you could include into your git repository. This makes it easy to bind the hooks to the project and share between all the developers.
Available hooks
pre-commit have a range of supported hooks, here is a list of those hooks that is currently part of the repository:
- check-added-large-files - prevents giant files from being committed.
- check-ast - simply checks whether the files parse as valid python.
- check-byte-order-marker - forbids files which have a utf-8 byte-order marker.
- check-builtin-literals - requires literal syntax when initializing empty or zero python builtin types.
- check-case-conflict - checks for files that would conflict in case-insensitive filesystems.
- check-docstring-first - checks a common error of defining a docstring after code.
- check-executables-have-shebangs - ensures that (non-binary) executables have a shebang.
- check-json - checks json files for parseable syntax.
- check-shebang-scripts-are-executable - ensures that (non-binary) files with a shebang are executable.
- pretty-format-json - sets a standard for formatting json files.
- check-merge-conflict - checks for files that contain merge conflict strings.
- check-symlinks - checks for symlinks which do not point to anything.
- check-toml - checks toml files for parseable syntax.
- check-vcs-permalinks - ensures that links to vcs websites are permalinks.
- check-xml - checks xml files for parseable syntax.
- check-yaml - checks yaml files for parseable syntax.
- debug-statements - checks for debugger imports and py37+ breakpoint() calls in python source.
- destroyed-symlinks - detects symlinks which are changed to regular files with a content of a path which that symlink was pointing to.
- detect-aws-credentials - detects your aws credentials from the aws cli credentials file.
- detect-private-key - detects the presence of private keys.
- double-quote-string-fixer - replaces double quoted strings with single quoted strings.
- end-of-file-fixer - ensures that a file is either empty, or ends with one newline.
- file-contents-sorter - sorts the lines in specified files (defaults to alphabetical). you must provide list of target files as input in your .pre-commit-config.yaml file.
- fix-byte-order-marker - removes utf-8 byte order marker.
- fix-encoding-pragma - adds # -- coding: utf-8 -- to the top of python files.
- forbid-new-submodules - prevents addition of new git submodules.
- forbid-submodules - forbids any submodules in the repository
- mixed-line-ending - replaces or checks mixed line ending.
- name-tests-test - verifies that test files are named correctly.
- no-commit-to-branch - don't commit to branch
- requirements-txt-fixer - sorts entries in requirements.txt.
- sort-simple-yaml - sorts simple yaml files which consist only of top-level keys, preserving comments and blocks.
- trailing-whitespace - trims trailing whitespace.
It also has tons of supported hooks from third-parties. See [2] for a full list.
Quick start
The webpage [1] does a terrific job to describe this, so I will only briefly go through the steps.
Installation
pre-commit is available in most of the package managers out there. I use pacman as I'm hooked (phun intended) to Archlinux.
1pacman install pre-commit
pre-commit configuration
pre-commit use a configuration file in order to know wich hooks to use. In your repository, generate a basic configuration file using pre-commit sample-config:
1pre-commit sample-config > .pre-commit-config.yaml
The default configuration file looks as follow:
1 repos:
2 - repo: https://github.com/pre-commit/pre-commit-hooks
3 rev: v3.2.0
4 hooks:
5 - id: trailing-whitespace
6 - id: end-of-file-fixer
7 - id: check-yaml
8 - id: check-added-large-files
Install hooks
When we've the configuration file in place, install the hooks and we're ready to go:
1pre-commit install
Test it
Once the hooks are installed, we can make a test-commit:
As we can see, several tests are performed before the commit are made. It won't allow you to commit if any of the pass does not pass.
Git template directory
This is great, we now have a way to specify which hooks we want to run for a certain repository! But what if we want to use these hooks for all of our repositores?
Git supports a template directory [3], which basically is a directory with files that will be copied to $GIT_DIR after a repository is created (which includes both new and cloned repositories).
You can do this in several ways, even per repo by the --template option, but the global init.templateDir configuration variable is how I prefer to do it.
First, create a home for your git-template and tell git to use it:
1mkdir $HOME/.git-template
2git config --global init.templatedir '~/.git-template'
Then create a pre-commit configuration file and let pre-commit init-templatedir populate the directory as a templatedir:
1pre-commit sample-config > $HOME/.git-template/.pre-commit-config.yaml
2pre-commit init-templatedir -c $HOME/.git-template/.pre-commit-config.yaml $HOME/.git-template
Now these hooks will be applied to all repositories you will clone or create. Test it out:
1$ git init
2$ echo 'echo $PATH' > test.sh
3$ git add test.sh
4$ git commit -m "add test.sh"
5Trim Trailing Whitespace.................................Passed
6Fix End of Files.........................................Passed
7Check Yaml...........................(no files to check)Skipped
8Check for added large files..............................Passed
9shellcheck...............................................Failed
10- hook id: shellcheck
11- exit code: 1
12
13In test.sh line 1:
14echo $PATH
15^-- SC2148 (error): Tips depend on target shell and yours is unknown. Add a shebang or a 'shell' directive.
16 ^---^ SC2086 (info): Double quote to prevent globbing and word splitting.
17
18 Did you mean:
19 echo "$PATH"
20
21 For more information:
22 https://www.shellcheck.net/wiki/SC2148 -- Tips depend on target shell and y...
23 https://www.shellcheck.net/wiki/SC2086 -- Double quote to prevent globbing ...
As you can see, I have a few things to fix :-)
My pre-commit config file
My configuration file does not differ much from the default one.
In additional to the default config file, I've added shellcheck to check my shell-scripts: :
1 repos:
2 - repo: https://github.com/pre-commit/pre-commit-hooks
3 rev: v3.2.0
4 hooks:
5 - id: trailing-whitespace
6 - id: end-of-file-fixer
7 - id: check-yaml
8 - id: check-added-large-files
9 - repo: https://github.com/shellcheck-py/shellcheck-py
10 rev: v0.10.0.1
11 hooks:
12 - id: shellcheck