RFR: 1625: Validate input from /summary command

Zhao Song zsong at openjdk.org
Fri Oct 21 21:52:18 UTC 2022


On Fri, 21 Oct 2022 21:31:30 GMT, Erik Joelsson <erikj at openjdk.org> wrote:

> > If exceptions are thrown when the bot is evaluating the current status of the pr, the exceptions will be treated as integration blocker.
> 
> I didn't mean that any exception should be an integration blocker. I just wanted the very specific case when parsing the commit message fails to be one. For everything else, we should still let the exception go through.

Ok, my bad. I was thinking it is a little difficult to catch the specific exception. I look through the code again and find if the commit message is bad, `PullRequestCheckIssueVisitor` will throw `IllegalStateException`, so we just need to catch `IllegalStateException`, right?

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

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


More information about the skara-dev mailing list