Skip to content
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

fix: check node version #280

Merged
merged 1 commit into from
Aug 2, 2017
Merged

fix: check node version #280

merged 1 commit into from
Aug 2, 2017

Conversation

reznord
Copy link
Member

@reznord reznord commented Aug 2, 2017

Earlier we were just trying to do node check.js on postinstall script. This won't execute the function which we exported.

So, changed the postinstall script to node -p 'require(\"./check.js\")()'

@reznord reznord requested a review from prateekbh August 2, 2017 06:14
@reznord reznord merged commit b556a03 into master Aug 2, 2017
@prateekbh prateekbh deleted the fix/check-node-version branch August 2, 2017 06:21
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.

2 participants