Skip to content

Commit e817992

Browse files
authored
Merge pull request #2469 from github/aeisenberg/upload-eslint-sarif
Upload sarif for eslint results
2 parents 49021ad + 56b8418 commit e817992

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

70 files changed

+18213
-5
lines changed

.eslintignore

-4
This file was deleted.

.github/workflows/pr-checks.yml

+9-1
Original file line numberDiff line numberDiff line change
@@ -24,7 +24,15 @@ jobs:
2424
uses: actions/checkout@v4
2525

2626
- name: Lint
27-
run: npm run-script lint
27+
run: npm run-script lint-ci
28+
29+
- name: Upload sarif
30+
uses: github/codeql-action/upload-sarif@v3
31+
# Only upload SARIF for the latest version of Node.js
32+
if: matrix.node-types-version == 'current'
33+
with:
34+
sarif_file: eslint.sarif
35+
category: eslint
2836

2937
- name: Update version of @types/node
3038
if: matrix.node-types-version != 'current'

.gitignore

+2
Original file line numberDiff line numberDiff line change
@@ -5,3 +5,5 @@ node_modules/.cache/
55
*.class
66
# macOS
77
.DS_Store
8+
# eslint sarif report
9+
eslint.sarif

node_modules/.package-lock.json

+30
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

node_modules/@microsoft/eslint-formatter-sarif/README.md

+43
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

node_modules/@microsoft/eslint-formatter-sarif/package.json

+63
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

0 commit comments

Comments
 (0)