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

Improve macOS CI #1411

Merged
merged 1 commit into from
Oct 31, 2024
Merged

Conversation

waywardmonkeys
Copy link
Contributor

  • Use same capitalization as GitHub: macOS-* is now macos-*
  • build-cmake: Now depends on build-macos in addition to build-linux so that a build failure in build-macos will stop this step from running, just like already happens for Linux.
  • build-cmake, test-c-unix, test-cmake: Run on macos-13 and macos-latest. This was running on macos-14 and macos-latest which is from when macos-latest was still macos-13. We keep macos-13 for now since this is still an Intel build while macos-14 and later are Apple Silicon hardware.
  • test-cpp-macos: Remove macos-12 as this will be turned off in the coming months at GitHub. Add macos-15 which is in public preview. Update Xcode versions to the latest for each major version and run some of the older ones on macos-13 now that macos-12 is gone.

Information about the pending removal of macos-12 can be found here: actions/runner-images#10721

Information about Xcode releases can be found here: https://xcodereleases.com

* Use same capitalization as GitHub: `macOS-*` is now `macos-*`
* `build-cmake`: Now depends on `build-macos` in addition to `build-linux`
  so that a build failure in `build-macos` will stop this step from
  running, just like already happens for Linux.
* `build-cmake`, `test-c-unix`, `test-cmake`: Run on `macos-13` and
  `macos-latest`. This was running on `macos-14` and `macos-latest`
  which is from when `macos-latest` was still `macos-13`. We keep
  `macos-13` for now since this is still an Intel build while
  `macos-14` and later are Apple Silicon hardware.
* `test-cpp-macos`: Remove `macos-12` as this will be turned off in the
  coming months at GitHub. Add `macos-15` which is in public preview.
  Update Xcode versions to the latest for each major version and run
  some of the older ones on `macos-13` now that `macos-12` is gone.
  Additionally, remove Xcode 13.x as it isn't supported by the
  `setup-xcode` action on `macos-13`.

Information about the pending removal of `macos-12` can be found
here: actions/runner-images#10721

Information about Xcode releases can be found here:
https://xcodereleases.com
@SanderMertens
Copy link
Owner

Thanks for the PR!

@SanderMertens SanderMertens merged commit 54bf4ae into SanderMertens:master Oct 31, 2024
72 checks passed
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