RFR: 2189: Mark command as handled when JCheckConfiguration is missing or invalid in PullRequestCommandWorkItem [v2]

Zhao Song zsong at openjdk.org
Mon Mar 4 18:01:57 UTC 2024


On Mon, 4 Mar 2024 15:12:42 GMT, Erik Joelsson <erikj at openjdk.org> wrote:

>> Zhao Song has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Update bots/pr/src/main/java/org/openjdk/skara/bots/pr/PullRequestCommandWorkItem.java
>>   
>>   Co-authored-by: Erik Joelsson <37597443+erikj79 at users.noreply.github.com>
>
> bots/pr/src/main/java/org/openjdk/skara/bots/pr/PullRequestCommandWorkItem.java line 229:
> 
>> 227:                         "but it's missing or invalid. Skara admin has been noticed and will fix it as soon as possible.");
>> 228:             }
>> 229:             printer.print("Please issue this command again once the problem has been resolved.");
> 
> I don't think this should be printed on a separate line.

Sure, will fix it.

-------------

PR Review Comment: https://git.openjdk.org/skara/pull/1616#discussion_r1511557969


More information about the skara-dev mailing list