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

Wire up oneversion tool in java_tools #22246

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

fmeum
Copy link
Collaborator

@fmeum fmeum commented May 6, 2024

  • Add a prebuilt one_version tool as well as sources to java_tools.
  • Avoid passing --whitelist to one_version if no allowlist is configured in the toolchain as it isn't supported by the Bazel version of oneversion yet.
  • Document the one_version flags.
  • Clean up tests not updated after recent rules_java releases.

Work towards #1071

@fmeum fmeum requested a review from lberki as a code owner May 6, 2024 12:02
@fmeum fmeum requested review from cushon and removed request for lberki May 6, 2024 12:02
@github-actions github-actions bot added team-Rules-Java Issues for Java rules awaiting-review PR is awaiting review from an assigned reviewer labels May 6, 2024
* Adds a prebuilt `one_version` tool as well as sources to `java_tools`.
* Avoids passing `--whitelist` to `one_version` if no allowlist is configured in the toolchain, which isn't supported by the Bazel version of `oneversion` yet.
* Documents the `one_version` flags.
* Clean up tests not updated after recent rules_java releases.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
awaiting-review PR is awaiting review from an assigned reviewer team-Rules-Java Issues for Java rules
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant