Only warn about fixup commits once per run

The Danger GitLab CI job currently generates a separate error message
about fixup commits being present in a merge request for every such
commit found.  Prevent that by making it only log that error message
once per run.

diff --git a/dangerfile.py b/dangerfile.py
index ad40478f773b3c819d2195e2b20015ca13585429..68daa7876c3a0553ea147d82a77787b0add8e835 100644 (file)
--- a/dangerfile.py
+++ b/dangerfile.py
@@ -61,12 +61,16 @@ target_branch = danger.gitlab.mr.target_branch
 #       four spaces (useful for pasting compiler warnings, static analyzer
 #       messages, log lines, etc.)
 
+fixup_error_logged = False
 for commit in danger.git.commits:
     message_lines = commit.message.splitlines()
     subject = message_lines[0]
-    if subject.startswith('fixup!') or subject.startswith('Apply suggestion'):
+    if (not fixup_error_logged and
+            (subject.startswith('fixup!') or
+             subject.startswith('Apply suggestion'))):
         fail('Fixup commits are still present in this merge request. '
              'Please squash them before merging.')
+        fixup_error_logged = True
     if len(subject) > 72:
         warn(
             f'Subject line for commit {commit.sha} is too long: '