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

Supporting iOS8 frameworks and Carthage package manager #155

Open
wants to merge 16 commits into
base: develop
Choose a base branch
from

Conversation

dodikk
Copy link

@dodikk dodikk commented Feb 9, 2015

  • Using flat layout for public headers
  • Fixed import syntax for public headers #import "" ===> #import <>
  • Added target and shared scheme for compatibility with "Carthage" depencency manager
  • Using more strict warnings
  • Removed a dependency on UIKit/AppKit framework

@toto
Copy link
Contributor

toto commented Feb 9, 2015

Wow, thanks for doing this!

I will look this over as soon as possible.

@dodikk
Copy link
Author

dodikk commented Feb 11, 2015

Looking forward to hearing from you.

@dodikk
Copy link
Author

dodikk commented Feb 11, 2015

One more thing...

I have not fixed the podspec since I'm not good at cocoapods. So, it's kind of broken for this patch.
Hopefully, maintaining this won't be a problem.

@toto
Copy link
Contributor

toto commented Feb 11, 2015

Thanks for the heads up. Should not be a big deal.

@PaulTaykalo
Copy link

@toto Is it going to be merged in near future? :)
I'm trying to understand, should I have reference to own rep + this patch, or will it be merged in the main rep in near future?

@dimazen
Copy link

dimazen commented Apr 1, 2015

Upvote

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.

4 participants