lint

Author: hmarr

codeowners-cli/src/main.rs

@@ -74,7 +74,7 @@ impl Cli {
                 .map_err(|e| anyhow!("reading {:?}: {}", paths_from_file, e))?;
             let reader = std::io::BufReader::new(file);
             Ok(Box::new(
-                reader.lines().filter_map(|l| l.ok()).map(PathBuf::from),
+                reader.lines().map_while(Result::ok).map(PathBuf::from),
             ))
         } else {
             Ok(self.root_paths().into_iter().map(walk_files).fold(

codeowners-rs/src/ruleset.rs

@@ -48,13 +48,13 @@ impl RuleSet {
     /// Returns the owners for the given path, or `None` if no rules match the
     /// path or the matching rule has no owners.
     pub fn owners(&self, path: impl AsRef<Path>) -> Option<&[Owner]> {
-        return self.matching_rule(path).and_then(|rule| {
+        self.matching_rule(path).and_then(|rule| {
             if rule.owners.is_empty() {
                 None
             } else {
                 Some(rule.owners.as_ref())
             }
-        });
+        })
     }
 
     /// Returns the all rules that match the given path along with their indices.