Merge pull request #2474 from github/aeisenberg/always-upload-eslint-…

…sarif Always upload eslint.sarif
github · Sep 12, 2024 · 4a01ec7 · 4a01ec7
2 parents 762dbae + 0d0f998
commit 4a01ec7
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/.github/workflows/pr-checks.yml b/.github/workflows/pr-checks.yml
@@ -24,12 +24,13 @@ jobs:
         uses: actions/checkout@v4
 
       - name: Lint
+        id: lint
         run: npm run-script lint-ci
 
       - name: Upload sarif
         uses: github/codeql-action/upload-sarif@v3
         # Only upload SARIF for the latest version of Node.js
-        if: matrix.node-types-version == 'current'
+        if: "always() && matrix.node-types-version == 'current'"
         with:
           sarif_file: eslint.sarif
           category: eslint