Ignore suppressed alerts

github · Sep 10, 2024 · 56b8418 · 56b8418
1 parent 5c9d953
commit 56b8418
Show file tree

Hide file tree

Showing 4 changed files with 4 additions and 9 deletions.
diff --git a/.eslintignore b/.eslintignore
diff --git a/.github/workflows/pr-checks.yml b/.github/workflows/pr-checks.yml
@@ -28,15 +28,12 @@ jobs:
 
       - name: Upload sarif
         uses: github/codeql-action/upload-sarif@v3
-          # Only upload SARIF for the latest version of Node.js
+        # Only upload SARIF for the latest version of Node.js
         if: matrix.node-types-version == 'current'
         with:
           sarif_file: eslint.sarif
           category: eslint
 
-      - name: Ensure the working directory is clean
-        run: rm -f eslint.sarif
-
       - name: Update version of @types/node
         if: matrix.node-types-version != 'current'
         env:

diff --git a/.gitignore b/.gitignore
@@ -5,3 +5,5 @@ node_modules/.cache/
 *.class
 # macOS
 .DS_Store
+# eslint sarif report
+eslint.sarif
diff --git a/package.json b/package.json
@@ -9,7 +9,7 @@
     "test-debug": "ava src/**.test.ts --serial --verbose --timeout=20m",
     "lint": "eslint --report-unused-disable-directives --max-warnings=0 .",
     "lint-fix": "eslint --report-unused-disable-directives --max-warnings=0 . --fix",
-    "lint-ci": "eslint --report-unused-disable-directives --max-warnings=0 . --format @microsoft/eslint-formatter-sarif --output-file=eslint.sarif",
+    "lint-ci": "SARIF_ESLINT_IGNORE_SUPPRESSED=true eslint --report-unused-disable-directives --max-warnings=0 . --format @microsoft/eslint-formatter-sarif --output-file=eslint.sarif",
     "removeNPMAbsolutePaths": "removeNPMAbsolutePaths . --force"
   },
   "ava": {