-
Notifications
You must be signed in to change notification settings - Fork 13
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
Add test codifying behavior when two modules reuse the same extension number #171
Add test codifying behavior when two modules reuse the same extension number #171
Conversation
…e extension number
@@ -1,7 +1,7 @@ | |||
buildscript { repositories { gradlePluginPortal() } } | |||
|
|||
plugins { | |||
id("net.vivin.gradle-semantic-build-versioning") version "4.0.0" | |||
id("io.alcide.gradle-semantic-build-versioning") version "4.2.2" |
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 was having trouble running Gradle locally. This fixed the issue and is the spiritual successor to the unmaintained original plugin we use currently.
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 don't have experience with this plugin, but since it is now a fork of a fork we may encounter future ownership issues again at some point.
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.
Looks good!
@@ -1,7 +1,7 @@ | |||
buildscript { repositories { gradlePluginPortal() } } | |||
|
|||
plugins { | |||
id("net.vivin.gradle-semantic-build-versioning") version "4.0.0" | |||
id("io.alcide.gradle-semantic-build-versioning") version "4.2.2" |
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 don't have experience with this plugin, but since it is now a fork of a fork we may encounter future ownership issues again at some point.
It looks like this isn't something that will be configurable in Buf: bufbuild/buf#2688 But I'm going to merge this anyways since it's good to nail it down in case this does ever change. |
Beginning the process of addressing envoyproxy/java-control-plane#296.
The plugin should be able to be configured to ignore these conflicts since the user is not in control of the bad actor dependency.
The failing tests are due to user-level failure, but in principle (and this is what happened) one may have dependencies that conflict and there's nothing you can do.