Skip to content

Commit badcf15

Browse files
committed
refactor(ci): remove clippy-sarif in favor of CodeQL
CodeQL is already enabled for security scanning. Remove redundant clippy-sarif integration to simplify CI.
1 parent 1caf14f commit badcf15

File tree

2 files changed

+2
-24
lines changed

2 files changed

+2
-24
lines changed

.github/workflows/ci.yml

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -56,12 +56,6 @@ jobs:
5656
- name: Run lint checks
5757
run: cargo make ci-lint
5858

59-
- name: Upload SARIF to GitHub
60-
uses: github/codeql-action/upload-sarif@v4
61-
with:
62-
sarif_file: clippy-results.sarif
63-
wait-for-processing: true
64-
6559
# Security audit
6660
security:
6761
name: Security Audit

Makefile.toml

Lines changed: 2 additions & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -47,22 +47,6 @@ args = [
4747
"-D", "warnings"
4848
]
4949

50-
[tasks.clippy-sarif]
51-
description = "Generate SARIF output from clippy for GitHub integration"
52-
dependencies = ["install-clippy-sarif"]
53-
script = '''
54-
#!/bin/bash
55-
set -e
56-
cargo clippy --all-targets --all-features --workspace \
57-
--exclude feedparser-rs-py --message-format=json -- -D warnings | \
58-
clippy-sarif | tee clippy-results.sarif | sarif-fmt
59-
'''
60-
61-
[tasks.install-clippy-sarif]
62-
description = "Install clippy-sarif and sarif-fmt tools"
63-
install_crate = { crate_name = "clippy-sarif", binary = "clippy-sarif", test_arg = "--version" }
64-
install_crate_args = ["clippy-sarif", "sarif-fmt"]
65-
6650
[tasks.doc]
6751
description = "Generate documentation"
6852
command = "cargo"
@@ -339,8 +323,8 @@ echo " - Python: (console output above)"
339323
# ============================================================================
340324

341325
[tasks.ci-lint]
342-
description = "CI: Run all linting checks with SARIF generation"
343-
run_task = { name = ["fmt-check", "clippy", "clippy-sarif", "doc-check"] }
326+
description = "CI: Run all linting checks"
327+
run_task = { name = ["fmt-check", "clippy", "doc-check"] }
344328

345329
[tasks.ci-security]
346330
description = "CI: Run all security checks"

0 commit comments

Comments
 (0)