cstamas commented on PR #194: URL: https://github.com/apache/maven-resolver/pull/194#issuecomment-1257280023
> As far as I understand the Provider approach now is that the actual mappers might be incomplete and the provider will compose them properly to usable mappers? Exactly, not "incomplete" but they are now "simple" each doing one simple thing. And they are NOT components anymore, as providers are actually composing them. This is the "cleanup" I refer to in PR description: before, NameMapper implementations were highly duplicated, for example I'd need to duplicate file basedir resolution for any new file frinedly mapper. But with this PR, not anymore, all you need is new provider and create a new "chain" (well, not like real servlet chain, but you get the point) -- 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
