wilx commented on code in PR #1667:
URL: 
https://github.com/apache/maven-dependency-plugin/pull/1667#discussion_r3698817255


##########
src/main/java/org/apache/maven/plugins/dependency/fromConfiguration/ArtifactItem.java:
##########
@@ -53,6 +53,16 @@ public class ArtifactItem implements DependableCoordinate {
     @Parameter
     private String version = null;
 
+    /**
+     * Classpath scope from which to infer a missing version when the artifact 
is not found in direct dependencies or
+     * dependency management. Supported values are {@code compile}, {@code 
runtime}, and {@code test}. When omitted,
+     * all three dependency graphs are considered and must agree on the 
selected version.

Review Comment:
   No. This entire patch is strictly improving the situation. Before the patch, 
you had to add the version explicitly. With the patch, you can omit the version 
and it will get resolved but it has to be unambiguous. 



-- 
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]

Reply via email to