summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorOswald Buddenhagen <oswald.buddenhagen@gmx.de>2021-05-20 21:10:59 +0200
committerOswald Buddenhagen <oswald.buddenhagen@gmx.de>2021-05-21 09:29:29 +0000
commit63796e0140c471b95ec05490854a3db29adc83da (patch)
tree2a31ea0995b7661e33ff43f218c387b8b3099cdf
parenta88102f7a1e68e0f84ce82df3696a40004793d9e (diff)
downloadqtrepotools-63796e0140c471b95ec05490854a3db29adc83da.tar.gz
sanitize-commit: remove override keywords from hints
hints need no overrides by definition. Change-Id: Idd12ac9d2de56f0ab0c9939bf32814a066fa8d32 Reviewed-by: Tor Arne Vestbø <tor.arne.vestbo@qt.io>
-rwxr-xr-xgit-hooks/sanitize-commit10
1 files changed, 5 insertions, 5 deletions
diff --git a/git-hooks/sanitize-commit b/git-hooks/sanitize-commit
index 98f8ce5..71a1afd 100755
--- a/git-hooks/sanitize-commit
+++ b/git-hooks/sanitize-commit
@@ -197,7 +197,7 @@ sub styleFail($)
{
my $why = shift;
if ($gerrit_rest) {
- complain_ln("$why", "style", -1);
+ complain_ln("$why", "", -1);
} else {
push @style_fails, " $lineno: ".$why;
}
@@ -206,7 +206,7 @@ sub styleFail($)
sub complain_style()
{
if (@style_fails) {
- do_complain(1e9, "Style issues", "style", -1, @style_fails);
+ do_complain(1e9, "Style issues", "", -1, @style_fails);
@style_fails = ();
}
}
@@ -785,7 +785,7 @@ while (<DIFF>) {
$need_hash_bang = 0;
} else {
# Can still be valid if run via the right interpreter, so level -1:
- complain("Executable file with no initial #! line", "permissions", -1);
+ complain("Executable file with no initial #! line", "", -1);
}
}
$in_plus = 1;
@@ -976,13 +976,13 @@ if ($mixws_check) {
for (sort keys %ws_lines) {
$file = $_;
if ($gerrit_rest) {
- do_complain($_, "WS-only change", "mixws", -1) foreach (@{$ws_lines{$file}});
+ do_complain($_, "WS-only change", "", -1) foreach (@{$ws_lines{$file}});
} else {
push @extras, "WS-only in ".$file.": ".join(", ", @{$ws_lines{$file}});
}
}
$file = "~~~~~";
- do_complain(1e9, "Mixing whitespace-only changes with other changes", "mixws", -1, @extras);
+ do_complain(1e9, "Mixing whitespace-only changes with other changes", "", -1, @extras);
}
}