RFR: Add newline after closed issue warning
Jorn Vernee
jvernee at openjdk.java.net
Fri Nov 27 11:00:47 UTC 2020
On Fri, 27 Nov 2020 08:14:41 GMT, Robin Westberg <rwestberg at openjdk.org> wrote:
>> Adds a missing newline after the 'issue is not open' warning, which is currently missing, and leads to the list not being formatted correctly. See e.g. https://github.com/openjdk/jdk/pull/1444
>>
>> 
>>
>> Any testing needed for this?
>
> 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?
-------------
PR: https://git.openjdk.java.net/skara/pull/963
More information about the skara-dev
mailing list