danger 5.13.0 → 5.14.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
-SHA256:
-  metadata.gz: f22280b15d14004d5087e64d8d576819b63694288d5541a71f25eb2d6baeecdc
-  data.tar.gz: 2964dbf7b7a4389c7be8e293dc0bbabcfe21419fe4a18b04b6d77e6b129ebf2d
+SHA1:
+  metadata.gz: 3ad7a6d0ad0c2cac62253cf27996a549026159df
+  data.tar.gz: 247abc76bb8cd25047c242835330e09413268d40
 SHA512:
-  metadata.gz: 2a6de8ea2f5cf3a80d4aa45dab2bd47980d11fcde6569d321623155b244a67668e213af664d9ee54f39f8653c7ce0d15683dcfe9030c169f28d194c65a778027
-  data.tar.gz: cb4d77b72b305259ca17670f334988c229ab18604db04896c39a0d954fc3d742ae0a35e54d8ca56b4c89fb19a7a4a4d97a07853b2b5e358bfd49f578655f0f38
+  metadata.gz: fc508c5fea76dab88a9fdbc0574e8d8d8674e8a21a7f53bc4d39c94bc9ba413ca4dae3a87d2f7081c1e180234ee2c659b7922897eba840e156cb30c4636d2ac2
+  data.tar.gz: 3863dd8cf1b5470bda0af673ea8bf3a6b22709959335b6fc7f29394523bb96a325aed0e499bb8b844c2bc6239b5a446415acdd434dae334d9696c3718a145dc1

data/lib/danger/commands/runner.rb

@@ -38,6 +38,7 @@ module Danger
       @base = argv.option("base")
       @head = argv.option("head")
       @fail_on_errors = argv.option("fail-on-errors", false)
+      @fail_if_no_pr = argv.option("fail-if-no-pr", false)
       @new_comment = argv.flag?("new-comment")
       @remove_previous_comments = argv.flag?("remove-previous-comments")
       @danger_id = argv.option("danger_id", "danger")
@@ -59,6 +60,7 @@ module Danger
         ["--base=[master|dev|stable]", "A branch/tag/commit to use as the base of the diff"],
         ["--head=[master|dev|stable]", "A branch/tag/commit to use as the head"],
         ["--fail-on-errors=<true|false>", "Should always fail the build process, defaults to false"],
+        ["--fail-if-no-pr=<true|false>", "Should fail the build process if no PR is found (useful for CircleCI), defaults to false"],
         ["--dangerfile=<path/to/dangerfile>", "The location of your Dangerfile"],
         ["--danger_id=<id>", "The identifier of this Danger instance"],
         ["--new-comment", "Makes Danger post a new comment instead of editing its previous one"],
@@ -74,6 +76,7 @@ module Danger
         danger_id: @danger_id,
         new_comment: @new_comment,
         fail_on_errors: @fail_on_errors,
+        fail_if_no_pr: @fail_if_no_pr,
         remove_previous_comments: @remove_previous_comments
       )
     end

data/lib/danger/danger_core/executor.rb

@@ -13,12 +13,13 @@ module Danger
             danger_id: nil,
             new_comment: nil,
             fail_on_errors: nil,
+            fail_if_no_pr: nil,
             remove_previous_comments: nil)
       # Create a silent Cork instance if cork is nil, as it's likely a test
       cork ||= Cork::Board.new(silent: false, verbose: false)
 
       # Run some validations
-      validate!(cork)
+      validate!(cork, fail_if_no_pr: fail_if_no_pr)
 
       # OK, we now know that Danger can run in this enviroment
       env ||= EnvironmentManager.new(system_env, cork)
@@ -41,9 +42,9 @@ module Danger
       exit(1) if fail_on_errors && ran_status
     end
 
-    def validate!(cork)
+    def validate!(cork, fail_if_no_pr: false)
       validate_ci!
-      validate_pr!(cork)
+      validate_pr!(cork, fail_if_no_pr)
     end
 
     private
@@ -58,11 +59,12 @@ module Danger
     end
 
     # Could we determine that the CI source is inside a PR?
-    def validate_pr!(cork)
+    def validate_pr!(cork, fail_if_no_pr)
       unless EnvironmentManager.pr?(system_env)
         ci_name = EnvironmentManager.local_ci_source(system_env).name.split("::").last
         cork.puts "Not a #{ci_name} Pull Request - skipping `danger` run".yellow
-        exit(0)
+
+        exit(fail_if_no_pr ? 1 : 0)
       end
     end
 

data/lib/danger/version.rb

@@ -1,4 +1,4 @@
 module Danger
-  VERSION = "5.13.0".freeze
+  VERSION = "5.14.0".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: 5.13.0
+  version: 5.14.0
 platform: ruby
 authors:
 - Orta Therox
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2019-01-31 00:00:00.000000000 Z
+date: 2019-02-11 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: claide
@@ -300,7 +300,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.7.6
+rubygems_version: 2.5.2.3
 signing_key: 
 specification_version: 4
 summary: Like Unit Tests, but for your Team Culture.