Skip to content

Commit f7cdf40

Browse files
committed
fix: upgrade CodeQL to v4, remove SARIF upload
- CodeQL actions updated to v4 - Removed SARIF upload (requires Code Scanning enabled) - Simplified ESLint check
1 parent b0de514 commit f7cdf40

File tree

3 files changed

+2
-14
lines changed

3 files changed

+2
-14
lines changed

.github/workflows/code-quality.yml

Lines changed: 0 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -27,17 +27,6 @@ jobs:
2727
run: npm install
2828

2929
- name: Run ESLint
30-
run: npm run lint -- --format @microsoft/eslint-formatter-sarif --output-file eslint-results.sarif
31-
continue-on-error: true
32-
33-
- name: Upload ESLint results
34-
uses: github/codeql-action/upload-sarif@v3
35-
if: always()
36-
with:
37-
sarif_file: eslint-results.sarif
38-
wait-for-processing: true
39-
40-
- name: Run ESLint (annotations)
4130
run: npm run lint
4231

4332
format:

.github/workflows/codeql.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -28,7 +28,7 @@ jobs:
2828
uses: actions/checkout@v4
2929

3030
- name: Initialize CodeQL
31-
uses: github/codeql-action/init@v3
31+
uses: github/codeql-action/init@v4
3232
with:
3333
languages: ${{ matrix.language }}
3434
queries: security-extended,security-and-quality
@@ -45,6 +45,6 @@ jobs:
4545
run: npm run build
4646

4747
- name: Perform CodeQL Analysis
48-
uses: github/codeql-action/analyze@v3
48+
uses: github/codeql-action/analyze@v4
4949
with:
5050
category: "/language:${{ matrix.language }}"

package.json

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -70,7 +70,6 @@
7070
},
7171
"devDependencies": {
7272
"@eslint/js": "^9.0.0",
73-
"@microsoft/eslint-formatter-sarif": "^3.1.0",
7473
"@types/jest": "^29.5.12",
7574
"@types/node": "^20.10.0",
7675
"eslint": "^9.0.0",

0 commit comments

Comments
 (0)