Skip to content

ci: require the macOS build again now that macos.rs compiles#18

Merged
falkoro merged 1 commit into
masterfrom
ci/require-macos-again
Jun 11, 2026
Merged

ci: require the macOS build again now that macos.rs compiles#18
falkoro merged 1 commit into
masterfrom
ci/require-macos-again

Conversation

@falkoro

@falkoro falkoro commented Jun 11, 2026

Copy link
Copy Markdown
Owner

Follow-up to #16 / #17: the macOS leg compiles green now, so make it a required check again to prevent regressions. Comment updated to reflect reality.

🤖 Generated with Claude Code

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Copilot AI review requested due to automatic review settings June 11, 2026 06:57
@github-actions github-actions Bot added the ci label Jun 11, 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

Re-enables macOS as a gating CI signal now that the Rust macOS build compiles again, to prevent regressions across all supported platforms.

Changes:

  • Update CI documentation comment to reflect that macOS builds are expected to pass again.
  • Make the macOS matrix leg non-optional by setting allow-failure: false (thus continue-on-error: false) for macOS.

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

Comment thread .github/workflows/ci.yml
Comment on lines +31 to +33
# Cross-platform Rust compile signal. All three platforms must pass:
# windows/linux are this fork's release targets, and macOS compiles again
# since the PanelHandle port (#16, fixed #17).
@falkoro falkoro merged commit e02ce54 into master Jun 11, 2026
6 checks passed
@falkoro falkoro deleted the ci/require-macos-again branch June 11, 2026 07:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants