From 35590d469a11a7411326d28005fcef1fe3842df0 Mon Sep 17 00:00:00 2001 From: klensy Date: Tue, 11 Apr 2023 15:13:37 +0300 Subject: [PATCH] rollups: don't print 'Failed merges:' if there no failed merges --- homu/server.py | 8 +++++--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/homu/server.py b/homu/server.py index b778c0b..20502b8 100644 --- 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