-
Notifications
You must be signed in to change notification settings - Fork 28
Fix travis #38
Fix travis #38
Conversation
@@ -1,10 +1,14 @@ | |||
language: node_js | |||
|
|||
node_js: | |||
- '4.4.3' | |||
- 'node' |
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.
Shouldn't this suppose to be a 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.
If you use 'node'
then it just uses the latest stable node version. You can check out how this works here.
I updated this because I saw that there was a package-lock.json
file, but that would be ignored by node v4. I can change this to use a fixed version if you prefer.
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.
BTW it seems that there are some issues with nvm on travis, I think that is why this build failed.
You can see that the exact same build went fine for me here so maybe you just need to retrigger this build to get it fixed.
@christian-bromann any chance of merging this? |
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.
👍 thanks!
Thank you! |
I noticed that some PRs are stale because travis is not building properly. I forked the project and, even though the tests were running successfully locally, they were not working correctly on travis.
After a few tests, I found this article that explained how to use browsers in headless mode on travis. I tried with firefox with no success, instead with chrome it seems to be working correctly.