]> git.ipfire.org Git - thirdparty/binutils-gdb.git/commitdiff
[pre-commit] Set default_stages to pre-commit
authorTom de Vries <tdevries@suse.de>
Tue, 30 Sep 2025 16:45:44 +0000 (18:45 +0200)
committerTom de Vries <tdevries@suse.de>
Tue, 30 Sep 2025 16:45:44 +0000 (18:45 +0200)
I realized I was seeing the newly added tclint check twice:
...
$ touch gdb/testsuite/gdb.base/foo.exp
$ git add gdb/testsuite/gdb.base/foo.exp
$ git commit -a -m foo 2>&1 | grep tclint
tclint..................................................................Passed
tclint..............................................(no files to check)Skipped
$
...

The hook is run once for stage pre-commit, and once for stage commit-msg.

Since the hook doesn't specify a stage at which it's supposed to be run, it
takes its default from default_stages, which defaults to all stages.

Fix this by setting default_stages to pre-commit:
...
$ git commit -a -m foo 2>&1 | grep tclint
tclint..................................................................Passed
$
...

The only hook sofar that needs a different stage than pre-commit is
codespell-log, and it's not affected by this change because it has an explicit
"stages: [commit-msg]" setting.

Approved-By: Tom Tromey <tom@tromey.com>
.pre-commit-config.yaml

index 8217bad0a0ac716449c66a28d0b5d30ca3a1c044..33e3f2b8900f93c295baed80bd1264af087496cd 100644 (file)
@@ -39,6 +39,7 @@
 
 minimum_pre_commit_version: 3.2.0
 default_install_hook_types: [pre-commit, commit-msg]
+default_stages: [pre-commit]
 repos:
   - repo: https://github.com/psf/black-pre-commit-mirror
     rev: 25.9.0