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(ios)!: set minimum xcode to 11.x #12022

Closed
wants to merge 1 commit into from

Conversation

sgtcoolguy
Copy link
Contributor

JIRA: https://jira.appcelerator.org/browse/TIMOB-28072

Optional Description:

Sets Xcode 11 as minimum

@sgtcoolguy sgtcoolguy added the ios label Sep 8, 2020
@build build added this to the 9.2.0 milestone Sep 8, 2020
@build
Copy link
Contributor

build commented Sep 8, 2020

Messages
📖

💾 Here's the generated SDK zipfile.

📖 ✊ The commits in this PR match our conventions! Feel free to Rebase and Merge this PR when ready.
📖

✅ All tests are passing
Nice one! All 7433 tests are passing.
(There are 711 skipped tests not included in that total)

Generated by 🚫 dangerJS against 2835e97

@sgtcoolguy
Copy link
Contributor Author

cherry-picked from #11955

@@ -19,7 +19,7 @@
"minIosVersion": "10.0",
"minWatchosVersion": "2.0",
"vendorDependencies": {
"xcode": ">=9.0 <=12.x",
"xcode": ">=11.0 <=12.x",
"ios sdk": ">=10.0 <=14.x"
Copy link
Contributor

Choose a reason for hiding this comment

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

  1. With Xcode 11, iOAS SDK is 13. So update-
Suggested change
"ios sdk": ">=10.0 <=14.x"
"ios sdk": ">=13.0 <=14.x"
  1. As we are targeting minimus iOS SDK 13.0, we need to remove all macros IS_SDK_IOS_12 and IS_SDK_IOS_13. Because these will be irrelevant.

@sgtcoolguy
Copy link
Contributor Author

Closing as this is part of #12026

@sgtcoolguy sgtcoolguy closed this Sep 10, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants