-
-
Notifications
You must be signed in to change notification settings - Fork 197
Verify CocoaPods #1385
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
Verify CocoaPods #1385
Conversation
5488199
to
b33c9da
Compare
hm, why should we create new project, add platform and verify |
Well, CocoaPods need a Xcode project to work on and that was the easiest way I could think of creating one. |
We can print dots while waiting for pod install (we have a method in helpers that does this :) ) |
@rosen-vladimirov, without existing {N} project, CocoaPods complains with |
Can we mock it - for example we can add one default proj file in our resources, copy it to temp folder, create Podfile there and call |
54498b2
to
b7691b2
Compare
let temp = require("temp"); | ||
temp.track(); | ||
let projDir = temp.mkdirSync("nativescript-check-cocoapods"); | ||
this.$npm.install("tns-ios", projDir).wait(); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
maybe we should try-catch this one as well. As we know - npm fails sometimes and in this case our doctor command will fail
b7691b2
to
5cc9485
Compare
5cc9485
to
96112ef
Compare
👍 |
See #1294
How it works: