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

feat: support Xcode 12 #117

Closed
wants to merge 3 commits into from
Closed

Conversation

hansemannn
Copy link
Contributor

@hansemannn hansemannn commented Jun 22, 2020

  • Yes, I dropped watchOS 4 support in Xcode 12 (need to validate if thats actually true)
  • No, I don't assume this will merged in the near future, but I'll keep it here for those requiring it asap

Fixes #117

lib/xcode.js Outdated Show resolved Hide resolved
@@ -144,6 +144,31 @@ const simulatorDevicePairCompatibility = {
'5.x': true, // watchOS 5.x
'6.x': true // watchOS 6.x
}
},
'12.x': { // Xcode 12.x
'10.x': { // iOS 10.x
Copy link
Contributor

Choose a reason for hiding this comment

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

Xcode 12 does not appear to support anything older than iOS 10.3, so this should be '>=10.3 <11'.

'>=3.2': true, // watchOS 3.2
'4.x': true // watchOS 4.x
},
'12.x': { // iOS 12.x
Copy link
Contributor

Choose a reason for hiding this comment

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

Xcode 12 allows you to pair watchOS 7 simulators with iOS 12 simulators, so this needs:

'7.x': true         // watchOS 7.x

'5.x': true, // watchOS 5.x
'6.x': true // watchOS 6.x
},
'13.x': { // iOS 13.x
Copy link
Contributor

Choose a reason for hiding this comment

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

Xcode 12 allows you to pair watchOS 7 simulators with iOS 13 simulators, so this also needs:

'7.x': true         // watchOS 7.x

'5.x': true, // watchOS 5.x
'6.x': true // watchOS 6.x
},
'14.x': { // iOS 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.

Xcode 12 allows you to pair watchOS 4 simulators with iOS 14 simulators, so this needs:

'4.x': true         // watchOS 4.x

@hansemannn
Copy link
Contributor Author

Please take over this PR, I don't understand the feedback here. Thank you!

@hansemannn hansemannn closed this Jun 23, 2020
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

2 participants