On Fri, 3 Apr 2026 09:58:54 GMT, serhiysachkov <[email protected]> wrote:

>> Update testng and junit tests to use diagnoseConfigurationIssue() method and 
>> post processing
>
> serhiysachkov has updated the pull request incrementally with two additional 
> commits since the last revision:
> 
>  - fixing merge issue
>  - fixing merge issue

LGTM. If you could make a pass to remove left over imports that might no longer 
be needed that would be great.

test/jdk/java/nio/channels/etc/LocalSocketAddressType.java line 45:

> 43: import java.util.stream.Stream;
> 44: 
> 45: import static java.lang.System.out;

Are these imports still needed? Can you do a global pass on all files and 
remove all unneeded imports?

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

PR Review: https://git.openjdk.org/jdk/pull/30446#pullrequestreview-4055286292
PR Review Comment: https://git.openjdk.org/jdk/pull/30446#discussion_r3032442923

Reply via email to