-
Notifications
You must be signed in to change notification settings - Fork 456
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
Set up CI services #45
Conversation
Looks nice |
- rm -rf ~/.nvm | ||
- git clone --branch v0.33.2 https://github.com/creationix/nvm.git ~/.nvm | ||
- source ~/.nvm/nvm.sh | ||
- nvm install "$TRAVIS_NODE_VERSION" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'd suggest using nvs
instead so that Node-ChakraCore can be included in the matrix. Also maybe the latest Node-V8 nightly build. See https://github.com/jasongin/nvs/blob/master/doc/CI.md
Or I can make that change later.
- export CFLAGS="$CFLAGS -O3 --coverage" LDFLAGS="$LDFLAGS --coverage" | ||
- echo "CFLAGS=\"$CFLAGS\" LDFLAGS=\"$LDFLAGS\"" | ||
install: | ||
- npm install $NPMOPT |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The npm install
is not really needed, since after #44 there are no package dependencies. But I guess we could keep it in case any dependencies are added in the future?
@boingoing You recently encountered that problem with |
I fixed the |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is a good start. I can add Node 8.x and Node-ChakraCore later, using NVS.
I'm going to go ahead and merge this, so I can do some more work on top of it. |
Run for this PR:
Notably, builds are currently failing on Node 4 or 6 on macOS with:
Everything else is green, though.
(Coverage doesn’t seem to be working quite yet but that might just be because tests are failing.)