-
Notifications
You must be signed in to change notification settings - Fork 106
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
Carthage & Swift Package Manager support #33
Comments
Hmm this is actually going to be more complicated than I realized. Because this was probably set up using the cocoapods gem, I think that carthage won't work because I'll have to put up a larger PR or a series of PRs but there's a couple of things I want to change here:
So really it's a lot of things but I'll probably do things in that order when I get a free afternoon. |
I see two options:
I'm no CocoaPods wiz, so I don't know if that will cause more trouble than good. |
Yeah that's a better idea actually, great suggestions 👍 This repo desperately needs tests as well. No open-source library should be distributed without tests. |
@startupthekid 😢you know I'm not good at tests! |
Someday young padawon, someday you will be. |
@startupthekid do you think you'll get a chance to work on this? No worries if not! |
Title says it all, I'll put a PR up
The text was updated successfully, but these errors were encountered: