Merge pull request #2469 from github/aeisenberg/upload-eslint-sarif

Upload sarif for eslint results
github · Sep 10, 2024 · e817992 · e817992
2 parents 49021ad + 56b8418
commit e817992
Show file tree

Hide file tree

Showing 70 changed files with 18,213 additions and 5 deletions.
diff --git a/.eslintignore b/.eslintignore
diff --git a/.github/workflows/pr-checks.yml b/.github/workflows/pr-checks.yml
@@ -24,7 +24,15 @@ jobs:
         uses: actions/checkout@v4
 
       - name: Lint
-        run: npm run-script 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'
+        with:
+          sarif_file: eslint.sarif
+          category: eslint
 
       - name: Update version of @types/node
         if: matrix.node-types-version != 'current'

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/node_modules/.package-lock.json b/node_modules/.package-lock.json
diff --git a/node_modules/@microsoft/eslint-formatter-sarif/README.md b/node_modules/@microsoft/eslint-formatter-sarif/README.md
diff --git a/node_modules/@microsoft/eslint-formatter-sarif/package.json b/node_modules/@microsoft/eslint-formatter-sarif/package.json