RFR: Add merge bot
Robin Westberg via github.com
duke at openjdk.java.net
Thu Aug 29 10:22:37 UTC 2019
On Thu, 29 Aug 2019 10:16:20 GMT, Erik Duveblad via github.com <duke at openjdk.java.net> wrote:
> Hi,
>
> this patch adds a merge bot. The merge automatically tries to merge a branch from one repository to another. If the merge fails then the bot will open a PR describing the commits that couldn't be merge. Once the conflicts have been resolved and pushed/integrated by a human contributor, then the bot will automatically close the PR.
>
> ## Testing
> - [x] `sh gradlew test` passes on Linux x86_64
> - [x] Added four new unit tests
>
> Thanks,
> Erik
>
> ----------------
>
> Commits:
> - 7fc5a814: Add merge bot
>
> Pull request:
> https://git.openjdk.java.net/skara/pull/104
>
> Webrev:
> https://webrevs.openjdk.java.net/skara/104/webrev.00
>
> Patch:
> https://git.openjdk.java.net/skara/pull/104.diff
>
> Fetch command:
> git fetch https://git.openjdk.java.net/skara pull/104/head:pull/104
This PR has been reviewed by Robin Westberg via github.com - changes are approved. Review comment:
Looks good, don't need to re-review if you update the merge conflict message!
PR: https://git.openjdk.java.net/skara/pull/104
bots/merge/src/main/java/org/openjdk/skara/bots/merge/MergeBot.java line 168:
> 167: message.add("Push the resulting merge conflict to your personal fork and " +
> 168: "create a pull request towards this repository. Finally close this pull request " +
> 169: "once the pull request with the resolved conflicts has been integrated.");
Since you close the PR automatically, this message should probably be updated.
PR: https://git.openjdk.java.net/skara/pull/104
More information about the skara-dev
mailing list