RFR: 1855: Make it configurable which repositories merges should be allowed from
Erik Joelsson
erikj at openjdk.org
Tue Apr 18 15:51:24 UTC 2023
On Mon, 17 Apr 2023 21:29:24 GMT, Zhao Song <zsong at openjdk.org> wrote:
> In this patch, PR bot has been added with the capability to control which repositories can be the source for Merge-style pull requests.
>
> By default, all repositories are allowed.
>
> However, if it is desired to configure "allowed" repos, the configuration can be added to the PR bot configuration as follows:
>
> {
> "pr": {
> "repositories": {
> "repo1": {
> "mergeSources": [openjdk/repo2, openjdk/repo3]
> }
> }
> }
> }
>
>
> And the repo itself can always be the source of Merge-style pull requests.
Looks good, just a minor nit.
bots/pr/src/main/java/org/openjdk/skara/bots/pr/CheckWorkItem.java line 319:
> 317: var mergeSourceInvalidText = "<!-- merge error -->\n" +
> 318: ":warning: @" + pr.author().username() + " " + repoName +
> 319: " can not be source repo for Merge-style pull requests in this repository.\n" +
Suggestion:
" can not be source repo for merge-style pull requests in this repository.\n" +
-------------
Marked as reviewed by erikj (Lead).
PR Review: https://git.openjdk.org/skara/pull/1503#pullrequestreview-1390439424
PR Review Comment: https://git.openjdk.org/skara/pull/1503#discussion_r1170240357
More information about the skara-dev
mailing list