What the developer wrote
Authored by Roman Zeyde
Disable warning on `configure_me_config.rs`
What changed, and why it matters
This commit is a minor build hygiene change. It removes a blanket Clippy flag that suppressed 'unused-attributes' warnings across the whole project, and instead suppresses that specific warning only inside the auto-generated configuration file where it occurs. There is no security-relevant behavior change to the application.
No security action needed. This is a routine linting/CI cleanup.
Security signals we found
No strong security signals were identified.
Evidence from the diff
The change updates the GitHub Actions Clippy invocation from ‘cargo clippy – -A unused-attributes -D warnings’ to ‘cargo clippy – -D warnings’, and adds ‘#![allow(unused_attributes)]’ alongside the existing ‘#![allow(unused_imports)]’ in the internal module that includes the generated ‘configure_me_config.rs’. This narrows the scope of the warning suppression and keeps CI treating all other warnings as errors. No runtime code, configuration parsing, network handling, or cryptographic logic is modified.
Changed components
CI/CD pipeline (.github/workflows/rust.yml)src/config.rs generated-code include wrapperInspect captured patch +2 / −2
diff --git a/.github/workflows/rust.yml b/.github/workflows/rust.yml
index 47b0e1e..db3a668 100644
--- a/.github/workflows/rust.yml
+++ b/.github/workflows/rust.yml
@@ -48,7 +48,7 @@ jobs:
run: cargo test ${{ matrix.build-args }} --all
- name: Clippy
- run: cargo clippy -- -A unused-attributes -D warnings
+ run: cargo clippy -- -D warnings
integration:
name: Integration
diff --git a/src/config.rs b/src/config.rs
index 851e0f6..b844dda 100644
--- a/src/config.rs
+++ b/src/config.rs
@@ -17,7 +17,7 @@ pub const ELECTRS_VERSION: &str = env!("CARGO_PKG_VERSION");
const DEFAULT_SERVER_ADDRESS: [u8; 4] = [127, 0, 0, 1]; // by default, serve on IPv4 localhost
mod internal {
- #![allow(unused_imports)]
+ #![allow(unused_attributes, unused_imports)]
include!(concat!(env!("OUT_DIR"), "/configure_me_config.rs"));
}
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.