fix: only remove label requested when no other requested reviews

christophehurpeau · Jul 18, 2018 · 298be2a · 298be2a
1 parent a291899
commit 298be2a
Showing 1 changed file with 4 additions and 2 deletions.
diff --git a/index.js b/index.js
@@ -426,8 +426,10 @@ module.exports = app => {
               state === 'changes_requested' && config.labels.review[reviewerGroup].changesRequested,
             ],
             remove: [
-              config.labels.review[reviewerGroup].needsReview,
-              config.labels.review[reviewerGroup].requested,
+              (!hasRequestedReviewsForGroup || state === 'changes_requested') &&
+                config.labels.review[reviewerGroup].needsReview,
+              (!hasRequestedReviewsForGroup || state === 'changes_requested') &&
+                config.labels.review[reviewerGroup].requested,
               state === 'approved' &&
                 !hasRequestedReviewsForGroup &&
                 config.labels.review[reviewerGroup].changesRequested,