diff options
| author | Christian Breunig <christian@breunig.cc> | 2024-04-03 21:12:26 +0200 |
|---|---|---|
| committer | Christian Breunig <christian@breunig.cc> | 2024-04-03 21:15:40 +0200 |
| commit | 8205e3cf918142a55e00c00dc241a6a30914fbd9 (patch) | |
| tree | f1ab6216371b533833b2051ca601728f778bf06c | |
| parent | d4b53a38359d80a4578d6fb2d8c76241b310f866 (diff) | |
| download | vyos-1x-8205e3cf918142a55e00c00dc241a6a30914fbd9.tar.gz vyos-1x-8205e3cf918142a55e00c00dc241a6a30914fbd9.zip | |
GitHub: run unused imports action only for pull requests
| -rw-r--r-- | .github/workflows/unused-imports.yml | 14 |
1 files changed, 10 insertions, 4 deletions
diff --git a/.github/workflows/unused-imports.yml b/.github/workflows/unused-imports.yml index 6bb3d2cc7..83098ddf6 100644 --- a/.github/workflows/unused-imports.yml +++ b/.github/workflows/unused-imports.yml @@ -1,6 +1,7 @@ -name: Pylint - -on: [push] +name: Check for unused imports using Pylint +on: + pull_request_target: + types: [opened, reopened, ready_for_review, locked] jobs: build: @@ -20,4 +21,9 @@ jobs: pip install pylint - name: Analysing the code with pylint run: | - make unused-imports + tmp=$(git ls-files *.py | xargs pylint | grep W0611 | wc -l) + if [[ $tmp -gt 0 ]]; then + echo "Found $tmp occurrence of unused Python import statements!" + exit 1 + fi + exit 0 |