I've come across this problem many times this week when a developer has raised a pull request in our codebase and someone has requested urgent changes. These changes are then made and pushed, but the "changes requested" flags still remain.
To combat this, should PR authors be given the ability to request a re-review once an update to the branch has been made, or should these flags just disappear when a new commit has been added since the flag was added – what do people think?
I feel the need for a "re-review" request that could be made by the developer.
Using the "Changes Requested" removes the PR from the "needs my review" list, and it should be added back there once the changes are made
You must be a registered user to add a comment. If you've already registered, sign in. Otherwise, register and sign in.