RFR: 1393: PR bot fails on missing .jcheck/conf
Zhao Song
zsong at openjdk.org
Tue Nov 1 17:43:15 UTC 2022
On Tue, 1 Nov 2022 17:32:05 GMT, Erik Joelsson <erikj at openjdk.org> wrote:
> Maybe it would be even better to throw an exception instead of returning Optional.empty(). We can't recover from it in CheckWorkItem anyway, and with a typed exception we can communicate more exactly what went wrong, and we can catch exactly that exception type and write the message.
I thought about it in the very beginning and I found that if we choose to throw an exception, we will need to surround a lot of code with try catch because the scope of `census` is big.
-------------
PR: https://git.openjdk.org/skara/pull/1407
More information about the skara-dev
mailing list