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
Support for Swift Package Manager #106
Conversation
f4fc90b
to
635cc00
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't like the idea that we have to ifdef our code to make it work with Swift package manager. Maybe we should wait for Swift package manager to support platforms before we add support for it?
@@ -47,3 +49,5 @@ open class LaunchRouter<InteractorType, ViewControllerType>: ViewableRouter<Inte | |||
load() | |||
} | |||
} | |||
|
|||
#endif |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Newline please
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done
Yeah, I'm leaving the decision up to you. In other opensource projects, people are usually asking about SPM support. Let's monitor if that's gonna be the case with RIBs and get back to the topic if it is. |
I don't think this is needed. SPM can't even build iOS projects yet. |
Yes, good point. Closing the PR. |
Had to
#if os(iOS)
bunch of UIKit stuff, since SPM doesn't support it yet.Resolves #112