-
Notifications
You must be signed in to change notification settings - Fork 3.3k
[ci] Migrate Cirrus tests to Apple Silicon #2472
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
https://cirrus-ci.com/task/6657444817403904?logs=local_tests#L2396 @fkorotkov is this an expected difference between the images, and our script needs to handle installing Java? |
osx_instance: | ||
image: big-sur-xcode-13 | ||
image: monterey-xcode-13.3 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't believe 13.4 is available.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
Run
ios-platform_tests
andmacos-platform_tests
on Apple Silicon Macs. Keepios-custom_package_tests
on Intel for now until Java installation flutter/flutter#109821 can be resolved.Upgrade to macOS 12 Monterey and latest available Xcode 13.
Apple Silicon image options at https://github.com/cirruslabs/macos-image-templates/pkgs/container/macos-monterey-xcode
Fixes flutter/flutter#109819
See also plugin version of this at flutter/plugins#5794
Pre-launch Checklist
dart format
.)[shared_preferences]
pubspec.yaml
with an appropriate new version according to the pub versioning philosophy, or this PR is exempt from version changes.CHANGELOG.md
to add a description of the change, following repository CHANGELOG style.///
).If you need help, consider asking for advice on the #hackers-new channel on Discord.