From a4c7de371d6f82e12b4c1a9b544ce763be3cf74c Mon Sep 17 00:00:00 2001 From: Cristian Magherusan-Stanciu Date: Fri, 16 Oct 2026 02:39:28 +0200 Subject: [PATCH] chore(pre-commit): set default_install_hook_types so install arms both stages default_stages already lists pre-push, but without default_install_hook_types a plain `pre-commit install` wires only the pre-commit git hook, so the pre-push go-test stage silently never ran locally. Verified in a scratch repo: the old config installs only pre-commit, the new one installs pre-commit and pre-push. Closes #10 Co-Authored-By: Claude Opus 5.5 --- .pre-commit-config.yaml | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/.pre-commit-config.yaml b/.pre-commit-config.yaml index 4c3e422..c074845 100644 --- a/.pre-commit-config.yaml +++ b/.pre-commit-config.yaml @@ -247,9 +247,9 @@ repos: # on every push to feat/** (which # fires on every PR-branch update). # - # Moving them to the pre-push stage keeps the local safety net (devs - # who run `pre-commit install --hook-type pre-push` still get these - # tests on `git push`) while letting the CI pre-commit workflow stay + # Moving them to the pre-push stage keeps the local safety net (a plain + # `pre-commit install` arms pre-push via default_install_hook_types, so + # devs still get these tests on `git push`) while letting the CI pre-commit workflow stay # focused on style/security/syntax. Pre-commit's default stage filter # is `pre-commit`, so the CI workflow's `pre-commit run --all-files` # skips these hooks automatically. @@ -281,4 +281,5 @@ repos: # Global configuration default_stages: [pre-commit, pre-push] +default_install_hook_types: [pre-commit, pre-push] fail_fast: false