summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorRémi Verschelde <rverschelde@gmail.com>2024-05-11 16:19:36 +0200
committerRémi Verschelde <rverschelde@gmail.com>2024-05-11 16:19:36 +0200
commitbdc0316217940a8ccc80ce536547d42e6477adf4 (patch)
tree99e7a761bbbd81c08dc1a203e14377435d985503
parent10471a4c06da02d8d42501411c084c7fd984022b (diff)
parent734f820be48d6b5f0dbbdc30ca5a0e4225ab3d76 (diff)
downloadredot-engine-bdc0316217940a8ccc80ce536547d42e6477adf4.tar.gz
Merge pull request #91835 from akien-mga/ci-static-checks-readd-CHANGED_FILES-logic
CI: Re-add `CHANGED_FILES` logic for pre-commit checks
-rw-r--r--.github/workflows/static_checks.yml16
1 files changed, 15 insertions, 1 deletions
diff --git a/.github/workflows/static_checks.yml b/.github/workflows/static_checks.yml
index 2506cb7227..0c2d7d2e62 100644
--- a/.github/workflows/static_checks.yml
+++ b/.github/workflows/static_checks.yml
@@ -26,6 +26,20 @@ jobs:
pip3 install pytest==7.1.2
git config diff.wsErrorHighlight all
+ - name: Get changed files
+ env:
+ GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
+ run: |
+ if [ "${{ github.event_name }}" == "pull_request" ]; then
+ files=$(git diff-tree --no-commit-id --name-only -r HEAD^1..HEAD 2> /dev/null || true)
+ elif [ "${{ github.event_name }}" == "push" -a "${{ github.event.forced }}" == "false" -a "${{ github.event.created }}" == "false" ]; then
+ files=$(git diff-tree --no-commit-id --name-only -r ${{ github.event.before }}..${{ github.event.after }} 2> /dev/null || true)
+ fi
+ echo "$files" >> changed.txt
+ cat changed.txt
+ files=$(echo "$files" | grep -v 'thirdparty' | xargs -I {} sh -c 'echo "./{}"' | tr '\n' ' ')
+ echo "CHANGED_FILES=$files" >> $GITHUB_ENV
+
# This needs to happen before Python and npm execution; it must happen before any extra files are written.
- name: .gitignore checks (gitignore_check.sh)
run: |
@@ -34,7 +48,7 @@ jobs:
- name: Style checks via pre-commit
uses: pre-commit/action@v3.0.1
with:
- extra_args: --verbose
+ extra_args: --verbose --files ${{ env.CHANGED_FILES }}
- name: Python builders checks via pytest
run: |