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

[7.x] Set APPLICATION_EXTENSION_API_ONLY to YES #525

Merged
merged 1 commit into from
May 15, 2018

Conversation

ikesyo
Copy link
Member

@ikesyo ikesyo commented May 8, 2018

This enables us to use Swift internal classes from Objective-C side with importing the generated header.

Fixes #477.

This enables us to use Swift internal classes from Objective-C side with importing the generated header.
@ikesyo ikesyo force-pushed the APPLICATION_EXTENSION_API_ONLY branch from 6d189ed to ed94b22 Compare May 8, 2018 13:40
@wongzigii
Copy link
Member

Should we also turn on APPLICATION_EXTENSION_API_ONLY in Quick ?

@ikesyo
Copy link
Member Author

ikesyo commented May 11, 2018

Might be yes: Quick/Quick#749

@wongzigii
Copy link
Member

:shipit:

@wongzigii wongzigii merged commit a076568 into 7.x-branch May 15, 2018
@wongzigii wongzigii deleted the APPLICATION_EXTENSION_API_ONLY branch May 15, 2018 14:58
@ghost
Copy link

ghost commented Jun 11, 2018

This is causing a warning in my project:
image

Any fixes for that?

@axandreo
Copy link

I am getting the same warning as @afetrifork . Additionally, I can't tell if they're related but, I started getting this issue

Analyzing dependencies
[!] Unable to satisfy the following requirements:

- `Nimble` required by `Podfile`
- `Nimble` required by `Podfile`
- `Nimble (= 7.1.2)` required by `Podfile.lock`

None of your spec sources contain a spec satisfying the dependencies: `Nimble, Nimble (= 7.1.2)`.

The error instructs me to do pod repo update. However, invoking pod repo update doesn't solve this error.

Megal pushed a commit to Megal/Nimble that referenced this pull request Jul 31, 2019
[7.x] Set `APPLICATION_EXTENSION_API_ONLY` to `YES`
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.

3 participants