From 629d3e9608878c8702cf8b267170006f6ed1d876 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 | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/src/lib.rs b/src/lib.rs index df0383e..45e64c1 100644 --- a/src/lib.rs +++ b/src/lib.rs @@ -24,7 +24,7 @@ struct Cohort { #[fai_module] pub fn invoke(_ctx: Context, inputs: Inputs) -> Result { let id = inputs.require_text("cohort_id")?; - let cohort = resolve(&id).ok_or_else(|| { + let cohort = resolve(id).ok_or_else(|| { ModuleError::invalid_input(format!( "unknown cohort_id: {id} (v0.1.0 supports: \ berlin-kmu, de-kmu, gastro-de, handwerk-be, bauwirtschaft-be)" @@ -76,6 +76,7 @@ fn resolve(id: &str) -> Option { #[cfg(test)] mod tests { + #![allow(clippy::unwrap_used, clippy::expect_used, clippy::panic)] use super::*; #[test]