Skip to content

Fix tns_modules added to app dir #3154

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

Merged
merged 1 commit into from
Oct 13, 2017

Conversation

rosen-vladimirov
Copy link
Contributor

In some cases tns_modules dir may appear in project's app dir. The issue is caused by sending Ctrl + C while ios-sim-portable is executing xcrun simctl command.
In this case, it returns empty string for applicationPath instead of breaking the CLI process. Add new check if the projectRoot on device is not empty string and update ios-sim-portable.

#3016

In some cases `tns_modules` dir may appear in project's app dir. The issue is caused by sending Ctrl + C while `ios-sim-portable` is executing `xcrun simctl` command.
In this case, it returns empty string for applicationPath instead of breaking the CLI process. Add new check if the projectRoot on device is not empty string and update ios-sim-portable.
@rosen-vladimirov
Copy link
Contributor Author

run ci

3 similar comments
@dtopuzov
Copy link
Contributor

run ci

@rosen-vladimirov
Copy link
Contributor Author

run ci

@dtopuzov
Copy link
Contributor

run ci

@rosen-vladimirov rosen-vladimirov merged commit 8179168 into master Oct 13, 2017
@rosen-vladimirov rosen-vladimirov deleted the vladimirov/fix-tns_modules-in-app branch October 13, 2017 13:13
@dtopuzov dtopuzov added this to the 3.3.0 milestone Oct 25, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants