gnodet commented on code in PR #1391:
URL: https://github.com/apache/maven/pull/1391#discussion_r1466213690
##########
api/maven-api-core/src/main/java/org/apache/maven/api/Artifact.java:
##########
@@ -105,6 +105,14 @@ default String key() {
*/
boolean isSnapshot();
+ /**
+ * The artifact properties.
+ *
+ * @return the artifact properties, never {@code null}
+ */
+ @Nonnull
+ ArtifactProperties getArtifactProperties();
Review Comment:
> Sorry, used wrong lingo. You are right "artifact resolve" is really "just
download that single file for that artifact, no deps involved". I was talking
about `collect+resolve` that is the main use case of Resolver, as there, all
the artifacts you end up with (resolved) are dependencies except the root, but
that one as well may have properties if you used `ArtifactType` to construct it.
That's exactly why I think we should keep things split more clearly between
Artifact and Dependency. We have artifacts which are related to repositories
and dependencies for... dependencies. Different words for different meanings.
We should make that more clear rather than blur things...
--
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: issues-unsubscr...@maven.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
