Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Set PATH variable for node, npm and yarn #179

Merged
merged 4 commits into from
Jan 13, 2017
Merged

Conversation

s0x
Copy link
Contributor

@s0x s0x commented Dec 19, 2016

This closes #164, closes #149, closes #146 and closes #134

Thanks to @ChristianRamseier for postinstall integration tests

@s0x s0x force-pushed the fix-path-issues branch 2 times, most recently from efc0e45 to b977309 Compare December 19, 2016 14:04
@s0x s0x changed the title Fix path issues Set PATH variable so that it uses the proper version of node, npm and yarn Dec 19, 2016
@s0x s0x changed the title Set PATH variable so that it uses the proper version of node, npm and yarn Set PATH variable for node, npm and yarn Dec 19, 2016
@srs srs added this to the 1.1.0 milestone Jan 2, 2017
@ChristianRamseier
Copy link

Just pulled this request into my private fork. It works beautifully and solves the issues I had under Windows. Tested both Git Bash (MINGW64) and cmd. Thanks for the fix! Really hope it will make its way into a release very soon!

@srs srs merged commit 1039495 into srs:master Jan 13, 2017
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
3 participants