From b93cc581a5d6061c1e063a51c8d327f4d3cbe199 Mon Sep 17 00:00:00 2001 From: Frantisek Hrbata Date: Thu, 14 Dec 2023 12:17:03 +0100 Subject: [PATCH] fix: set default pre-commit hook stages to pre-commit only If pre-commit hook does not specify stage[1], neither in .pre-commit-config.yaml nor in .pre-commit-hooks.yaml, it's started for every installed pre-commit hook. Limit the default stages to pre-commit only by default. This also sets the default_install_hook_types and minimum_pre_commit_version and sets conventional-precommit-linter to v1.4.1. [1] https://pre-commit.com/#confining-hooks-to-run-at-certain-stages Signed-off-by: Frantisek Hrbata --- .pre-commit-config.yaml | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/.pre-commit-config.yaml b/.pre-commit-config.yaml index 2333765f92..e5377a467e 100644 --- a/.pre-commit-config.yaml +++ b/.pre-commit-config.yaml @@ -1,6 +1,10 @@ # See https://pre-commit.com for more information # See https://pre-commit.com/hooks.html for more hooks +minimum_pre_commit_version: 3.3.0 +default_install_hook_types: [pre-commit, commit-msg] +default_stages: [pre-commit] + repos: - repo: https://github.com/pre-commit/pre-commit-hooks rev: v4.0.1 @@ -194,7 +198,7 @@ repos: - id: check-copyright args: ['--ignore', 'tools/ci/check_copyright_ignore.txt', '--config', 'tools/ci/check_copyright_config.yaml'] - repo: https://github.com/espressif/conventional-precommit-linter - rev: v1.3.0 + rev: v1.4.1 hooks: - id: conventional-precommit-linter stages: [commit-msg]