Skip to content

Conversation

@asimmon
Copy link
Owner

@asimmon asimmon commented Apr 2, 2025

Future-proofing macOS support for arm64 instead of x64:

  • GitHub's macos-latest runner uses arm64
  • More developers will use arm64 chips over time

Modified CI to test on Ubuntu, macOS, and Windows.

@asimmon asimmon requested a review from Copilot April 2, 2025 01:42
Copy link

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

This PR updates the macOS platform support from x64 to arm64 across the codebase. Key changes include:

  • Updating the executable mapping in MongoExecutableLocator.cs to use "osx-arm64".
  • Modifying ProjectInfo definitions in build/Program.cs for macOS to reference arm64.
  • Adjusting the CI workflow to run tests on Ubuntu, Windows, and macOS using updated configurations.

Reviewed Changes

Copilot reviewed 6 out of 11 changed files in this pull request and generated no comments.

File Description
src/EphemeralMongo.Core/MongoExecutableLocator.cs Changed the macOS executable mapping from "osx-x64" to "osx-arm64".
build/Program.cs Updated ProjectInfo entries for macOS to use "osx-arm64" and "arm64".
.github/workflows/ci.yml Modified CI matrix to include macos-latest for testing.
Files not reviewed (5)
  • src/EphemeralMongo.Runtimes/build/copy-runtimes.targets: Language not supported
  • src/EphemeralMongo.sln: Language not supported
  • src/EphemeralMongo6/EphemeralMongo6.csproj: Language not supported
  • src/EphemeralMongo7/EphemeralMongo7.csproj: Language not supported
  • src/EphemeralMongo8/EphemeralMongo8.csproj: Language not supported

@asimmon asimmon marked this pull request as ready for review April 2, 2025 01:43
@asimmon asimmon merged commit cac0d55 into main Apr 2, 2025
12 checks passed
@asimmon asimmon deleted the branch1 branch April 2, 2025 01:51
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