rmannibucau commented on code in PR #1391:
URL: https://github.com/apache/maven/pull/1391#discussion_r1466206736


##########
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:
   We need open properties (my 2cts would be it is `Map<String, Object>`, no 
need to type it stronger) to enable a (custom) type - I include the 
"module-jar" kind of types, ie all but extensions types - to complete the meta 
of a dependency - the old artifact handler somehow which is propagated to 
properties.
   Having or not a wrapper is a style choice but we already have some 
properties (is added to classpath etc) no?



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

Reply via email to