Skip to content
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

Maven listener isn't called in shared sample #1394

Merged
merged 3 commits into from
Aug 29, 2024

Conversation

erichaagdev
Copy link
Member

This was missed when creating the shared build. It meant the listener was never called. I never encountered it because I always ran with -Ddevelocity.url=https://ge.solutions-team.gradle.com.

@erichaagdev erichaagdev requested review from etiennestuder and a team August 29, 2024 13:38
@erichaagdev erichaagdev self-assigned this Aug 29, 2024
Copy link
Contributor

github-actions bot commented Aug 29, 2024

Job Requested goals Build Tool Version Build Outcome Build Scan®
verification_maven-[object Object] wrapper:wrapper 3.9.9 Build Scan NOT_PUBLISHED
verification_maven-[object Object] clean verify 3.8.8 Build Scan PUBLISHED
Generated by gradle/develocity-actions

@etiennestuder
Copy link
Member

LGTM, but I cannot assess the overall impact. I will rely on @erichaagdev having verified it works.

@etiennestuder
Copy link
Member

@erichaagdev do we need to do a similar change for the shared-plugin-extension ?

@erichaagdev erichaagdev force-pushed the erichaagdev/fix-maven-listener-not-called branch from 0054d24 to 0ceecfb Compare August 29, 2024 13:40
@etiennestuder
Copy link
Member

LGTM, but I cannot assess the overall impact. I will rely on @erichaagdev having verified it works.

I meant the non-shared convention extension.

Copy link
Member

@jprinet jprinet left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, I could test that locally!
Great fast work Eric 👍

@erichaagdev
Copy link
Member Author

erichaagdev commented Aug 29, 2024

@etiennestuder it only is a problem in the shared sample.

In the standalone Maven sample, it is built with Maven and the plexus-component-metadata plugin generates the component.xml file as part of the build.

This isn't the case in the shared sample, because it is built with Gradle.

Now that I think about it, this is actually redundant:

@Component(
role = DevelocityListener.class,
hint = "convention-develocity-maven-extension",
description = "Configures the Develocity Maven extension for com.myorg"
)

We don't include it in the extension built by Gradle in the BVS repo.

@erichaagdev erichaagdev merged commit 467a73d into main Aug 29, 2024
41 checks passed
@erichaagdev erichaagdev deleted the erichaagdev/fix-maven-listener-not-called branch August 29, 2024 18:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants