-
Notifications
You must be signed in to change notification settings - Fork 33
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
Conversation
LGTM, but I cannot assess the overall impact. I will rely on @erichaagdev having verified it works. |
@erichaagdev do we need to do a similar change for the shared-plugin-extension ? |
0054d24
to
0ceecfb
Compare
I meant the non-shared convention extension. |
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, I could test that locally!
Great fast work Eric 👍
@etiennestuder it only is a problem in the shared sample. In the standalone Maven sample, it is built with Maven and the 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: Lines 13 to 17 in 0ceecfb
We don't include it in the extension built by Gradle in the BVS repo. |
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
.