Merge pull request #81 from rylev/clean-up-logs

Clean up logs by not showing headers on empty diffs
rust-lang · Apr 4, 2024 · a1a6e75 · a1a6e75
2 parents c9a63f6 + 3a07888
commit a1a6e75
Showing 1 changed file with 6 additions and 2 deletions.
diff --git a/src/github/mod.rs b/src/github/mod.rs
@@ -866,11 +866,15 @@ impl std::fmt::Display for UpdateRepoDiff {
         for permission_diff in &self.permission_diffs {
             write!(f, "{permission_diff}")?;
         }
-        writeln!(f, "  Branch Protections:")?;
+        if !self.branch_protection_diffs.is_empty() {
+            writeln!(f, "  Branch Protections:")?;
+        }
         for branch_protection_diff in &self.branch_protection_diffs {
             write!(f, "{branch_protection_diff}")?;
         }
-        writeln!(f, "  App installation changes:")?;
+        if !self.app_installation_diffs.is_empty() {
+            writeln!(f, "  App installation changes:")?;
+        }
         for diff in &self.app_installation_diffs {
             write!(f, "{diff}")?;
         }