On Wed, 8 Oct 2025 15:16:47 GMT, Mikhail Yankelevich <[email protected]> 
wrote:

>> Hi @myankelev, can you just take a look at all our tests that call 
>> `ArtifactResolver.fetchOne` to see if it can directly throw a 
>> `SkippedException`? Thanks.
>
> @wangweij `ArtifactResolver.fetchOne` is called in a couple of places but 
> calls different artifacts. I think if it is changed to throw skipped 
> exception itself it might end up not very descriptive. Also, it throws an 
> IOException if it can't find a file.

Sorry for the delay response. I have update the Exception messages, can you 
review again.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/27491#discussion_r2415496330

Reply via email to