RFR: 1625: Validate input from /summary command [v3]
Zhao Song
zsong at openjdk.org
Mon Oct 24 17:53:14 UTC 2022
On Mon, 24 Oct 2022 17:28:00 GMT, Erik Joelsson <erikj at openjdk.org> wrote:
> Looking more at this, communicating through an exception seems wrong. The visitor already has a way to communicate problems through the addFailureMessage method. We should just stop throwing exception and treat this as any other "normal" failure.
Yes, this way is better. I was just trying to make bad commit message as an `integrationBlocker`. If we choose the way you mentioned, we also no longer need the `BadCommitMessageException`, right?
-------------
PR: https://git.openjdk.org/skara/pull/1401
More information about the skara-dev
mailing list