Merge pull request #190 from klensy/no-failures

rollups: don't print 'Failed merges:' if there no failed merges
rust-lang · May 18, 2023 · 0d97917 · 0d97917
2 parents 982fa68 + 35590d4
commit 0d97917
Showing 1 changed file with 5 additions and 3 deletions.
diff --git a/homu/server.py b/homu/server.py
@@ -344,9 +344,11 @@ def rollup(user_gh, state, repo_label, repo_cfg, repo):
     body = 'Successful merges:\n\n'
     for x in successes:
         body += ' - #{} ({})\n'.format(x.num, x.title)
-    body += '\nFailed merges:\n\n'
-    for x in failures:
-        body += ' - #{} ({})\n'.format(x.num, x.title)
+
+    if len(failures) != 0:
+        body += '\nFailed merges:\n\n'
+        for x in failures:
+            body += ' - #{} ({})\n'.format(x.num, x.title)
     body += '\nr? @ghost\n@rustbot modify labels: rollup'
 
     # Set web.base_url in cfg to enable