RFR: 1707: Bot warns about "No .jcheck/conf found" when it should not [v5]
Zhao Song
zsong at openjdk.org
Tue Dec 6 19:48:45 UTC 2022
On Tue, 6 Dec 2022 19:34:40 GMT, Zhao Song <zsong at openjdk.org> wrote:
>> forge/src/main/java/org/openjdk/skara/forge/github/GitHubRepository.java line 276:
>>
>>> 274: }
>>> 275: throw e;
>>> 276: }
>>
>> Instead of catching this exception, please use the onError method on the request to register an error handler.
>
> Sorry, I couldn't find a good way to use errorHandler with the current `RestRequest#transformBadResponse`.
> We will not be able to return `Optional.empty()` when the error is 404 Not Found.
Ah, see your comments below. I know how to do it now. Thank you
-------------
PR: https://git.openjdk.org/skara/pull/1435
More information about the skara-dev
mailing list