Skip to content

Bugfix/build architecture#30

Merged
mjreed-wbd merged 4 commits intomasterfrom
bugfix/build-architecture
Oct 16, 2025
Merged

Bugfix/build architecture#30
mjreed-wbd merged 4 commits intomasterfrom
bugfix/build-architecture

Conversation

@mjreed-wbd
Copy link
Collaborator

Correct binaries workflow so that it really does build an x86_64 macOS executable.

@mjreed-wbd mjreed-wbd requested a review from Copilot October 15, 2025 18:38
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

Adjusts workflow configuration to correctly target architecture-specific runners for building release binaries, aiming to ensure a true x86_64 macOS build.

  • Replace invalid/unused top-level description keys with comments.
  • Modify OS matrix to explicitly select an Intel macOS runner and introduce an arch dimension.
  • Remove architecture inputs from actions/setup-python and add unused arch matrix entries.

Reviewed Changes

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

File Description
.github/workflows/release.yml Converted unsupported description key into a comment (no functional change).
.github/workflows/binaries.yml Altered OS matrix (added macos-15-intel), added arch matrix entries, removed setup-python architecture inputs.

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@mjreed-wbd mjreed-wbd merged commit 3569033 into master Oct 16, 2025
7 checks passed
@mjreed-wbd mjreed-wbd deleted the bugfix/build-architecture branch October 16, 2025 14:08
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.

2 participants