Commit 76d784b
committed
fix(pre-commit): Confine hook to commit-msg stage
The Commitizen pre-commit hook runs cz check, which only makes sense to
do on a commit message, not at other Git hook stages. By default the
hook is currently run on staged files pre-commit, which is nonsensical
unless the files in the repository themselves contain Git commit
messages. The hook is an implausible candidate even for the
prepare-commit-msg stage since this stage is intended for hooks that
modify the commit message rather than check it. Specifying the hook
stage centrally makes it less error-prone to use and saves one line of
configuration for users of the pre-commit hook.1 parent ddfb773 commit 76d784b
File tree
5 files changed
+3
-4
lines changed- commitizen/commands
- docs
- tests/commands
5 files changed
+3
-4
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
2 | 2 | | |
3 | 3 | | |
4 | 4 | | |
| 5 | + | |
5 | 6 | | |
6 | 7 | | |
7 | 8 | | |
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
114 | 114 | | |
115 | 115 | | |
116 | 116 | | |
117 | | - | |
| 117 | + | |
118 | 118 | | |
119 | 119 | | |
120 | 120 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
111 | 111 | | |
112 | 112 | | |
113 | 113 | | |
114 | | - | |
115 | 114 | | |
116 | 115 | | |
117 | 116 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
22 | 22 | | |
23 | 23 | | |
24 | 24 | | |
25 | | - | |
26 | 25 | | |
27 | 26 | | |
28 | 27 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
21 | 21 | | |
22 | 22 | | |
23 | 23 | | |
24 | | - | |
| 24 | + | |
25 | 25 | | |
26 | 26 | | |
27 | 27 | | |
| |||
0 commit comments