danger 6.0.0 → 6.0.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 4d07daba46ea8c061e649a70aea12ae0861e959401e6231f4fa234bb90f6ca41
-  data.tar.gz: d61955050a1978795d286e1fd871ba7a00171ba219be928d14a5eb78c3dea86d
+  metadata.gz: df3df4ffaefafa6d1345b6dcd7b02ffad8001548e74a70eaa1226dae5efd173d
+  data.tar.gz: 6d739d7c9f007eaee64f019b83026596aa64204c7df94cd005fdf4aa52ffc013
 SHA512:
-  metadata.gz: 58c8913b443458cfebfd6d7cb257fc5bbebe7c1d024f37aeec9d9b9a11c7c629bf31e610e1a965c3a574270aa86f7268f5ecb6d47e57539e02a4c421b6e51555
-  data.tar.gz: 8b497b276d5c04fda4704c1c018d088dbd9b729b6a91750b988e7e1aac6d9cd8ab24975fac75b1f3371638278e13fe62bcf888b0606f775557dba7bd8f25e72a
+  metadata.gz: 62305cb2f4c3a49a8e9e46a769c89229b124dde8d14946ec5b32899553acc154f9ac7112d26cefd5f9ae82b2bffc90ef61dd70ad88752721e1bb95e84a76ce06
+  data.tar.gz: 45b7199bb54b2fa90622539f79206b186d8aabf1e55821ad56a9dbb746b3891f894b4bbdf5ea75911d983bb81211a32b6032cc5123757c85774429d106b90a80

data/lib/danger/request_sources/gitlab.rb

@@ -399,33 +399,40 @@ module Danger
             end
           end
 
-          begin
-            if matching_comments.empty?
-              params = {
-                body: body,
-                position: {
-                  position_type: 'text',
-                  new_path: m.file,
-                  new_line: m.line,
-                  base_sha: self.mr_json.diff_refs.base_sha,
-                  start_sha: self.mr_json.diff_refs.start_sha,
-                  head_sha: self.mr_json.diff_refs.head_sha
-                }
+          if matching_comments.empty?
+            params = {
+              body: body,
+              position: {
+                position_type: 'text',
+                new_path: m.file,
+                new_line: m.line,
+                base_sha: self.mr_json.diff_refs.base_sha,
+                start_sha: self.mr_json.diff_refs.start_sha,
+                head_sha: self.mr_json.diff_refs.head_sha
               }
+            }
+            begin
               client.create_merge_request_discussion(ci_source.repo_slug, ci_source.pull_request_id, params)
-            else
-              # Remove the surviving comment so we don't strike it out
-              danger_comments.reject! { |c| matching_comments.include? c }
+            rescue Gitlab::Error::Error => e
+              message = [e, "body: #{body}", "position: #{params[:position].inspect}"].join("\n")
+              puts message
 
-              # Update the comment to remove the strikethrough if present
-              comment = matching_comments.first
-              client.update_merge_request_discussion_note(ci_source.repo_slug, ci_source.pull_request_id, comment["discussion_id"], comment["id"], body)
+              next false
             end
-          rescue Gitlab::Error::Error => e
-            message = [e, "body: #{body}", "position: #{params[:position].inspect}"].join("\n")
-            puts message
+          else
+            # Remove the surviving comment so we don't strike it out
+            danger_comments.reject! { |c| matching_comments.include? c }
 
-            next false
+            # Update the comment to remove the strikethrough if present
+            comment = matching_comments.first
+            begin
+              client.update_merge_request_discussion_note(ci_source.repo_slug, ci_source.pull_request_id, comment["discussion_id"], comment["id"], body)
+            rescue Gitlab::Error::Error => e
+              message = [e, "body: #{body}"].join("\n")
+              puts message
+              
+              next false
+            end
           end
 
           # Remove this element from the array

data/lib/danger/version.rb

@@ -1,4 +1,4 @@
 module Danger
-  VERSION = "6.0.0".freeze
+  VERSION = "6.0.1".freeze
   DESCRIPTION = "Like Unit Tests, but for your Team Culture.".freeze
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: danger
 version: !ruby/object:Gem::Version
-  version: 6.0.0
+  version: 6.0.1
 platform: ruby
 authors:
 - Orta Therox
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2019-03-11 00:00:00.000000000 Z
+date: 2019-03-14 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: claide