[ 
https://issues.apache.org/jira/browse/MRESOLVER-276?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17614756#comment-17614756
 ] 

ASF GitHub Bot commented on MRESOLVER-276:
------------------------------------------

michael-o commented on code in PR #200:
URL: https://github.com/apache/maven-resolver/pull/200#discussion_r990814613


##########
maven-resolver-impl/src/main/java/org/eclipse/aether/internal/impl/DefaultArtifactResolver.java:
##########
@@ -426,6 +442,14 @@ else if ( local.getFile() != null )
             }
         }
 
+        for ( ArtifactResolverPostProcessor artifactResolverPostProcessor : 
artifactResolverPostProcessors.values() )
+        {
+            if ( artifactResolverPostProcessor.postProcess( session, results ) 
)
+            {
+                failures = true;
+            }

Review Comment:
   Since all of them are processed, is it worthwhile to document that PP impls 
shall not rely on any order and/or the result of a previous call?





> Resolver post-processor
> -----------------------
>
>                 Key: MRESOLVER-276
>                 URL: https://issues.apache.org/jira/browse/MRESOLVER-276
>             Project: Maven Resolver
>          Issue Type: Improvement
>          Components: Resolver
>            Reporter: Tamas Cservenak
>            Assignee: Tamas Cservenak
>            Priority: Major
>             Fix For: resolver-next
>
>
> Introduce new feature, resolver post-processor that is able to post process 
> resolution results just before artifact resolver returns them to caller. Post 
> processor should be able to signal resolution failure (along with errors) 
> just like existing resolution may fail.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

Reply via email to