cstamas commented on PR #913: URL: https://github.com/apache/maven/pull/913#issuecomment-1396560239
Am fine with this go into 3.9 and not impose any change in resolver, and here is why: * there is already this MavenWorkspaceReader that already extends resolver WorkspaceReader by adding "maven specific" calls onto it. * Resolver workspace reader has all of its requirements fulfilled with existing WorkspaceReader interface, so adding methods there ONLY to fulfill some 3rd party (integrating) library is a no go for me (and just like Maven, 3rd party user can extend WorkspaceReader to suit its own needs) * hence, doing this or similar change on resolver WorkspaceReader is IMHO no go and not needed, OTOH, this change here (in "maven realm") is completely OK to me -- 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
