Skip to content
This repository has been archived by the owner on Nov 11, 2017. It is now read-only.

Adding WatchOS Target #23

Merged
merged 4 commits into from
Apr 21, 2016
Merged

Adding WatchOS Target #23

merged 4 commits into from
Apr 21, 2016

Conversation

tmspzz
Copy link
Contributor

@tmspzz tmspzz commented Apr 21, 2016

  • Added Target for watch OS

@tmspzz tmspzz mentioned this pull request Apr 21, 2016
@CodaFi
Copy link
Member

CodaFi commented Apr 21, 2016

A command that builds this new target should be added to the build script as well.

@tmspzz
Copy link
Contributor Author

tmspzz commented Apr 21, 2016

@CodaFi where should I add such command? Do you mean .travis.yml ? I'm not familiar with Travis

@mpurland
Copy link
Member

Yes, .travis.yml. It should just be another line for the watch target. Let me know if you need help.

@tmspzz
Copy link
Contributor Author

tmspzz commented Apr 21, 2016

Yes I think I need help because I'm a bit confused.

All I see are xcodebuild commands that specify test ... but for watchOS there is no possiblity to add a unit test target.

Should the command be build instead? Maybe like so?

xcodebuild build -scheme Tyro-watchOS -destination 'platform=watchOS Simulator,name=Apple Watch - 42mm' | xcpretty -c

@tmspzz
Copy link
Contributor Author

tmspzz commented Apr 21, 2016

I guess that worked :)

@CodaFi
Copy link
Member

CodaFi commented Apr 21, 2016

Perfect, thanks.

@mpurland
Copy link
Member

Great, almost there! Only a few more things. A test target needs to be added for watchOS, existing tests need to be added to the target, and the tests need to be run with the Travis build through the watchOS framework scheme.

@CodaFi
Copy link
Member

CodaFi commented Apr 21, 2016

@mpurland That would be all well and good, except Xcode doesn't allow for WatchOS testing targets (at least, not that I'm aware of). That's why not even SwiftCheck supports it yet.

@mpurland
Copy link
Member

@CodaFi Bummer! I think I remember that now.

In that case, for now, the testing for iOS should suffice for testing purposes then.

@mpurland
Copy link
Member

Looks good. Thanks! @blender

👍 ⛵

@mpurland mpurland merged commit bbfdb0f into typelift:master Apr 21, 2016
@tmspzz tmspzz deleted the feature/watchOS branch April 25, 2016 09:08
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants