diff options
author | Oswald Buddenhagen <oswald.buddenhagen@theqtcompany.com> | 2015-04-10 19:18:26 +0200 |
---|---|---|
committer | Oswald Buddenhagen <oswald.buddenhagen@theqtcompany.com> | 2015-04-13 12:05:42 +0000 |
commit | f05180acbd859f63bb8424a7f5f85b398c7fc754 (patch) | |
tree | 62b5368fe0f7bca76013f610f8b1f9395fce03fe /git-hooks/gerrit-bot | |
parent | b27358073f18c43b48bd279c89ee13380648aa7e (diff) |
make work-in-progress marker match even stricter
observed conventions: the marker is typically used at the beginning of
the summary, optionally in square (or angle) brackets, optionally
followed by a colon - all-lowercase, all-uppercase, or capitalized.
alternatively, it's used somewhere else in the summary, but always in
all-uppercase.
Change-Id: I52b3eb2162f281450aebe6b770ba4639ed1557b2
Reviewed-by: Orgad Shaneh <orgads@gmail.com>
Diffstat (limited to 'git-hooks/gerrit-bot')
-rwxr-xr-x | git-hooks/gerrit-bot | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/git-hooks/gerrit-bot b/git-hooks/gerrit-bot index 6b4bb82..db2aac3 100755 --- a/git-hooks/gerrit-bot +++ b/git-hooks/gerrit-bot @@ -212,7 +212,7 @@ sub process_commit($$$$$) $subject = `git show --pretty=\%s -s $rev`; chop($subject); } - if ($subject =~ /\b(?:WIP|Wip|wip(?!\/))\b|\*{3}|^(?:squash|fixup)! |^(.)\1*$/) { + if ($subject =~ /^[<[]?[Ww]ip\b|\bWIP\b|\*{3}|^(?:squash|fixup)! |^(.)\1*$/) { $iswip = 1; $message = "Apparently pushing a Work In Progress\n\n".$message; } |