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]