RFR: 430: /summary command does not support multi-line summaries [v3]

Erik Helin ehelin at openjdk.java.net
Wed Jun 24 16:11:36 UTC 2020


On Wed, 24 Jun 2020 13:30:48 GMT, Erik Helin <ehelin at openjdk.org> wrote:

>> bots/pr/src/main/java/org/openjdk/skara/bots/pr/SummaryCommand.java line 64:
>> 
>>> 63:         } else {
>>> 64:             // A single-line summary
>>> 65:             var summary = args.strip();
>> 
>> Check if lines.size > 1 here? Perhaps that should be an error / warning?
>
> Hm, yeah, good comment. Let me fix that!

Fixed in 202a7d3

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

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


More information about the skara-dev mailing list