[java] Use byte-buddy 1.18.8 instead of 1.18.8-jdk5#17382
Open
pierluigilenoci wants to merge 3 commits intoSeleniumHQ:trunkfrom
Open
[java] Use byte-buddy 1.18.8 instead of 1.18.8-jdk5#17382pierluigilenoci wants to merge 3 commits intoSeleniumHQ:trunkfrom
pierluigilenoci wants to merge 3 commits intoSeleniumHQ:trunkfrom
Conversation
The -jdk5 flavor of byte-buddy targets Java versions lower than 8, which is not required by Selenium. Using the -jdk5 variant causes Maven enforcer plugin failures because Maven considers 1.18.8-jdk5 as greater than the managed dependency 1.18.8. Fixes SeleniumHQ#17355 Signed-off-by: Pierluigi Lenoci <pierluigilenoci@gmail.com>
Contributor
Review Summary by QodoUpdate byte-buddy dependency to remove unnecessary JDK5 variant
WalkthroughsDescription• Update byte-buddy dependency from 1.18.8-jdk5 to 1.18.8 • Resolves Maven enforcer convergence errors in downstream projects • Removes unnecessary Java 5 compatibility variant File Changes1. MODULE.bazel
|
Contributor
Code Review by Qodo
1.
|
Update the Bazel Maven lockfile after changing byte-buddy from 1.18.8-jdk5 to 1.18.8. Recomputed input and resolved artifact hashes and updated SHA256 checksums from Maven Central. Signed-off-by: Pierluigi Lenoci <pierluigilenoci@gmail.com>
Author
|
Regenerated Changes in
This is equivalent to running |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
🔗 Related Issues
Fixes #17355
💥 What does this PR do?
Changes the
net.bytebuddy:byte-buddydependency from1.18.8-jdk5to1.18.8inMODULE.bazel.The
-jdk5flavor of byte-buddy is intended for Java versions lower than 8, which Selenium does not target. Using the-jdk5variant causes Maven enforcer plugin failures in downstream projects because Maven considers1.18.8-jdk5as a higher version than the managed dependency1.18.8, triggering convergence errors.Note: The
java/maven_install.jsonlock file will need to be regenerated after this change. A maintainer with Bazel set up can run:🤖 AI assistance
🔄 Types of changes