Skip to content

Commit f613318

Browse files
terrikoinosmeet
authored andcommitted
ci: improve interrogate/pre-commit config (intel#3714)
Switch our config to use pre-commit's file exclusions instead of interrogate's because it works better for local & github actions. Signed-off-by: Terri Oda <[email protected]>
1 parent 5d5107f commit f613318

File tree

2 files changed

+5
-8
lines changed

2 files changed

+5
-8
lines changed

.github/workflows/linting.yml

Lines changed: 2 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -12,12 +12,10 @@ jobs:
1212
linting:
1313
name: Linting
1414
runs-on: ubuntu-22.04
15-
permissions:
16-
contents: write # for stefanzweifel/git-auto-commit-action to create a PR
1715
strategy:
1816
fail-fast: false
1917
matrix:
20-
tool: ['isort', 'black', 'pyupgrade', 'flake8', 'bandit', 'gitlint', 'mypy','interrogate']
18+
tool: ['isort', 'black', 'pyupgrade', 'flake8', 'bandit', 'gitlint', 'mypy', 'interrogate']
2119
steps:
2220
- name: Harden Runner
2321
uses: step-security/harden-runner@eb238b55efaa70779f274895e782ed17c84f2895 # v2.6.1
@@ -46,5 +44,4 @@ jobs:
4644
run: |
4745
python -m pip install --upgrade gitlint
4846
echo "$TITLE" | gitlint
49-
- uses: stefanzweifel/git-auto-commit-action@8756aa072ef5b4a080af5dc8fef36c5d586e521d
50-
47+

.pre-commit-config.yaml

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -76,6 +76,6 @@ repos:
7676
hooks:
7777
- id: interrogate
7878
verbose: True
79-
exclude: ^(locales|presentation)
80-
args: ["-vv", "-i", "-I", "-M", "-C", "-n", "-p", "-f", "60.0", "-e", "fuzz/", "-e", "test/", "-e", "cve_bin_tool/checkers/", "-e", "build/"]
81-
pass_filenames: false
79+
exclude: ^(locales|presentation|fuzz|test|cve_bin_tool/checkers|build)
80+
args: ["-vv", "-i", "-I", "-M", "-C", "-n", "-p", "-f", "60.0"]
81+

0 commit comments

Comments
 (0)