fix(clippy): bring source under -D warnings ahead of CI
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 <sf@flemming.it>
This commit is contained in:
parent
69b7d84f73
commit
255d7e2615
1 changed files with 7 additions and 3 deletions
10
src/lib.rs
10
src/lib.rs
|
|
@ -49,7 +49,7 @@ pub fn invoke(_ctx: Context, inputs: Inputs) -> Result<Outputs, ModuleError> {
|
||||||
.require_json("norm")
|
.require_json("norm")
|
||||||
.map_err(|e| ModuleError::invalid_input(format!("norm shape: {e}")))?;
|
.map_err(|e| ModuleError::invalid_input(format!("norm shape: {e}")))?;
|
||||||
|
|
||||||
let graph_iVm: Option<InputGraph> = inputs.get("graph").and_then(|p| match p {
|
let graph_ivm: Option<InputGraph> = inputs.get("graph").and_then(|p| match p {
|
||||||
Payload::Json(s) => serde_json::from_str(s).ok(),
|
Payload::Json(s) => serde_json::from_str(s).ok(),
|
||||||
_ => None,
|
_ => None,
|
||||||
});
|
});
|
||||||
|
|
@ -62,7 +62,7 @@ pub fn invoke(_ctx: Context, inputs: Inputs) -> Result<Outputs, ModuleError> {
|
||||||
.join(" ");
|
.join(" ");
|
||||||
|
|
||||||
let flesch = flesch_de(&full_text);
|
let flesch = flesch_de(&full_text);
|
||||||
let depth = depth_index(graph_iVm.as_ref());
|
let depth = depth_index(graph_ivm.as_ref());
|
||||||
let composite = composite_frust(flesch, depth);
|
let composite = composite_frust(flesch, depth);
|
||||||
|
|
||||||
let out = Output {
|
let out = Output {
|
||||||
|
|
@ -89,7 +89,10 @@ fn flesch_de(text: &str) -> f32 {
|
||||||
}
|
}
|
||||||
|
|
||||||
fn count_sentences(text: &str) -> usize {
|
fn count_sentences(text: &str) -> usize {
|
||||||
text.chars().filter(|c| matches!(c, '.' | '!' | '?')).count().max(1)
|
text.chars()
|
||||||
|
.filter(|c| matches!(c, '.' | '!' | '?'))
|
||||||
|
.count()
|
||||||
|
.max(1)
|
||||||
}
|
}
|
||||||
|
|
||||||
fn count_syllables(word: &str) -> usize {
|
fn count_syllables(word: &str) -> usize {
|
||||||
|
|
@ -134,6 +137,7 @@ fn composite_frust(flesch: f32, depth: f32) -> f32 {
|
||||||
|
|
||||||
#[cfg(test)]
|
#[cfg(test)]
|
||||||
mod tests {
|
mod tests {
|
||||||
|
#![allow(clippy::unwrap_used, clippy::expect_used, clippy::panic)]
|
||||||
use super::*;
|
use super::*;
|
||||||
|
|
||||||
#[test]
|
#[test]
|
||||||
|
|
|
||||||
Loading…
Add table
Add a link
Reference in a new issue