-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
Fix the podlint error #1428
Fix the podlint error #1428
Conversation
This pull request was exported from Phabricator. Differential Revision: D20036436 |
1 similar comment
This pull request was exported from Phabricator. Differential Revision: D20036436 |
Summary: Pull Request resolved: Instagram#1428 We kept getting an error as the following: ``` - ERROR | [OSX] unknown: Encountered an unknown error (The platform of the target `App` (macOS 10.11) is not compatible with `IGListSwiftKit (4.1.0)`, which does not support `macOS`.) during validation. ``` Looks like IGListSwiftKit did not support macos, wondering why this is the case here. Potentially this error was introduced in this PR: Instagram#1388 cc natestedman who might know the context here Differential Revision: D20036436 fbshipit-source-id: dc77dfae335420fe310ad39a662753af831a1054
2ae9e6a
to
6ac15a5
Compare
Summary: Pull Request resolved: Instagram#1428 We kept getting an error as the following: ``` - ERROR | [OSX] unknown: Encountered an unknown error (The platform of the target `App` (macOS 10.11) is not compatible with `IGListSwiftKit (4.1.0)`, which does not support `macOS`.) during validation. ``` Looks like IGListSwiftKit did not support macos, wondering why this is the case here. Potentially this error was introduced in this PR: Instagram#1388 cc natestedman who might know the context here Reviewed By: natestedman Differential Revision: D20036436 fbshipit-source-id: e9344c847c4be837bc43306a3128ffc87246efd5
6ac15a5
to
11f7ea5
Compare
This pull request was exported from Phabricator. Differential Revision: D20036436 |
1 similar comment
This pull request was exported from Phabricator. Differential Revision: D20036436 |
Summary: Pull Request resolved: Instagram#1428 We kept getting an error as the following: ``` - ERROR | [OSX] unknown: Encountered an unknown error (The platform of the target `App` (macOS 10.11) is not compatible with `IGListSwiftKit (4.1.0)`, which does not support `macOS`.) during validation. ``` Looks like IGListSwiftKit did not support macos, wondering why this is the case here. Potentially this error was introduced in this PR: Instagram#1388 cc natestedman who might know the context here The solution here is to lint the specs separately in travis, so that for IGListSwiftKit we only lint against ios and tvos. And the other podspecs will still lint against on all platform, including the macos. Reviewed By: natestedman Differential Revision: D20036436 fbshipit-source-id: a2567e439f53593639bff949207af87ff42f6507
11f7ea5
to
98dbc50
Compare
Summary: Pull Request resolved: Instagram#1428 We kept getting an error as the following: ``` - ERROR | [OSX] unknown: Encountered an unknown error (The platform of the target `App` (macOS 10.11) is not compatible with `IGListSwiftKit (4.1.0)`, which does not support `macOS`.) during validation. ``` Looks like IGListSwiftKit did not support macos, wondering why this is the case here. Potentially this error was introduced in this PR: Instagram#1388 cc natestedman who might know the context here The solution here is to lint the specs separately in travis, so that for IGListSwiftKit we only lint against ios and tvos. And the other podspecs will still lint against on all platform, including the macos. Reviewed By: natestedman Differential Revision: D20036436 fbshipit-source-id: 67c1578779a20435af0c43428c40a38ad18265b3
98dbc50
to
38fffaa
Compare
This pull request was exported from Phabricator. Differential Revision: D20036436 |
This pull request has been merged in 884b97d. |
Summary:
We kept getting an error as the following:
Looks like IGListSwiftKit did not support macos, wondering why this is the case here.
Potentially this error was introduced in this PR: #1388
cc natestedman who might know the context here
Differential Revision: D20036436