Bukama commented on code in PR #29:
URL: https://github.com/apache/maven-executor/pull/29#discussion_r3598204876
##########
maven-executor/src/test/java/org/apache/maven/executor/MavenExecutorTestSupport.java:
##########
@@ -400,8 +408,10 @@ protected ExecutorRequest.Builder customizedRequest() {
}
protected ExecutorRequest.Builder
customizedRequest(ExecutorRequest.Builder builder) {
- builder =
-
builder.cwd(cwd).userHomeDirectory(userHome).argument("-Daether.remoteRepositoryFilter.prefixes=false");
+ builder = builder.cwd(cwd)
+ .userHomeDirectory(userHome)
+ .argument("-Daether.remoteRepositoryFilter.prefixes=false")
Review Comment:
Passing this argument is not mentioned in the PR discription and seems
unrelated to MAVEN_ARGS to me? I have in mind that the need to set this flag is
some resolver 1.x/2.x thing - so I guess it was passed via MAVEN_ARGS before?
--
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]
