Skip to content

Commit ece6354

Browse files
author
User
committed
address review: revert return None -> Some(Self::Unknown) change
Per ntBre's feedback, the else branch should remain `return None` as in the original code.
1 parent c370861 commit ece6354

1 file changed

Lines changed: 1 addition & 2 deletions

File tree

crates/ruff_linter/src/rules/fastapi/rules/fastapi_unused_path_parameter.rs

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -411,8 +411,7 @@ impl<'a> Dependency<'a> {
411411
.map(|param| param.name().as_str())
412412
.collect()
413413
} else {
414-
// No matching method found — treat as unknown to suppress false positives.
415-
return Some(Self::Unknown);
414+
return None;
416415
};
417416

418417
Some(Self::Class(parameter_names))

0 commit comments

Comments
 (0)