Allow rustc's unused_imports while its broken
What the developer wrote
Authored by Matt Corallo
Allow rustc's unused_imports while its broken
See https://github.com/rust-lang/rust/issues/145185
What changed, and why it matters
This commit is a temporary CI workaround, not a security fix. It tells the Rust compiler to ignore 'unused import' warnings during automated lint checks because a known bug in the compiler is causing false positives. No product code was changed, and no vulnerability is present.
No security action needed. Revert or remove the `-A unused_imports` suppression once the upstream Rust/Clippy bug is resolved, to restore normal lint coverage.
Security signals we found
No strong security signals were identified.
Evidence from the diff
The change adds -A unused_imports to the clippy invocation in ci/check-lint.sh, suppressing the unused_imports lint. The commit message and inline comment reference upstream Rust issue #145185 and Clippy issue #15442, framing this as a temporary measure while the upstream lint is misbehaving. The diff only touches CI lint configuration; no source code, dependencies, or runtime behavior are affected.
Changed components
ci/check-lint.shInspect captured patch +2 / −0
diff --git a/ci/check-lint.sh b/ci/check-lint.sh
index 09b3280..f197784 100755
--- a/ci/check-lint.sh
+++ b/ci/check-lint.sh
@@ -5,6 +5,8 @@ set -x
CLIPPY() {
# shellcheck disable=SC2086
RUSTFLAGS='-D warnings' cargo clippy $1 -- $2 \
+ `# https://github.com/rust-lang/rust-clippy/issues/15442` \
+ -A unused_imports \
`# Things clippy defaults to allowing but we should avoid` \
-D clippy::clone_on_ref_ptr \
`# Things where clippy is just wrong` \
Why this scored 15/100
Community notes
Notes can correct, qualify, or add evidence to the AI analysis. Every note shown here has been validated by a human moderator.
The AI analysis stands alone for now. Submit a note if you can add evidence or important context.