diff options
author | Oswald Buddenhagen <oswald.buddenhagen@gmx.de> | 2021-05-20 21:10:59 +0200 |
---|---|---|
committer | Oswald Buddenhagen <oswald.buddenhagen@gmx.de> | 2021-05-21 09:29:29 +0000 |
commit | 63796e0140c471b95ec05490854a3db29adc83da (patch) | |
tree | 2a31ea0995b7661e33ff43f218c387b8b3099cdf | |
parent | a88102f7a1e68e0f84ce82df3696a40004793d9e (diff) | |
download | qtrepotools-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-x | git-hooks/sanitize-commit | 10 |
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); } } |