Bukama commented on code in PR #11353:
URL: https://github.com/apache/maven/pull/11353#discussion_r2473149202
##########
its/core-it-suite/src/test/java/org/apache/maven/it/MavenITmng7470ResolverTransportTest.java:
##########
@@ -107,10 +105,11 @@ private void performTest(/* nullable */ final String
transport, final String log
private static final String JDK_LOG_SNIPPET = "[DEBUG] Using transporter
JdkTransporter";
/**
- * Returns {@code true} if JDK HttpClient transport is usable (Java11 or
better).
+ * Returns {@code true} if JDK HttpClient transport is usable.
+ * Since Maven 4 requires JDK 17, and JDK transport is usable since JDK
11, this always returns true.
*/
private boolean isJdkTransportUsable() {
- return JDK_TRANSPORT_USABLE_ON_JDK_SINCE.compareTo(getJavaVersion()) <
1;
+ return true;
Review Comment:
Can't we remove this method at all then?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
