Skip to content

ci: drop Windows and macOS runners#10845

Open
upbqdn wants to merge 3 commits into
mainfrom
nuke-win-mac
Open

ci: drop Windows and macOS runners#10845
upbqdn wants to merge 3 commits into
mainfrom
nuke-win-mac

Conversation

@upbqdn

@upbqdn upbqdn commented Jun 28, 2026

Copy link
Copy Markdown
Member

Motivation

Closes #10844

Windows and macOS unit-test runners are unnecessary CI overhead. No relevant node operators run those OSes; other enthusiasts should use Docker.

Solution

Drop the Windows/macOS unit-test matrix and the brew/choco install branches from setup-zebra-build. Linux only.

Demote macOS and Windows from tier 2 to tier 3 in the platform-support docs, since CI no longer builds or tests them.

AI Disclosure

  • AI tools were used: Claude for the CI edits, docs, and PR text.

Remove the Windows/macOS unit-test matrix and the brew/choco install
branches from setup-zebra-build. Unnecessary CI overhead: no relevant
node operators run those OSes, and other enthusiasts can use Docker.

Claude-Session: https://claude.ai/code/session_01VUPwpe81Ws8c5MeXWWgZb7
Copilot AI review requested due to automatic review settings June 28, 2026 11:05
@upbqdn upbqdn self-assigned this Jun 28, 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.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

CI no longer builds or tests them, so they no longer meet the tier 2
"guaranteed to build" bar. Tier 3 = supported in code, untested, may or
may not work — use Docker.

Claude-Session: https://claude.ai/code/session_01VUPwpe81Ws8c5MeXWWgZb7

@upbqdn upbqdn left a comment

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

This turned out to be a lot more trivial than I thought. Adding any OS back should be trivial as well, then, if we ever find we need it back.

@upbqdn upbqdn requested review from andres-pcg and Copilot June 28, 2026 11:23

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.

Copilot wasn't able to review any files in this pull request.

Remove the `cfg(not(windows))` / `cfg(not(macos))` skips and the nextest
`delete_old_databases` Windows filter, now that neither platform runs in
CI. For feature-combined gates, keep the `#[cfg(feature = ...)]` and drop
only the OS predicate. Genuine platform-abstraction code (`stop()`,
`random_known_port`, `default_lwd_cache_dir`) is left intact.

Claude-Session: https://claude.ai/code/session_01VUPwpe81Ws8c5MeXWWgZb7
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.

Drop Windows and macOS from CI

2 participants