danger-reviewer 0.0.4 → 0.0.5

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 22232c98b56cf47802bd550a4cd785d01ec19c08d4af87d08d99426f4acdc2ae
-  data.tar.gz: e5699cd16c9b3507b68a1135a6db5e295b2a8e3fbb659a438dccc8a0320798e1
+  metadata.gz: 986550a4967226d9bd65ddde539227323417ebf35e44c6bddb1436793eeed209
+  data.tar.gz: da424356a4e826dc023cd324d33cce31f25d42cb8e7c4db156bbaab0301640ef
 SHA512:
-  metadata.gz: a4f2dc88efe9d83bbf96548c6ee9212f93318009c7c4f4a63c6e10c49b531328792c3532bcd5917a0dbf39ba001515360e03067c28486ce15756b8114ebfe53b
-  data.tar.gz: 5afd46cf2fe3af636c5ab7f136ef8d1aaa0078e6432115e240e0144297a6a035cf7462e2b4a6466a1e3791f8049abb47c2d10616ba3866de443fc4317f5ff0f0
+  metadata.gz: ea68c83a38e6dc365467e1849c95498b4afc23001121d7debbc12cbf1f78d77c00e53d2254b308106dc32f94ecbc4999a81037f0e8f568f09d28bd2ea8f570e0
+  data.tar.gz: bf2aee355857a5fc30bb0a5b2c847f8d4878f5ca0aefd5d9a3edef97b1949dda7ae64b4f422c34047ce5e666a3f7b9a16e6de0926f281a1562295bd139386075

data/lib/reviewer/gem_version.rb

@@ -1,3 +1,3 @@
 module Reviewer
-  VERSION = "0.0.4".freeze
+  VERSION = "0.0.5".freeze
 end

data/lib/reviewer/plugin.rb

@@ -9,7 +9,7 @@ module Danger
       current_count = current_reviewers_count
 
       # Check if we already have enough reviewers
-      return if current_count >= max_reviewers
+      return if current_count >= max_reviewers || current_count  == -1
 
       authors = find_authors
       members = team_members(team)
@@ -42,7 +42,7 @@ module Danger
 
       git.modified_files.each do |file|
         result = GitHub::Client.query(GitHub::BlameQuery, variables: { repository: repo, owner: owner, ref: branch, file: file })
-        next if result.data.repository.ref.nil?
+        next if result.data.nil? || result.data.repository.ref.nil?
         result.data.repository.ref.target.blame.ranges.each do |range|
           lines = (range.ending_line - range.starting_line) + 1
           users[range.commit.author.user.login] += lines unless range.commit.author.user.nil?
@@ -73,12 +73,16 @@ module Danger
       owner, repo = env.ci_source.repo_slug.split('/')
 
       result = GitHub::Client.query(GitHub::ReviewerQuery, variables: { repo: repo, owner: owner, number: github.pr_json[:number] })
+      return -1 if result.data.nil?
+
       result.data.repository.pull_request.review_requests.edges.count + result.data.repository.pull_request.reviews.edges.count
     end
 
     def team_members(team)
       owner, repo = env.ci_source.repo_slug.split('/')
       result = GitHub::Client.query(GitHub::MemberQuery, variables: { organization: owner, team: team})
+      return [] if result.data.nil?
+
       result.data.organization.team.members.edges.map { |edge| edge.node.login }
     end
   end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: danger-reviewer
 version: !ruby/object:Gem::Version
-  version: 0.0.4
+  version: 0.0.5
 platform: ruby
 authors:
 - Mikko Kokkonen
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2018-06-11 00:00:00.000000000 Z
+date: 2019-02-05 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: danger-plugin-api
@@ -208,8 +208,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubyforge_project: 
-rubygems_version: 2.7.3
+rubygems_version: 3.0.1
 signing_key: 
 specification_version: 4
 summary: Danger plugin to automatically request reviews from potential users on pull