RFR: 2082: Use correct .jcheck/conf in AdditionalConfiguration.java [v7]
Erik Joelsson
erikj at openjdk.org
Fri Dec 1 18:38:19 UTC 2023
On Fri, 1 Dec 2023 18:28:01 GMT, Erik Duveblad <ehelin at openjdk.org> wrote:
>> Hi all,
>>
>> please review this patch that ensures that we pass the correct `.jcheck/conf` to [AdditionalConfiguration.java](https://github.com/openjdk/skara/blob/master/bots/pr/src/main/java/org/openjdk/skara/bots/pr/AdditionalConfiguration.java). The `.jcheck/conf` should either come from the "overriding" `.jcheck/conf` (if the PR bot is configured to use that) or the pull request's target branch.
>>
>> I have added that test that passes with this patch but fails without it.
>>
>> Thanks,
>> Erik
>
> Erik Duveblad has updated the pull request incrementally with one additional commit since the last revision:
>
> Last Path.of
Marked as reviewed by erikj (Lead).
-------------
PR Review: https://git.openjdk.org/skara/pull/1578#pullrequestreview-1760373211
More information about the skara-dev
mailing list