summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorMarc Mutz <[email protected]>2025-04-22 18:37:19 +0200
committerDaniel Smith <[email protected]>2025-05-06 10:09:26 +0000
commitc4da0dbfd0c0eb5ce15b2f0cca52502f8fba5007 (patch)
tree2a96c999b651691f40565703b380052eef616cc8
parenta83819f9c4eddd5b0cd5234f54427a93e59057fe (diff)
sanitize-commit: Validate QUIP footersHEADmaster
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-xgit-hooks/sanitize-commit5
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;
}