diff options
author | Marc Mutz <[email protected]> | 2025-04-22 18:37:19 +0200 |
---|---|---|
committer | Daniel Smith <[email protected]> | 2025-05-06 10:09:26 +0000 |
commit | c4da0dbfd0c0eb5ce15b2f0cca52502f8fba5007 (patch) | |
tree | 2a96c999b651691f40565703b380052eef616cc8 | |
parent | a83819f9c4eddd5b0cd5234f54427a93e59057fe (diff) |
Like for other footers, complain if
- the capitalization is wrong
- more than one QUIP is listed per line
- the QUIPs are not numerical (catching QUIP: QUIP-0004)
Unlike Coverity-Id, there is a tradition to write QUIP numbers with
leading zeros (QUIP-0004), so allow that here.
Task-number: QTQAINFRA-7033
Change-Id: I8a2302054f3205d8c1777c42f28671de79812c20
Reviewed-by: Daniel Smith <[email protected]>
-rwxr-xr-x | git-hooks/sanitize-commit | 5 |
1 files changed, 5 insertions, 0 deletions
diff --git a/git-hooks/sanitize-commit b/git-hooks/sanitize-commit index d6e3dc3..f60fcc3 100755 --- a/git-hooks/sanitize-commit +++ b/git-hooks/sanitize-commit @@ -544,6 +544,11 @@ while (<MSG>) { } } complain_ln("Capitalization of \"Pick-to\" is wrong", "") if (!/^Pick-to:/); + } elsif (/^QUIP: *(.*)/i) { + my $quip = $1; + complain_ln("Multiple QUIPs in one footer", "") if ($quip =~ /[ ,]/); + complain_ln("Non-numeric QUIP identifier", "") if ($quip =~ /^[^0-9]/); + complain_ln("Capitalization of \"QUIP\" is wrong", "") if (!/^QUIP:/); } elsif (/^Coverity-Id: /) { $cid = 1; } |