version 0.2.1

Author: perryqh

Cargo.lock

@@ -1,6 +1,6 @@
 [package]
 name = "codeowners"
-version = "0.2.0"
+version = "0.2.1"
 edition = "2021"
 
 [profile.release]

Cargo.toml

@@ -174,14 +174,18 @@ impl Error {
         match self {
             Error::FileWithoutOwner { path } => vec![format!("- {}", path.to_string_lossy())],
             Error::FileWithMultipleOwners { path, owners } => {
-                let mut output = vec![format!("\n{}", path.to_string_lossy().to_string())];
-                for owner in owners.iter().sorted_by_key(|owner| owner.team_name.to_lowercase()) {
-                    output.push(format!(" owner: {}", owner.team_name));
-                    for source in &owner.sources {
-                        output.push(format!("  - {}", source));
-                    }
-                }
-                vec![output.join("\n")]
+                let path_display = path.to_string_lossy();
+                let mut messages = vec![format!("\n{path_display}")];
+
+                owners
+                    .iter()
+                    .sorted_by_key(|owner| owner.team_name.to_lowercase())
+                    .for_each(|owner| {
+                        messages.push(format!(" owner: {}", owner.team_name));
+                        messages.extend(owner.sources.iter().map(|source| format!("  - {source}")));
+                    });
+
+                vec![messages.join("\n")]
             }
             Error::CodeownershipFileIsStale => vec![],
             Error::InvalidTeam { name, path } => vec![format!("- {} is referencing an invalid team - '{}'", path.to_string_lossy(), name)],