-
Notifications
You must be signed in to change notification settings - Fork 126
Change macos runners to macos-12. (requires updating xcode to 13.1) #917
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
Conversation
✅ Integration test succeeded!Requested by @jonsimantov on commit 25da83c |
|
FYI The Xcode upgrade could experience build errors if compiling against the iOS SDK that does not have an Xcode 13.3.1 fix (firebase/firebase-ios-sdk#9705). |
Thanks. I'll be trying this in the 9.0.0 branch as well. |
|
Updated the macos runner name with a matrix variable. |
Nice, thanks! I just kicked off a new tests-requested on this PR so you can cancel your separate test run if you want. |
Description
Update macos runners to use macos-12 (public beta) instead of macos-latest (equivalent to macos-11).
This change requires updating to Xcode 13.1 as a stopgap solution; we will update to Xcode 13.2.1 or Xcode 13.3.1 in a subsequently-merged PR. Updating to Xcode 13 necessitates updating iOS simulator versions as well.
Note: skipping release notes check, because the subsequent PR (#915) updates the release notes with the new Xcode version requirement.
Testing
Integration test run in the PR.
C++ packaging step here: https://github.com/firebase/firebase-cpp-sdk/actions/runs/2260453371
Type of Change
Place an
xthe applicable box:Notes
Release Notessection ofrelease_build_files/readme.md.