Skip to content

Merge 1.5.1 in master #1304

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 20 commits into from
Dec 3, 2015
Merged

Merge 1.5.1 in master #1304

merged 20 commits into from
Dec 3, 2015

Conversation

rosen-vladimirov
Copy link
Contributor

Merge 1.5.1 in master

rosen-vladimirov and others added 20 commits December 1, 2015 09:12
When any of the hooks fail, we show help of the command.
Hooks failure is not bad user input, so we should not show help.
Set version to 1.5.1
Update npm version to latest 2.x as the old one (2.6.1) has some bugs that prevent us from using git urls, installing some plugins, etc.
…uput-release

Improve XML parser validation output
Add information about 1.5.1 release in CHANGELOG
Add information about 1.5.1 release in CHANGELOG
When ANDROID_HOME environment variable is not set, we call childProcess exec and spawn with undefined command.
This leads to "futures left behind" errors.
Make sure the commands will be executed only in case they are not undefined.
Also make sure that `tns prepare android` will fail immediately in case ANDROID_HOME is not set (currently it shows warning for ANDROID_HOME and shows error for compile SDK).
Silent shelljs for unit-tests and update to latest common lib
@ns-bot
Copy link

ns-bot commented Dec 3, 2015

Test PASSed.

@teobugslayer
Copy link
Contributor

👍

rosen-vladimirov added a commit that referenced this pull request Dec 3, 2015
@rosen-vladimirov rosen-vladimirov merged commit 9e8c34d into master Dec 3, 2015
@rosen-vladimirov rosen-vladimirov deleted the vladimirov/merge-rel-master branch December 3, 2015 14:57
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