Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(NODE-6226): build macos universal binary #21

Merged
merged 3 commits into from
Jun 17, 2024
Merged

Conversation

nbbeeken
Copy link
Collaborator

@nbbeeken nbbeeken commented Jun 17, 2024

Description

What is changing?

  • We build one universal binary on macos-latest
  • Copy the tar file to have an x64 name for prebuild's sake
Is there new documentation needed for these changes?

No

What is the motivation for this change?

  • Stop using macos 11 because it will be removed soon

Release Highlight

MacOS prebuilds are now universal binaries

The MacOS prebuilds are now identical tar files that have the same .node file. The contents is a universal binary that can run on ARM and x64 Apple machines.

Double check the following

  • Ran npm run check:lint script
  • Self-review completed using the steps outlined here
  • PR title follows the correct format: type(NODE-xxxx)[!]: description
    • Example: feat(NODE-1234)!: rewriting everything in coffeescript
  • Changes are covered by tests
  • New TODOs have a related JIRA ticket

@nbbeeken nbbeeken requested a review from baileympearson June 17, 2024 20:10
@nbbeeken nbbeeken added Primary Review In Review with primary reviewer, not yet ready for team's eyes labels Jun 17, 2024
@nbbeeken nbbeeken merged commit a044ccd into main Jun 17, 2024
12 checks passed
@nbbeeken nbbeeken deleted the NODE-6226-fix-macos branch June 17, 2024 20:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Primary Review In Review with primary reviewer, not yet ready for team's eyes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants