slawekjaranowski commented on code in PR #12329:
URL: https://github.com/apache/maven/pull/12329#discussion_r3448208880


##########
impl/maven-core/src/main/java/org/apache/maven/plugin/internal/PluginDependenciesResolver.java:
##########
@@ -70,11 +72,63 @@ DependencyNode resolve(
             RepositorySystemSession session)
             throws PluginResolutionException;
 
+    /**
+     * Resolves the runtime dependencies of the specified core extension (as 
{@link Plugin} as GAV carrier).
+     *
+     * @param plugin The plugin for which to resolve the dependencies, must 
not be {@code null}.
+     * @param dependencyFilter A filter to exclude artifacts from resolution 
(but not collection), may be {@code null}.
+     * @param repositories The plugin repositories to use for resolving the 
plugin artifacts, must not be {@code null}.
+     * @param session The repository session to use for resolving the plugin 
artifacts, must not be {@code null}.
+     * @return The dependency resolution result having the resolved extension 
class path but also the tree, never {@code null}.
+     * @throws PluginResolutionException If any dependency could not be 
resolved.
+     * @since 3.10.0

Review Comment:
   maybe we can also add a 4.x version here, or use only 4.x as @gnodet mention



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