From 1ac93cb736f82002934a498edbae175ae3eb7e6a Mon Sep 17 00:00:00 2001 From: Rafael Guterres Jeffman <rjeffman@redhat.com> Date: Wed, 25 Nov 2020 20:21:37 -0300 Subject: [PATCH] yamllint: Run yaml linter only on modified files in pre-commit. With the parameter `args: ['.']`, yamllint would run over every file during pre-commit, including those not being commited, and it would allow for false negatives, not allowing a commit, even if commited yaml files had no issues, but another file, not par of the commit, had. By changing the yamllint parameter to `files: \.(yaml|yml)$` it will only check files being commited, preventing false negatives, and allowing for faster commits. --- .pre-commit-config.yaml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/.pre-commit-config.yaml b/.pre-commit-config.yaml index 0e3e2373..93fda93b 100644 --- a/.pre-commit-config.yaml +++ b/.pre-commit-config.yaml @@ -12,7 +12,7 @@ repos: rev: v1.25.0 hooks: - id: yamllint - args: ['.'] + files: \.(yaml|yml)$ - repo: https://gitlab.com/pycqa/flake8 rev: 3.8.4 hooks: -- GitLab