Add support for Swift Package Manager #55
Conversation
nikitskynikita
force-pushed
the
swift-package-manager
branch
from
bc09ba1
to
7d445a6
Compare
| @@ -5,6 +5,7 @@ | |||
| // Copyright © 2019 Tiny Speck, Inc. All rights reserved. | |||
| // | |||
|
|
|||
| #if os(iOS) | |||
There was a problem hiding this comment.
Heya! Thanks for working on this. QQ, Is is needed in all classes to limit it to iOS only? Not able to define in SPM that it's iOS only?
I'm not familiar with SPM yet, thanks!
There was a problem hiding this comment.
Hello.
No, #if os(iOS) needed in only classes using specific iOS library such as UIKit. SPM allows to define target platform only for binary, but it doesn't include this property in build source. I found the article with solution for this problem, but I think that isn't good idea.
There was a problem hiding this comment.
Since we now have Catalyst, this should be:
#if canImport(UIKit). I'm a little late! :(
nikitskynikita
force-pushed
the
swift-package-manager
branch
from
7d445a6
to
959916a
Compare
|
Any update on this? |
|
cc @nikitskynikita ❤️ |
|
Just merged this change + updated for Swift 5.0. |
|
@nikitskynikita Thanks so much for your PR. That was a great help & apologies for the delay in merging. |
Summary
IOS developers have a need to install a pan modal through SPM. Added the ability to connect a pan modal through SPM.
Requirements (place an
xin each[ ])