RFR: Add newline after closed issue warning [v2]
Robin Westberg
rwestberg at openjdk.java.net
Fri Nov 27 12:05:26 UTC 2020
On Fri, 27 Nov 2020 10:58:36 GMT, Jorn Vernee <jvernee at openjdk.org> wrote:
>> bots/pr/src/main/java/org/openjdk/skara/bots/pr/CheckRun.java line 487:
>>
>>> 485: progressBody.append(" ⚠️ Issue is not open.\n");
>>> 486: }
>>> 487: continue;
>>
>> I know I wrote this, but another solution would be to drop the `continue` here, I don't remember why it's there. :)
>
> Ok, will do that than. (seems a little cleaner to me). I think that will only change whether the `if` below is evaluated. Maybe you wanted only one warning to be printed at a time?
Yeah I guess, but then it was a bit inconsistent with the rest of the warnings anyway... So this looks good to me!
-------------
PR: https://git.openjdk.java.net/skara/pull/963
More information about the skara-dev
mailing list