Skip to content

chore(pre-commit): set default_install_hook_types so install arms both stages #1221

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
IAmJSD merged 1 commit into from
Oct 25, 2026
Merged

chore(pre-commit): set default_install_hook_types so install arms both stages #1221

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
19 changes: 12 additions & 7 deletions .pre-commit-config.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,6 +1,7 @@
# Pre-commit hooks configuration
# Install: pip install pre-commit
# Setup: pre-commit install
# Setup: pre-commit install (installs both the pre-commit and pre-push
# git hooks via default_install_hook_types below)

repos:
# Go formatting and module checks
Expand Down Expand Up @@ -240,12 +241,13 @@ repos:
# on every PR touching frontend/**;
# the sentinel re-runs it post-merge.
#
# 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
# 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.
# Moving them to the pre-push stage keeps the local safety net (a plain
# `pre-commit install` also arms the pre-push git hook, see
# default_install_hook_types at the bottom of this file, 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.
- repo: local
hooks:
- id: go-test
Expand Down Expand Up @@ -274,4 +276,7 @@ repos:

# Global configuration
default_stages: [pre-commit, pre-push]
# Make a plain `pre-commit install` wire both git hooks; without this it
# only installs the pre-commit hook and the pre-push stage never runs.
default_install_hook_types: [pre-commit, pre-push]
fail_fast: false
Loading