Skip to content

Test more node versions in travis #196

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 3 commits into from
Oct 9, 2015

Conversation

Rayman
Copy link
Contributor

@Rayman Rayman commented Sep 10, 2015

I added some more node versions to test for and cleaned the travis script. The current version of node is v4.0.0 and roslib doesn't work on that due to an issue in Automattic/node-canvas#622. We could fix this with #194.

Version 1 and 2 are versions of io.js but apparently roslib works fine on there. Version 3 was the 'alpha' of node v4.

@rctoris
Copy link
Contributor

rctoris commented Sep 11, 2015

Looks like the builds are failing

@Rayman Rayman force-pushed the test-more-node-versions branch from ab532e3 to 302dbdb Compare September 14, 2015 10:06
@Rayman
Copy link
Contributor Author

Rayman commented Sep 14, 2015

I rebased and it looks like travis still fails but I'm unable to reproduce that issue on my machine. During an other build it looked like issue #114 occured again... https://travis-ci.org/Rayman/roslibjs/builds/80210067. I'll continue looking for a solution.

@Rayman Rayman force-pushed the test-more-node-versions branch from 302dbdb to 19502fd Compare September 29, 2015 13:07
@Rayman
Copy link
Contributor Author

Rayman commented Sep 29, 2015

I added the dependencies for chai and mocha and now everything works again.

Rayman added 3 commits October 9, 2015 16:19
Make sure to only update npm in the 0.8 and 0.10 versions to not run
into this issue: nodejs/node#433
@Rayman Rayman force-pushed the test-more-node-versions branch from 5b7b059 to 2cbd3a4 Compare October 9, 2015 14:40
@Rayman
Copy link
Contributor Author

Rayman commented Oct 9, 2015

Rebase to fix a merge conflic in package.json

rctoris added a commit that referenced this pull request Oct 9, 2015
@rctoris rctoris merged commit abca0d2 into RobotWebTools:develop Oct 9, 2015
@Rayman Rayman deleted the test-more-node-versions branch October 9, 2015 20:26
k-aguete pushed a commit to k-aguete/roslibjs that referenced this pull request Oct 21, 2022
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