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

chore(actions): bump min swift version to 5.9 and updated CI jobs #4916

Merged
merged 22 commits into from
May 28, 2024

Conversation

Equartey
Copy link
Contributor

@Equartey Equartey commented May 22, 2024

Description of changes:

  • Updates Canary tests to Amplify Flutter V2
  • Migrated Canary Android app to newer Plugin DSL syntax
  • Added a latest tag for iOS CI jobs
  • Bumped min swift_version to 5.9 in preparation for Amplify Swift V2
  • Github actions is deprecating their macOS-12 runner, which raises the min iOS simulator version we can test on to 15.0. In the meantime, we need to be on macOS-14 to avoid breaking CI jobs when they finish their migration
  • Bumped Android compileSdk to version 34
  • Resolved Android build errors for DataStore category

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@Equartey Equartey requested a review from a team as a code owner May 22, 2024 18:51
@@ -16,8 +16,8 @@
"dart.onlyAnalyzeProjectsWithOpenFiles": true,
"files.insertFinalNewline": true,
"editor.codeActionsOnSave": {
"source.fixAll": true,
"source.organizeImports": true
"source.fixAll": "explicit",
Copy link
Contributor

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

Copy link
Contributor Author

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.

@NikaHsn
Copy link
Contributor

NikaHsn commented May 28, 2024

nit: Can you please update the PR change description to include upgrading the Android compileSdk to version 34.

@Equartey Equartey merged commit 6944287 into main May 28, 2024
426 of 427 checks passed
@Equartey Equartey deleted the chore/bump-swift-5.9 branch May 28, 2024 18:04
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.

None yet

3 participants