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