RFR: pr: detect clean backports [v2]
Robin Westberg
rwestberg at openjdk.java.net
Fri Oct 16 10:02:25 UTC 2020
On Fri, 16 Oct 2020 09:59:58 GMT, Erik Helin <ehelin at openjdk.org> wrote:
>> Hi all,
>>
>> please review this patch that adds support to `CheckRun` for identifying "clean" backport pull requests. A backport
>> pull request is "clean" if the diff of the pull request is the same as the diff for the original _modulo_ exact lines
>> being changed. That is, the same backport pull request must contain the same _changes_ as the original commit, but the
>> changes don't have to be applied to the exact same lines. The result is equivalent to running `patch` and passing a
>> fuzzing factor via `--fuzz` that is the same as the number of context lines. A clean backport also doesn't require
>> reviews. Note though that non-Committers still need a sponsor. I plan to make this configurable in a follow-up patch.
>> Testing:
>> - [x] Added a bunch of tests
>> - [x] `make test` passes on Linux x64
>>
>> Thanks,
>> Erik
>
> Erik Helin has updated the pull request incrementally with one additional commit since the last revision:
>
> Fix spelling
>
> Co-authored-by: Robin Westberg <robin at westberg.se>
Looks good!
bots/pr/src/main/java/org/openjdk/skara/bots/pr/CheckRun.java line 962:
> 960: try {
> 961: // Do not pass eventual original commit even for backports since it will cause
> 962: // the reviwer check to be ignored.
Suggestion:
// the reviewer check to be ignored.
-------------
Marked as reviewed by rwestberg (Reviewer).
PR: https://git.openjdk.java.net/skara/pull/885
More information about the skara-dev
mailing list