Merge pull request #161 from voxpupuli/updates2

Only add a comment of PR is eligible
voxpupuli · May 1, 2020 · e259cc9 · e259cc9
2 parents a5e1293 + a12096c
commit e259cc9
Showing 1 changed file with 5 additions and 0 deletions.
diff --git a/app/models/pull_request.rb b/app/models/pull_request.rb
@@ -96,11 +96,16 @@ def ensure_label_is_detached(label)
   ##
   #  Add a comment with the given text
   def add_comment(text)
+    # TODO: why can request be nil and what is request
     req = begin
             request
           rescue StandardError
             nil
           end
+    # Only attach a comment if eligible_for_comment is true (The first iteration
+    # after the mergeable state changed to false)
+    return unless eligible_for_comment
+
     Raven.capture_message('Added a comment',
                           extra: { text: text,
                                    repo: repository.github_url,