RFR: 1584: Require re-review of a PR if the target branch changes [v2]
Erik Joelsson
erikj at openjdk.org
Fri Sep 23 20:00:24 UTC 2022
> This change improves reviewer handling when the target branch of a PR is changed. Currently this only invalidates a review on GitHub, but not on GitLab. With this change the behavior is made uniform. In addition to this, the following new behavior is added.
>
> 1. When the target branch of a PR is changed, any existing reviews in the Reviewers list will be flagged as "Re-review required (review applies to pull request targeting <branch>)" to make what happened clearer.
> 2. If the target branch of a PR is changed back to a previously targeted branch, any reviews made against that branch may now become valid again (if all other requirements are met).
>
> Note that changes to and from pr/X branches are ignored, just like before.
>
> PullRequest::reviews is no longer specifying the order of reviews returned. This order was actually not consistent between implementations and I would rather we use explicit sorts when needed instead of assuming an order.
>
> I'm not really happy with the placement of the new static method `calculateReviewTargetRefs`, but I couldn't really find a better location, and I really wanted to share this between different `PullRequest` implementations. I think the correct solution would be to introduce an `AbstractPullRequest` common super class, but I didn't want to do this for a single method.
>
> I have verified this with the new and modified tests, as well as by running against the playground repo(s).
Erik Joelsson has updated the pull request with a new target base due to a merge or a rebase. The pull request now contains two commits:
- Merge branch 'master' into SKARA-1584-rereview-targetref
- SKARA-1584
-------------
Changes: https://git.openjdk.org/skara/pull/1383/files
Webrev: https://webrevs.openjdk.org/?repo=skara&pr=1383&range=01
Stats: 299 lines in 13 files changed: 246 ins; 13 del; 40 mod
Patch: https://git.openjdk.org/skara/pull/1383.diff
Fetch: git fetch https://git.openjdk.org/skara pull/1383/head:pull/1383
PR: https://git.openjdk.org/skara/pull/1383
More information about the skara-dev
mailing list