RFR: 1625: Validate input from /summary command [v6]

Erik Joelsson erikj at openjdk.org
Mon Oct 24 21:07:57 UTC 2022


On Mon, 24 Oct 2022 20:50:58 GMT, Zhao Song <zsong at openjdk.org> wrote:

>> 1.  Added validation to SummaryCommand.
>> 2. If exceptions are thrown when the bot is evaluating the current status of the pr, the exceptions will be treated as integration blocker.
>
> Zhao Song has updated the pull request incrementally with one additional commit since the last revision:
> 
>   won't print commit message when adding failure message

bots/pr/src/main/java/org/openjdk/skara/bots/pr/PullRequestCheckIssueVisitor.java line 239:

> 237:     public void visit(MessageIssue issue) {
> 238:         var message = String.join("\n", issue.commit().message());
> 239:         log.info("Incorrectly formatted commit message: " + message);

This warrants a warning level message I think. We really don't expect this to happen.

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

PR: https://git.openjdk.org/skara/pull/1401


More information about the skara-dev mailing list