cstamas commented on code in PR #11249:
URL: https://github.com/apache/maven/pull/11249#discussion_r2428442351
##########
its/core-it-support/maven-it-helper/src/main/java/org/apache/maven/it/Verifier.java:
##########
@@ -108,6 +108,9 @@ public class Verifier {
private final List<String> jvmArguments = new ArrayList<>();
+ // TestSuiteOrdering creates Verifier in non-forked JVM as well, and there
no prop set is set (so use default)
+ private final String toolboxVersion =
System.getProperty("version.toolbox", "0.14.0");
Review Comment:
No, that ctor is deprecated and we do not use it, verifier uses ctor to
explicitly set wanted toolbox. We can remove that ctor in fact, but that would
be a breakage?
--
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]
