Apple Watch devices support

[vincentneo]

Hey @Nirma,

Recently needed something like UIDeviceComplete but for the Apple Watch, so I wrote this.

I've added a bunch of code for a separate target in the Xcode project, called WKDeviceComplete.
Currently:

• Unsure how to package for SPM, CocoaPods, etc
• Tests

But otherwise, the code does work. Only System.swift is shared.

Usage wise, it's pretty similar:

WKInterfaceDevice.current().dc.commonDeviceName // Apple Watch Series 7, 45mm case (GPS + Cellular)
WKInterfaceDevice.current().dc.deviceModel // DeviceModel.series7
WKInterfaceDevice.current().dc.screenSize?.caseSize // 45 

I'm unsure if this should belong as a fork of UIDeviceComplete instead (because of SPM, etc). What do you think?

[Nirma]

@vincentneo I am so sorry for replying late! Yes lets add in watchOS support.

I think we can add a watchOS target for both SPM and CocoaPods and use symlinks to smooth out any rough edges.
Let me look into this later today and try something.

For now this looks good, let me update the CI settings.

By the way what are you concerned with on this point?

I'm unsure if this should belong as a fork of UIDeviceComplete instead (because of SPM, etc)

[vincentneo]

By the way what are you concerned with on this point?

Tried adding in another Package.swift (since the watch codes are on separate directories), doesn't seem to work. So my only concern is that it couldn't be packaged correctly that's all!

[vincentneo]

Thinking it through, maybe we can just throw a bunch of #if os(watchOS) everywhere, and SPM should work properly?

[vincentneo]

Thinking it through, maybe we can just throw a bunch of #if os(watchOS) everywhere, and SPM should work properly?

Have implemented this. Code now looks like a mess, but (1) it works fine (tested on simulator), (2) SPM easily cooperates with this.

@Nirma What do you think about this? Wish to hear some thoughts on this, before I write the test cases.

[Nirma]

@vincentneo Yes, it looks fine to me. Lets roll with it.

[vincentneo]

Have written the tests, let me know if I missed out on something, thanks!

[Nirma]

@vincentneo LGTM!