Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(assign-rfc-reviewer):Avoid re-requesting reviews #10

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

jgpacker
Copy link

When this action re-requests review from a team, it's PR Review status is set to "Pending", even if it was "Commented"/"Approved"/"Requested Changes" before.
The PR Review status shouldn't be automatically changed.

Also, there can be false-positive cases for the keywords search.
Now, when a false-positive is detected by a team, they can just make a "Comment Review" in the PR.
By doing this, the slack bot "Github Scheduled Reminders" shouldn't notify the team about these false-positives.

When this action re-requests review from a team, it's PR Review status is set to "Pending", even if it was "Commented"/"Approved"/"Requested Changes" before.
The PR Review status shouldn't be automatically changed.

Also, there can be false-positive cases for the keywords search.
Now, when a false-positive is detected by a team, they can just make a "Comment Review" in the PR.
By doing this, the slack bot "Github Scheduled Reminders" shouldn't notify the team about these false-positives.
@jgpacker jgpacker requested a review from rodrigowirth June 21, 2022 20:21
@jgpacker jgpacker marked this pull request as ready for review June 21, 2022 20:21
@jgpacker jgpacker marked this pull request as draft June 21, 2022 21:08
@jgpacker
Copy link
Author

Endpoint GET /repos/{owner}/{repo}/pulls/{pull_number}/requested_reviewers only contains teams with "Pending" PR Reviews, so this approach doesn't work.

Another alternative could be to get the PR reviews from https://docs.github.com/rest/pulls/reviews#list-reviews-for-a-pull-request and look up the team for each member ( https://docs.github.com/pt/rest/teams/members ).
To be done in another day

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant