Will-thom commented on issue #11378: URL: https://github.com/apache/maven/issues/11378#issuecomment-4492344240
Hi guys and @gnodet I opened a PR with a partial implementation for this proposal: https://github.com/apache/maven/pull/12117 The PR focuses on the smallest Maven core change first: for Maven 4 enhanced parameter configuration, sealed parameter types can now resolve `implementation` hints by permitted subclass simple name, while keeping fully-qualified implementation hints working. Covered in the PR: - `implementation="LocalArtifact"` resolves against `sealed` permitted subclasses. - Fully-qualified implementation hints keep working. - Unknown simple hints fail with a clear error. - Unit coverage in `DefaultBeanConfiguratorTest`. - End-to-end Maven 4 plugin IT for GH-11378. Not covered yet: - A new `@Parameter(defaultImplementation = ...)` API. - Custom type hint annotations such as `@SealedTypeHint`. - Generated plugin documentation/XSD improvements. I kept this intentionally narrow to make the first step easier to review and to avoid introducing new public API before there is maintainer feedback. -- 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]
