Skip to content
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 support #7

Open
wants to merge 11 commits into
base: master
Choose a base branch
from
Open

Conversation

IgorBulyga
Copy link

Added Carthage support.

Also, make some refactoring in the project hierarchy. Move project file in the root folder to produce .framework for Carthage and make Example inner target. Replace Cocoapods with Carthage for Example.

Example folder contains only code for Example
Integrate Carthage frameworks to Example
Make RxAnimated framework scheme shared
Change version number
@icanzilb
Copy link
Member

icanzilb commented Nov 7, 2017

Thanks @IgorBulyga, I'll have a look at the PR. I have to check out the PR locally but if you removed the development setup from the example app I'll have to reject it.

@IgorBulyga
Copy link
Author

IgorBulyga commented Nov 29, 2017

@icanzilb I have merged the latest commit, where AppleTV support was added, into my PR. And provide AppleTV Framework for Carthage.

For check it out for correct work you have to run

carthage update --no-use-binaries --platform iOS,tvOS

And then just open RxAnimated.xcodeproj and run examples. Everything should be fine. If you will have any questions, I'm ready to answer them.

@IgorBulyga IgorBulyga mentioned this pull request Nov 29, 2017
@iandundas
Copy link

This worked great for me in terms of Carthage support, if someone can test it with Coocapods I'd say it's mergeable.

@npjk
Copy link

npjk commented Apr 9, 2018

Hello,

Any update on this merge ? 🙂

@Budyn
Copy link

Budyn commented Jun 21, 2018

Hey guys! Would be great to merge this PR :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants