Skip to content

fix: preview fallback path #436

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 4 commits into from
Nov 18, 2019
Merged

fix: preview fallback path #436

merged 4 commits into from
Nov 18, 2019

Conversation

endarova
Copy link
Contributor

@endarova endarova commented Nov 15, 2019

When preview_folder_path is not passed as env. variable the fallback is only to /tns-dist but this is not working on windows.
Fix: set preview falback path according to the os
This will fix failing pr in nativescript-dev-webpack wneh they try to run preview tests on win

@cla-bot cla-bot bot added the cla: yes label Nov 15, 2019
@endarova endarova merged commit 681574e into master Nov 18, 2019
endarova added a commit that referenced this pull request Nov 20, 2019
* fix: docker on win (#434)

* fix: cases when iOS version is 3 digits (#437)

* fix: cases when Xcode version is 3 digits

* chore: remove unused noinspection

* fix: preview fallback path (#436)

* set preview falback path according to os

* fix win path

* fix lint errors

* fix: livesync failures (#438)

* update master detail changes

* update changes in hello world ng template

* fix process name

* chore: remove doctor templates (#439)

- Remove doctor templates.
- Add Tab Navigation Vue template.

* chore: change the way we assert changes to improve tests stability

* fix: flake8 error

* fix: win path in package.json (#443)

* fix path on win

* fix paths on windows

* update variable name

* use fail_safe option
@endarova endarova deleted the endarova/fix_preview_path branch November 21, 2019 11:16
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.

2 participants