Change skip condition

amblerhq · Jan 24, 2021 · 763c468 · 763c468
1 parent 372f69c
commit 763c468
Show file tree

Hide file tree

Showing 3 changed files with 11 additions and 7 deletions.
diff --git a/dist/index.js b/dist/index.js
diff --git a/dist/index.js.map b/dist/index.js.map
diff --git a/src/main.ts b/src/main.ts
@@ -113,14 +113,16 @@ async function getUniqueReviews(pullRequestNumber: number): Promise<Reviews> {
     pull_number: pullRequestNumber
   })
   const uniqueByUserReviews: Reviews = []
-  for (const review of reviews.reverse()) {
+  for (const candidate of reviews
+    .filter(review => ['CHANGES_REQUESTED', 'APPROVED'].includes(review.state))
+    .reverse()) {
     if (
       !uniqueByUserReviews.find(
         uniqueReview =>
-          review.user && uniqueReview.user?.login === review.user.login
+          candidate.user && uniqueReview.user?.login === candidate.user.login
       )
     ) {
-      uniqueByUserReviews.push(review)
+      uniqueByUserReviews.push(candidate)
     }
   }
   return uniqueByUserReviews