Skip to content

Commit 0a9ab48

Browse files
update linting.yml
1 parent 1282e54 commit 0a9ab48

File tree

1 file changed

+18
-6
lines changed

1 file changed

+18
-6
lines changed

.github/workflows/linting.yml

Lines changed: 18 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -1,27 +1,39 @@
11
name: Linting on Pull Request
2+
23
on:
34
pull_request:
45
types: [opened,synchronize]
6+
57
jobs:
6-
lintonPR:
8+
lintonPR:
79
runs-on: ubuntu-latest
810
steps:
911
- name: Checkout code
1012
uses: actions/checkout@v2
13+
1114
- name: Install Dependencies
1215
run: |
1316
pip install pylint
17+
1418
- name: Fetch Main
1519
run: |
1620
git fetch origin main
17-
git diff --name-only origin/main..HEAD
21+
git diff --name-only origin/main..HEAD
22+
1823
- name: Run pylint on modified files
1924
run: |
20-
changed_files=$(git diff --name-only origin/main..HEAD | grep ".py$")
25+
set -e
26+
IFS=$'\n'
27+
changed_files=$(git diff --name-only origin/main..HEAD | grep ".py$" || true)
2128
if [ "$changed_files" ]; then
22-
echo "Running pylint on changed files:"
23-
echo "$changed_files"
24-
echo "$changed_files" | xargs pylint
29+
for file in $changed_files; do
30+
if [ -e "$file" ]; then
31+
echo "Linting $file"
32+
pylint "$file"
33+
else
34+
echo "Skipping lint for deleted file: $file"
35+
fi
36+
done
2537
else
2638
echo "No Python files have been changed."
2739
fi

0 commit comments

Comments
 (0)