Skip to content

feat: add rules_java to support Bazel 9#77

Merged
bu3 merged 1 commit intoOpenAPITools:masterfrom
tempacc21:master
Feb 2, 2026
Merged

feat: add rules_java to support Bazel 9#77
bu3 merged 1 commit intoOpenAPITools:masterfrom
tempacc21:master

Conversation

@tempacc21
Copy link
Contributor

@tempacc21 tempacc21 commented Jan 22, 2026

Summary by cubic

Add rules_java to enable Bazel 9 support and update the OpenAPI generator to use the new JavaRuntimeInfo provider. Refreshed module lock files to the latest format.

  • Dependencies

    • Added rules_java 9.3.0 to MODULE.bazel.
    • Updated MODULE.bazel.lock files to lockFileVersion 26 across the repo and tests.
  • Refactors

    • Switched from java_common.JavaRuntimeInfo to rules_java’s JavaRuntimeInfo in openapi_generator.bzl.
    • Loaded JavaInfo and JavaRuntimeInfo from rules_java to resolve the Java runtime and executable path under Bazel 9.

Written for commit 987afe0. Summary will update on new commits.

Copy link

@cubic-dev-ai cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

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

No issues found across 6 files

@kberezin-nshl
Copy link

This is great!

@wing328 could you tell us please, when can we expect a release with this?
It currently blocks us from migrating to Bazel 9

@wing328
Copy link
Member

wing328 commented Feb 2, 2026

thanks for the PR

cc @bu3 @BradHolmes for review

Copy link
Contributor

@bu3 bu3 left a comment

Choose a reason for hiding this comment

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

It looks good to me. Thanks @tempacc21.

I will try to have it released on BCR this week and I will give you a shout as soon as I'm done

@bu3 bu3 merged commit 660c6f7 into OpenAPITools:master Feb 2, 2026
2 checks passed
@bu3
Copy link
Contributor

bu3 commented Feb 4, 2026

Hey @tempacc21 @kberezin-nshl,
I released 0.2.3 with your contribution, it was merged in BCR repo a few minutes ago.

It should show up here shortly

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.

4 participants