Skip to content

chore: merge release in master #4365

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 15 commits into from
Feb 18, 2019
Merged

Conversation

rosen-vladimirov
Copy link
Contributor

chore: merge release in master

rosen-vladimirov and others added 15 commits February 7, 2019 16:44
docs: add missing resources command in index
fix: shouldRestart will now return true if the app is not running instead of throwing an exception
fix: update lodash because of "a potential security vulnerability"
In case there's no ANDROI_HOME on user's machine, CLI tries to use its local copy of adb. However, the current path to adb is not correct and CLI does not work with any Android devices in this case.
Fix the path to local copy of adb.
chore: merge release-patch in release
In case `--bundle` is passed CLI tries to clean the `tns_modules` directory produced from previous prepare operations. However, this is not required in latest versions as webpack cleans the directory on its own.
The current clean was required for Webpack before 0.14.0, particularly the one used with 3.x versions of NativeScript.
Delete unit test that is no longer relevant, as it tests functionality that is now handled by `nativescript-dev-webpack` plugin.
…s_modules

chore: skip tns_modules cleanup from CLI
@KristianDD
Copy link
Contributor

run ci

3 similar comments
@rosen-vladimirov
Copy link
Contributor Author

run ci

@miroslavaivanova
Copy link
Contributor

run ci

@miroslavaivanova
Copy link
Contributor

run ci

@miroslavaivanova miroslavaivanova self-requested a review February 18, 2019 10:45
@rosen-vladimirov rosen-vladimirov merged commit 2abccb5 into master Feb 18, 2019
@rosen-vladimirov rosen-vladimirov deleted the vladimirov/merge-rel-master branch February 18, 2019 10:53
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.

4 participants