Skip to content

Commit b49e961

Browse files
committed
[draft-bug] fix log
1 parent a90871d commit b49e961

File tree

2 files changed

+8
-5
lines changed

2 files changed

+8
-5
lines changed

Diff for: dist/index.js

+1-1
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

Diff for: src/runOnPullRequest.js

+7-4
Original file line numberDiff line numberDiff line change
@@ -91,10 +91,13 @@ const makeReviewRequests = async (
9191
// unfulfilled reviewers = everyone who hasn't reviewed
9292
const unfulfilledReviewers = reviewers.filter(reviewer => !alreadyReviewed.includes(reviewer));
9393

94-
console.log(
95-
`Not adding ${alreadyReviewed.join(', ')} to the review request list as ` +
96-
`they have already reviewed.`,
97-
);
94+
if (reviewers.length > unfulfilledReviewers.length) {
95+
console.log(
96+
`Not adding ` +
97+
`${reviewers.filter(r => !unfulfilledReviewers.includes(r)).join(', ')} ` +
98+
`to the review request list as they have already reviewed.`,
99+
);
100+
}
98101

99102
// List of any folks who have reviewed meaningfully (approved or requested changes), to determine if the
100103
// team reviewing has been fulfilled. Also, don't count the issuer, as they can add comments to their

0 commit comments

Comments
 (0)