-
Notifications
You must be signed in to change notification settings - Fork 375
chore: migrate build to Bazel 8.5.1 #8602
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
base: master
Are you sure you want to change the base?
Conversation
This bumps our bazel version in `.bazelversion` and upgrades a few rules, macros and targets that need adapting: * In general, `~` was replaced with `+` in (canonical) labels * Some bazel modules dependencies were updated * The `apalache-mc` binary was wrapped in a `java_binary` to let Bazel handle the call to `java` (the old way broke in Bazel 8, and this new way breaks in other ways in Bazel 7 so was not done separately)
64b0c49 to
f3d8b81
Compare
basvandijk
left a comment
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.
🎉
venkkatesh-sekar
left a comment
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.
LGTM for fuzz_testing.bzl
…5dca522c018817ffb48ac26929528c Image tag: 3997bc4a66f90367ea1bbaf46ddb7f8c0debd78d9fb59e1bb5c9006f638d8d9a
|
Run URL: https://github.com/dfinity/ic/actions/runs/21589339419 New container image: |
This bumps our bazel version in
.bazelversionand upgrades a few rules, macros and targets that need adapting:~was replaced with+in (canonical) labelsapalache-mcbinary was wrapped in ajava_binaryto let Bazel handle the call tojava(the old way broke in Bazel 8, and this new way breaks in other ways in Bazel 7 so was not done separately)