Skip to content

Commit 224218f

Browse files
committed
Always upload eslint.sarif
1 parent e817992 commit 224218f

File tree

1 file changed

+8
-1
lines changed

1 file changed

+8
-1
lines changed

.github/workflows/pr-checks.yml

Lines changed: 8 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -24,16 +24,23 @@ jobs:
2424
uses: actions/checkout@v4
2525

2626
- name: Lint
27+
id: lint
2728
run: npm run-script lint-ci
2829

2930
- name: Upload sarif
3031
uses: github/codeql-action/upload-sarif@v3
3132
# Only upload SARIF for the latest version of Node.js
32-
if: matrix.node-types-version == 'current'
33+
if: matrix.node-types-version == 'current' && always()
3334
with:
3435
sarif_file: eslint.sarif
3536
category: eslint
3637

38+
- name: Check for lint errors
39+
if: steps.lint.outcome == 'failure'
40+
run: |
41+
echo "Please fix the linting issues above before committing."
42+
exit 1
43+
3744
- name: Update version of @types/node
3845
if: matrix.node-types-version != 'current'
3946
env:

0 commit comments

Comments
 (0)