Skip to content

Replace Windows Cargo check with Bazel CI - #4358

Draft
ScriptedAlchemy with Copilot wants to merge 3 commits into
masterfrom
copilot/replace-ci-windows-cargo-check
Draft

ScriptedAlchemy with Copilot wants to merge 3 commits into
masterfrom
copilot/replace-ci-windows-cargo-check

Conversation

Copilot AI commented Oct 19, 2026 •

Copy link
Copy Markdown
Contributor

Summary

  • Build the Bazel workspace on native Windows for every pull request and push, covering cfg(windows) code.
  • Keep the full Windows test suite opt-in via workflow_dispatch with run_os=true.

Motivation

The Linux Cargo check was the only automatic Windows compilation path, outside the workspace’s Bazel build authority.

Changes

  • Replace the Clippy job’s Cargo check and Cargo-only setup with a native Windows Bazel build:
    bazel build --config=ci //...
  • Reuse the existing Windows runner; retain its full Bazel test journey for explicit OS dispatches.

Test plan

  • The affected Bazel targets pass
  • bazel build //... passes when the change crosses package boundaries
  • Tested manually (describe below if applicable)

Checklist

  • CHANGELOG.md updated (under [Unreleased] if no version bump)
  • No secrets, credentials, or .env files included
  • Breaking changes documented (if any)

@changeset-bot

changeset-bot Bot commented Oct 19, 2026 •

Copy link
Copy Markdown

⚠️ No Changeset found

Latest commit: e4e019c

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

Copilot AI and others added 2 commits October 19, 2026 04:08
Co-authored-by: ScriptedAlchemy <25274700+ScriptedAlchemy@users.noreply.github.com>
Co-authored-by: ScriptedAlchemy <25274700+ScriptedAlchemy@users.noreply.github.com>
Copilot AI changed the title [WIP] Replace CI's Windows cargo check with Linux-hosted Bazel check Replace Windows Cargo check with Bazel CI Oct 19, 2026
Copilot AI requested a review from ScriptedAlchemy October 19, 2026 04:11

This branch has not been deployed

No deployments
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.

Replace CI's Windows cargo check with a Linux-hosted Windows Bazel check

2 participants