Skip to content

ci: run code-mode unit tests on all bazel targets#28562

Merged
cconger merged 1 commit into
mainfrom
cconger/reenable-code-mode-bazel-tests
Jun 16, 2026
Merged

ci: run code-mode unit tests on all bazel targets#28562
cconger merged 1 commit into
mainfrom
cconger/reenable-code-mode-bazel-tests

Conversation

@cconger

@cconger cconger commented Jun 16, 2026

Copy link
Copy Markdown
Contributor

Why

V8 should be stable under Bazel, so the codex-code-mode unit tests should run across the Bazel platform matrix. If these tests prove unstable, we should fix the tests rather than exclude them from CI.

What changed

  • Remove the explicit //codex-rs/code-mode:code-mode-unit-tests exclusion from the macOS and Linux Bazel test jobs.
  • Remove the same exclusion from the native Windows post-merge job.
  • Keep the existing Windows gnullvm shard coverage.

Bazel test coverage

The target contains 26 unit tests. A fresh uncached local Bazel execution ran all 26 with 0 failures, 0 ignored tests, and 0 filtered tests.

PR Bazel CI selected the target on every enabled platform and reported a cached pass:

Platform Passing CI job
macOS aarch64 Bazel test passed
macOS x86_64 Bazel test passed in 2.2s
Linux GNU Bazel test passed in 0.4s
Linux musl Bazel test passed in 0.4s
Windows gnullvm Bazel test passed in shard 4/4 in 1.6s

@cconger
cconger requested a review from anp-oai June 16, 2026 18:21

@anp-oai anp-oai 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.

Exciting! Would appreciate if you'd point a codex at monitoring main for flakes after this lands but otherwise lgtm!

@cconger
cconger merged commit 009a2bb into main Jun 16, 2026
47 checks passed
@cconger
cconger deleted the cconger/reenable-code-mode-bazel-tests branch June 16, 2026 18:26
@github-actions github-actions Bot locked and limited conversation to collaborators Jun 16, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants