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

Approved label added with 1 approval and 1 change request #244

Open
allout58 opened this issue Jun 5, 2020 · 0 comments
Open

Approved label added with 1 approval and 1 change request #244

allout58 opened this issue Jun 5, 2020 · 0 comments

Comments

@allout58
Copy link
Contributor

allout58 commented Jun 5, 2020

Summary

The approved label is added incorrectly when there is one approving review followed by one changes requested review.

Steps to reproduce

  1. Set up branch protections to require 1 (might not be relevant?)
  2. Request 2 reviews
  3. First reviewer approves the changes (should go to PR: partially-approved, which occurs)
  4. Second reviewer requests changes

Pull Request URL

Private repo unfortunately

What is the current bug behavior?

PR: reviewed-approved label is added

What is the expected correct behavior?

PR: reviewed-changes-requested label is added

Relevant logs and/or screenshots

pr-triage-approval-issue

Possible fixes

(If you have any idea to solve the problem, please help us.)
Most likely related to #223. I'm willing to take a crack at a PR in the future, but a little busy at the moment.

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

No branches or pull requests

1 participant