-
Notifications
You must be signed in to change notification settings - Fork 34
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Deprecated Plexus @Component annotations are removed #1398
base: main
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
role = DevelocityListener.class, | ||
hint = "convention-develocity-maven-extension", | ||
description = "Configures the Develocity Maven extension for com.myorg" | ||
) | ||
public final class ConventionDevelocityMavenExtensionListener implements DevelocityListener { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It would be good to have a consistent naming pattern across all these listener implementations.
- I suggest renaming to simply
ConventionDevelocityListener
- The CCUD listeners in the other PR LGTM
<components> | ||
<component> | ||
<role>com.gradle.develocity.agent.maven.api.DevelocityListener</role> | ||
<role-hint>convention-develocity-maven-extension</role-hint> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I would rename it to convention-develocity-listener
.
role = DevelocityListener.class, | ||
hint = "quarkus-build-cache", | ||
description = "Make the Quarkus build goal cacheable" | ||
) | ||
public final class QuarkusBuildCacheDevelocityListener implements DevelocityListener { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I would rename it to QuarkusBuildCachingDevelocityListener
.
<components> | ||
<component> | ||
<role>com.gradle.develocity.agent.maven.api.DevelocityListener</role> | ||
<role-hint>quarkus-build-cache</role-hint> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I would rename it to quarkus-build-caching-develocity-listener
.
<role>com.gradle.develocity.agent.maven.api.DevelocityListener</role> | ||
<role-hint>quarkus-build-cache</role-hint> | ||
<implementation>com.gradle.QuarkusBuildCacheDevelocityListener</implementation> | ||
<description>Make the Quarkus build goal cacheable</description> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I would rename it to Makes the Quarkus goals cacheable
.
…ocity Shared sample
…ching extension
5d7e8c1
to
a8656d3
Compare
This PR removes the deprecated
@Component
annotations from the Quarkus extension and Maven Convention Develocity extension sample. They have been replaced with a dedicated file atsrc/main/resources/META-INF/plexus/components.xml
instead.