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

Add support for Bzlmod, Bazel 7/8, Ubuntu 24.04 (fixes #515, fixes #516, fixes #514) #520

Merged
merged 1 commit into from
Dec 18, 2024

Conversation

smolkaj
Copy link
Member

@smolkaj smolkaj commented Dec 18, 2024

I tried manually that both

USE_BAZEL_VERSION="8.x" bazel build //...

and

USE_BAZEL_VERSION="7.x" bazel build //...

work. #521 extends our CI scripts to test Bazel 7/8 and Ubuntu 22.04/24.04 automatically.

@smolkaj smolkaj force-pushed the bzlmod2 branch 6 times, most recently from 3291728 to 0db5b5c Compare December 18, 2024 03:04
@smolkaj smolkaj changed the title Add support for Bzlmod (fixes #515) Add support for Bzlmod and Bazel 7/8 (fixes #515 and #516) Dec 18, 2024
@smolkaj smolkaj changed the title Add support for Bzlmod and Bazel 7/8 (fixes #515 and #516) Add support for Bzlmod and Bazel 7/8 (fixes #515, fixes #516) Dec 18, 2024
@smolkaj smolkaj force-pushed the bzlmod2 branch 2 times, most recently from caaddaa to 662e950 Compare December 18, 2024 03:48
@smolkaj smolkaj marked this pull request as ready for review December 18, 2024 03:48
@smolkaj smolkaj requested review from jonathan-dilorenzo and removed request for jonathan-dilorenzo December 18, 2024 05:21
@smolkaj smolkaj changed the title Add support for Bzlmod and Bazel 7/8 (fixes #515, fixes #516) Add support for Bzlmod, Bazel 7/8, Ubuntu 24.04 (fixes #515, fixes #516, fixes #514) Dec 18, 2024
Copy link
Contributor

@jonathan-dilorenzo jonathan-dilorenzo left a comment

Choose a reason for hiding this comment

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

Amazing work! Thanks!

@smolkaj
Copy link
Member Author

smolkaj commented Dec 18, 2024

@chrispsommers let me know if you also want to take a look before we merge this.

Copy link
Collaborator

@chrispsommers chrispsommers left a comment

Choose a reason for hiding this comment

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

LGTM, I'm not qualified to review it technically, but I have confidence in your and Jonathan's assessment. Thanks!

@chrispsommers chrispsommers merged commit da2e344 into main Dec 18, 2024
6 checks passed
@chrispsommers chrispsommers deleted the bzlmod2 branch December 18, 2024 18:02
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.

3 participants