You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
This repository has been archived by the owner on Nov 18, 2021. It is now read-only.
In our workflow, approving Draft PR and a normal PR means different things. Approving a draft PR often means a prototype is deemed good approach for the actual implementation, and it doesn't mean "ready to be merged". So when click "Ready for review", we should make it easier to notify reviewers to look at it again.
A feature where existing approvals can be automatically dismissed upon "Ready for review" will be very helpful.
Current behaviour
When a Draft PR is marked as Ready for review, all approvals remain.
Expected behavior
When a Draft PR is marked as Ready for review, with some configuration, all approvals are to be dismissed automatically.
The text was updated successfully, but these errors were encountered:
Sign up for freeto subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Feature request
In our workflow, approving Draft PR and a normal PR means different things. Approving a draft PR often means a prototype is deemed good approach for the actual implementation, and it doesn't mean "ready to be merged". So when click "Ready for review", we should make it easier to notify reviewers to look at it again.
A feature where existing approvals can be automatically dismissed upon "Ready for review" will be very helpful.
Current behaviour
When a Draft PR is marked as
Ready for review
, all approvals remain.Expected behavior
When a Draft PR is marked as
Ready for review
, with some configuration, all approvals are to be dismissed automatically.The text was updated successfully, but these errors were encountered: