Skip to content

Conversation

@matthewfeickert
Copy link
Contributor

Types of changes

Motivation and context / Related issue

x86 macOS should still be tested.

How has this been tested (if it applies)

PR checklist

  • I have read the CONTRIBUTING document.
  • The documentation is up-to-date with the changes I made (check build artifacts).
  • All tests passed, and additional code has been covered with new tests.
  • [N/A] I have added the PR and Issue fix to the RELEASES.md file.
* Use the 'macos-latest' runs-on option which now defaults to 'macos-14' which are Apple silicon runners. - c.f. https://github.com/actions/runner-images/blob/d31241d2c8c646cd0ba18579836636e62c38f1c9/images/macos/macos-14-arm64-Readme.md * Keep a 'macos-13' runner to continue to test x86 based macOS for the latest Python version. - c.f. https://github.com/actions/runner-images/blob/d31241d2c8c646cd0ba18579836636e62c38f1c9/images/macos/macos-13-Readme.md
@matthewfeickert
Copy link
Contributor Author

This is passing CI (the "1 expected check" is because this changes the name of the required tests from macos (3.11) to macos ({macos-latest,macos-13}, 3.11)) and is now ready for review.

@rflamary rflamary merged commit 628a089 into PythonOT:master Jun 21, 2024
@matthewfeickert matthewfeickert deleted the ci/add-macos-13 branch June 21, 2024 11:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

2 participants