diff options
author | Oswald Buddenhagen <oswald.buddenhagen@theqtcompany.com> | 2015-01-23 14:45:25 +0100 |
---|---|---|
committer | Oswald Buddenhagen <oswald.buddenhagen@theqtcompany.com> | 2015-01-28 14:40:19 +0000 |
commit | 995f2c3b42eb9375236c8510ad68da5d069bb941 (patch) | |
tree | 3a389cb10e9c1f5c334546cb4bd37f03246ae990 /git-hooks | |
parent | d615ea50a0ab4e9657284573f4c7e31e747adeca (diff) |
make the w.i.p. detector a bit more strict
specifically, don't complain about references to wip/ branches.
also, don't be excessively case-insensitive .
Change-Id: I0fa1ee32d8c8381dad42a67e03e89ad346fbdff7
Reviewed-by: Orgad Shaneh <orgads@gmail.com>
Diffstat (limited to 'git-hooks')
-rwxr-xr-x | git-hooks/sanitize-commit | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/git-hooks/sanitize-commit b/git-hooks/sanitize-commit index bc35314..0b8a348 100755 --- a/git-hooks/sanitize-commit +++ b/git-hooks/sanitize-commit @@ -250,7 +250,7 @@ while (<MSG>) { if (/\bQT[A-Z]+-\d+\b/) { complain_cln("Bug reference in summary", "log"); } - if (!$iswip && $parents < 2 && /\bWIP\b|\*{3}|^(?:squash|fixup)! |^(.)\1*$/i) { + if (!$iswip && $parents < 2 && /\b(?:WIP|Wip|wip(?!\/))\b|\*{3}|^(?:squash|fixup)! |^(.)\1*$/) { complain_cln("Apparently pushing a Work In Progress", "wip", 1); } elsif (!$iswip && !$badlog && length($_) < 7) { complain_cln("Summary is too short", "log"); |