gnodet commented on PR #1337: URL: https://github.com/apache/maven/pull/1337#issuecomment-1858101613
I'm not sure I'm buying the use of the additional indirection for the `VersionSchemeSelector`. I wonder if it would be just as simple to be injected with a `VersionScheme`. If users want to use a different scheme, they just need to register one with a higher `@Priority`. And if we want pluggability, we could also define a `@SessionSCoped` component that would delegate to the `VersionSchemeSelector` underneath. But at least, components would simply be injected with a single `VersionScheme`. -- 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
