Skip to content

Update Bazel version to 8.4.2#105

Merged
nradakovic merged 1 commit into
mainfrom
nira_update_bazel
Jul 16, 2026
Merged

Update Bazel version to 8.4.2#105
nradakovic merged 1 commit into
mainfrom
nira_update_bazel

Conversation

@nradakovic

@nradakovic nradakovic commented Jul 16, 2026

Copy link
Copy Markdown
Member

Update Bazel version to be aligned with reference integration. Add a temporary workaround pinning rules_go to 0.61.1 for running tests with Bazel 9: rules_go is a transitive (not direct) dependency, and the version pulled in transitively does not yet support Bazel 9. Remove once upstream dependencies are updated.

resolves #104

@nradakovic nradakovic self-assigned this Jul 16, 2026
Copilot AI review requested due to automatic review settings July 16, 2026 08:30
@nradakovic nradakovic added p2 Medium/High - key service impacted - needing rapid resolution module dependency update Add or update MODULE.bazel file with new dependency version labels Jul 16, 2026

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Updates the repository’s pinned Bazel version to 8.4.2 (root and tests) and refreshes Bzlmod lockfiles to match the new resolution, including a temporary pin of rules_go in the tests module as a Bazel 9-related workaround.

Changes:

  • Bump .bazelversion and tests/.bazelversion from 8.3.1 to 8.4.2.
  • Update tests/MODULE.bazel dependency pins (platforms, rules_cc) and add a direct rules_go pin (0.61.1) as a temporary workaround.
  • Regenerate MODULE.bazel.lock and tests/MODULE.bazel.lock to reflect updated module and extension digests.

Reviewed changes

Copilot reviewed 3 out of 5 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
.bazelversion Pins Bazel to 8.4.2 at repo root.
tests/.bazelversion Pins Bazel to 8.4.2 for the tests module.
tests/MODULE.bazel Adds a temporary rules_go pin and bumps selected module deps used by tests.
MODULE.bazel.lock Updates lockfile digests to reflect new resolution after Bazel/version changes.
tests/MODULE.bazel.lock Updates the tests module lockfile to reflect the revised dependency graph and digests.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread tests/MODULE.bazel Outdated
Comment thread tests/MODULE.bazel
Comment on lines +32 to 33
bazel_dep(name = "platforms", version = "1.1.0")
bazel_dep(name = "score_bazel_platforms", version = "0.1.2")
Comment thread tests/MODULE.bazel
# C++ Rules for Bazel
# *******************************************************************************
bazel_dep(name = "rules_cc", version = "0.2.14")
bazel_dep(name = "rules_cc", version = "0.2.17")
antonkri
antonkri previously approved these changes Jul 16, 2026
antonkri
antonkri previously approved these changes Jul 16, 2026
Update Bazel version to be aligned with reference integration.
Add a temporary workaround pinning rules_go to 0.61.1 for running
tests with Bazel 9: rules_go is a transitive (not direct)
dependency, and the version pulled in transitively does not yet
support Bazel 9. Remove once upstream dependencies are updated.
@nradakovic
nradakovic merged commit b742f0f into main Jul 16, 2026
12 checks passed
@nradakovic
nradakovic deleted the nira_update_bazel branch July 16, 2026 14:48
@github-project-automation github-project-automation Bot moved this from Backlog to Done in S-CORE Roadmap Jul 16, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

module dependency update Add or update MODULE.bazel file with new dependency version p2 Medium/High - key service impacted - needing rapid resolution

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

Update Bazel version to align with reference integration

3 participants