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

Enabled APPLICATION_EXTENSION_API_ONLY in tvOS framework target. #771

Merged
merged 5 commits into from
Sep 15, 2015

Conversation

Baza207
Copy link
Contributor

@Baza207 Baza207 commented Sep 15, 2015

Set APPLICATION_EXTENSION_API_ONLY to YES in tvOS Framework target.

Baza207 and others added 5 commits September 13, 2015 02:54
@cnoon
Copy link
Member

cnoon commented Sep 15, 2015

Thanks @Baza207! 🍻

cnoon added a commit that referenced this pull request Sep 15, 2015
Enabled APPLICATION_EXTENSION_API_ONLY in tvOS framework target.
@cnoon cnoon merged commit 4ba997e into Alamofire:tvOS Sep 15, 2015
@cnoon cnoon added this to the 2.1.0 milestone Sep 15, 2015
@cnoon cnoon self-assigned this Sep 15, 2015
@cnoon
Copy link
Member

cnoon commented Sep 15, 2015

I ended up resetting the merge and just cherry picking your change in 2d8466c. In the future, if you could rebase before you push up your PR, that would be greatly appreciated.

Thanks again!

@Baza207
Copy link
Contributor Author

Baza207 commented Sep 15, 2015

Ops sorry. I thought I'd done that properly. I'll rebase from now on so I don't cause any more issues.

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

Successfully merging this pull request may close these issues.

3 participants