Allow author to easily reset approvals on PRs

XMLWordPrintable

    • 1

      We've had to disable the "Reset approvals when source branch is modified" merge check as it was generating a great deal of noise in some of our projects (because reasons, we've raised other issues ).

      So, the suggestion I have is that the PR's author should be able to identify the need to reset approvals and request reviewers re-review, regardless of the settings applied.

      So the flow would be:

      1. author raises PR
      2. reviewer approves
      3. author or another approver realises they need to make a substantial change
      4. substantial change is pushed to PR
      5. (here's the suggestion) author hits "Request re-review" kind of button
      6. approvals are reset and reviewers notified of the request

      There are workarounds like declining the whole PR and raising a fresh one; but this would be a nicely nuanced option. I've noticed people are relatively uncomfortable declining PRs but they are quite happy to update and at-mention people requesting re-review.

            Assignee:
            Unassigned
            Reporter:
            Ben Buchanan
            Votes:
            6 Vote for this issue
            Watchers:
            7 Start watching this issue

              Created:
              Updated: