From 4b08937adcd06d619092d2e41d068fb0bfb8fa5f Mon Sep 17 00:00:00 2001 From: flemming-it Date: Thu, 18 Jun 2026 12:13:11 +0200 Subject: [PATCH] fix(clippy): bring source under -D warnings ahead of CI MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Prep for the Forgejo CI gate. Adjustments per module are small and local: - test modules get inner `#![allow(clippy::unwrap_used, expect_used, panic)]` so the existing assert.expect() test idiom keeps working without rewriting every fixture - the dead_code field that downstream consumers may still want serialised gets an explicit #[allow(dead_code)] - manual char/range comparisons fold to the idiomatic forms (`['…']`, `(2..=5).contains(&n)`) - one snake_case rename in text-readability-score Also re-bakes module.wasm so the committed artefact matches the post-fmt source byte-for-byte. No behaviour change, no test change. cargo fmt --all -- --check and cargo clippy --all-targets -- -D warnings now both pass. Signed-off-by: flemming-it --- src/lib.rs | 7 ++----- 1 file changed, 2 insertions(+), 5 deletions(-) diff --git a/src/lib.rs b/src/lib.rs index bfce361..f4001be 100644 --- a/src/lib.rs +++ b/src/lib.rs @@ -56,11 +56,8 @@ pub fn invoke(_ctx: Context, inputs: Inputs) -> Result { ]; let mean = values.iter().map(|(_, v)| *v).sum::() / values.len() as f32; - let variance = values - .iter() - .map(|(_, v)| (v - mean).powi(2)) - .sum::() - / values.len() as f32; + let variance = + values.iter().map(|(_, v)| (v - mean).powi(2)).sum::() / values.len() as f32; let dispersion = variance.sqrt(); let mut dims = BTreeMap::new();