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)],

src/ownership/validator.rs

@@ -79,4 +79,4 @@ fn test_for_file_multiple_owners() -> Result<(), Box<dyn Error>> {
             - Owner specified in `ruby/app/services/.codeowner`
         "}));
     Ok(())
-}          
+}

tests/invalid_project_test.rs

@@ -46,7 +46,8 @@ fn test_for_file() -> Result<(), Box<dyn Error>> {
         .stdout(predicate::eq(indoc! {"
             Team: Payroll
             Team YML: config/teams/payroll.yml
-            Description: Owner annotation at the top of the file
+            Description:
+            - Owner annotation at the top of the file
         "}));
     Ok(())
 }