danger-reviewer 0.0.3 → 0.0.4

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 9c086304da503565e7523ba4fd203c7dbffd58d9ca46b04eee2c9fc87e98818c
-  data.tar.gz: 2c2c3913f9a401f4de921977e2945bdc7780bea9df79d3bc02aaf9da557c0ae7
+  metadata.gz: 22232c98b56cf47802bd550a4cd785d01ec19c08d4af87d08d99426f4acdc2ae
+  data.tar.gz: e5699cd16c9b3507b68a1135a6db5e295b2a8e3fbb659a438dccc8a0320798e1
 SHA512:
-  metadata.gz: 0f946629afb02b2508f770abb971af05cd1ff4a4fcc4e92b8c103bd4edae8a9ed50bf8cbdc16b5e5deb1735dac3e6e5388287eac93a8861e7bb28b1040e346f5
-  data.tar.gz: b4f95a32ac94b35158f4da8de3a6eb94939d78e9e8e80c1d13a05a624f75417d0f02f9fd503dfe4bf9cbb95e8535bb745b94456575e077e5b23e653d2710cd07
+  metadata.gz: a4f2dc88efe9d83bbf96548c6ee9212f93318009c7c4f4a63c6e10c49b531328792c3532bcd5917a0dbf39ba001515360e03067c28486ce15756b8114ebfe53b
+  data.tar.gz: 5afd46cf2fe3af636c5ab7f136ef8d1aaa0078e6432115e240e0144297a6a035cf7462e2b4a6466a1e3791f8049abb47c2d10616ba3866de443fc4317f5ff0f0

data/lib/reviewer/gem_version.rb

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

data/lib/reviewer/github.rb

@@ -70,18 +70,14 @@ module GitHub
           reviewRequests(first: 10) {
             edges {
               node {
-                reviewer {
-                  login
-                }
+                id
               }
             }
           }
           reviews(first: 10) {
             edges {
               node {
-                author {
-                  login
-                }
+                id
               }
             }
           }

data/lib/reviewer/plugin.rb

@@ -6,14 +6,14 @@ require_relative './github'
 module Danger
   class DangerReviewer < Plugin
     def assign(team, max_reviewers = 2, user_blacklist = [])
-      current = current_reviewers
+      current_count = current_reviewers_count
 
       # Check if we already have enough reviewers
-      return if current.count >= max_reviewers
+      return if current_count >= max_reviewers
 
       authors = find_authors
       members = team_members(team)
-      reviewers = find_reviewers((authors & members), members, user_blacklist, (max_reviewers - current.count))
+      reviewers = find_reviewers((authors & members), members, user_blacklist, (max_reviewers - current_count))
 
       request_reviews(reviewers)
     end
@@ -69,14 +69,11 @@ module Danger
       reviewers
     end
 
-    def current_reviewers
+    def current_reviewers_count
       owner, repo = env.ci_source.repo_slug.split('/')
 
       result = GitHub::Client.query(GitHub::ReviewerQuery, variables: { repo: repo, owner: owner, number: github.pr_json[:number] })
-      [
-        result.data.repository.pull_request.review_requests.edges.map { |edge| edge.node.reviewer.login },
-        result.data.repository.pull_request.reviews.edges.map { |edge| edge.node.author.login },
-      ].flatten.uniq
+      result.data.repository.pull_request.review_requests.edges.count + result.data.repository.pull_request.reviews.edges.count
     end
 
     def team_members(team)

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: danger-reviewer
 version: !ruby/object:Gem::Version
-  version: 0.0.3
+  version: 0.0.4
 platform: ruby
 authors:
 - Mikko Kokkonen
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2018-05-29 00:00:00.000000000 Z
+date: 2018-06-11 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: danger-plugin-api