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

Build macos wheel on macos-latest github runner #212

Merged
merged 2 commits into from
Sep 29, 2022

Conversation

nhuet
Copy link
Contributor

@nhuet nhuet commented Sep 26, 2022

Macos-10.15 github runner is deprecated and will be removed starting December (cf actions/runner-images#5583). So we need to move the build on macos-latest (= macos-11 currently) and see if tests still pass on macos-10.15.

The other github runner names are also change for their "$os-latest" aliases when possible to follow more easily the update of github runners.

Copy link
Contributor

@galleon galleon left a comment

Choose a reason for hiding this comment

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

Ok for builds done on latest but I think tests should still using exact os names macos-10.15, macos-11 and macos-12

@nhuet nhuet marked this pull request as draft September 27, 2022 13:18
Macos-10.15 github runner is deprecated and will be removed starting
December (cf actions/runner-images#5583).
So we need to move the build on macos-latest (= macos-11 currently).

We ensure that the resulting wheel is still compatible with macos-10.15.
@nhuet nhuet force-pushed the nht/build_macos_latest branch from 2d029ac to 1239338 Compare September 27, 2022 14:57
@nhuet
Copy link
Contributor Author

nhuet commented Sep 27, 2022

I had to work a bit so that the wheel still works on macos 10.15 (it needs at least to have the proper name). I rebase with the solution i found and let the tests as before (not mentioning **-latest in tests oses)

@nhuet
Copy link
Contributor Author

nhuet commented Sep 27, 2022

I also added macos-12 in tests.

@nhuet nhuet marked this pull request as ready for review September 27, 2022 15:11
@nhuet
Copy link
Contributor Author

nhuet commented Sep 27, 2022

Nice! The workflow passed this time! And with the requested changes.

@galleon galleon merged commit f75a88b into airbus:master Sep 29, 2022
@nhuet nhuet deleted the nht/build_macos_latest branch October 6, 2022 12:18
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