-
Notifications
You must be signed in to change notification settings - Fork 78
Conversation
Thanks for this PR! It's been a while, but I'm finally working om merging this. I'm getting a build error about watchOS not knowing UITraitCollection, when I try to build the Rswift-watchOS target in this library. I'm guessing those files shouldn't be part of the Watch target at all? |
That's correct. I guess there is some new code added using
|
Hi! How it looks? I started adding watch support because it would be good for me and I'm looking and here is this PR already :). |
@lammertw I'm not seeing that |
Right, I just noticed it as well. It seems I added this code once locally at a later time after merging in a new version but never put that into this pull request. I've pushed it now. |
Any updates on this? Would be great if this could be merged in :) |
Merging with mac-cain13/R.swift#441 |
This basically disabled all functions for watchOS that are not supported by the watchOS sdk by wrapping them in preprocessor statements.
For example: