RFR: SKARA-1096: New command and label for JEPs, similar to CSR [v12]

Guoxiong Li gli at openjdk.java.net
Thu Apr 21 17:56:34 UTC 2022


On Thu, 21 Apr 2022 17:14:28 GMT, Erik Joelsson <erikj at openjdk.org> wrote:

>> Guoxiong Li has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Use static import.
>
> bots/jep/src/main/java/org/openjdk/skara/bots/jep/JEPBot.java line 74:
> 
>> 72:             if (jepComment == null) {
>> 73:                 if (pr.labelNames().contains(JEP_LABEL)) {
>> 74:                     pr.removeLabel(JEP_LABEL);
> 
> Missed this remove label in the previous review due to the long thread in the middle of the block. Please add logging here too.

I missed it, too. Added. Use the `info` level instead of `fine` because the condition only occurs when the developers add the `jep` label manually in the PR.

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

PR: https://git.openjdk.java.net/skara/pull/1297


More information about the skara-dev mailing list