-
Notifications
You must be signed in to change notification settings - Fork 247
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
chore(actions): bump min swift version to 5.9 and updated CI jobs #4916
Conversation
c22329b
to
3463248
Compare
@@ -16,8 +16,8 @@ | |||
"dart.onlyAnalyzeProjectsWithOpenFiles": true, | |||
"files.insertFinalNewline": true, | |||
"editor.codeActionsOnSave": { | |||
"source.fixAll": true, | |||
"source.organizeImports": true | |||
"source.fixAll": "explicit", |
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.
Not sure if this was intentional or not. I think it is fine to leave this though. It looks like VS code changed this prop from a boolean to an enum and your editor likely migrated it automatically
https://code.visualstudio.com/updates/v1_85#_code-actions-on-save-and-auto
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.
Yeah, I went to back it out and it looked like it got auto migrated so I left it.
nit: Can you please update the PR change description to include upgrading the Android compileSdk to version 34. |
Description of changes:
latest
tag for iOS CI jobsmacOS-12
runner, which raises the min iOS simulator version we can test on to 15.0. In the meantime, we need to be onmacOS-14
to avoid breaking CI jobs when they finish their migrationBy submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.