Skip to content

Add builds for macOS - #21

Open
dwhjames wants to merge 1 commit into
microsoft:mainfrom
dwhjames:darwin_builds
Open

Add builds for macOS#21
dwhjames wants to merge 1 commit into
microsoft:mainfrom
dwhjames:darwin_builds

Conversation

@dwhjames

@dwhjames dwhjames commented Nov 5, 2025

Copy link
Copy Markdown

Include darwin amd64 & arm64 in binary releases.

@dwhjames

dwhjames commented Nov 6, 2025

Copy link
Copy Markdown
Author

@microsoft-github-policy-service agree company="Gemini"

@ashort96

Copy link
Copy Markdown

@micromaomao sorry for the cold ping on this PR; is there a better way to have this PR reviewed? It would be great to have darwin native binaries rather than relying on only using these in docker.

@micromaomao micromaomao left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

lgtm, thanks!

@micromaomao

Copy link
Copy Markdown
Member

can't merge due to conflict

Include darwin amd64 & arm64 in binary releases.
@dwhjames

Copy link
Copy Markdown
Author

@micromaomao thanks for your attention; the conflicts have been resolved.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Adds macOS (darwin) cross-compiled binaries to the GitHub Actions release workflow so tagged releases include both amd64 and arm64 macOS executables alongside existing Linux and Windows artifacts.

Changes:

  • Build darwin/amd64 and darwin/arm64 binaries in the release workflow (with CGO_ENABLED=0).
  • Upload the new macOS binaries as workflow artifacts.
  • Attach the macOS binaries to the draft GitHub release assets.

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

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.

4 participants