-
Notifications
You must be signed in to change notification settings - Fork 485
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
Error when run from CI #668
Comments
Thanks for the report! I'm not 100% sure what's going on here yet, but could you let me know which node version you're using? CircleCI's default, v0.10, is very old and is likely to cause problems with this and other libraries - eslint, for instance, doesn't even run with the CircleCI default. If it's the default, maybe trying upgrading to v4? |
Whoops, I should have mentioned that we're enforcing node v6 on Circle. |
@tmcw Any thoughts on this? I gave it a shot again but ran into the same issue. |
@bright-star it looks like you're running into this Node core bug that was introduced in 6.2.1 and fixed in 6.3.0. Try upgrading the version of Node on CircleCI to something 6.3.0 or newer? |
This guards against potential core errors like the one possibly found in #668 and makes sure documentation.js continues to be UTF8 friendly
Thanks for the update. We'll try that this week. |
That worked out great, thanks! |
Specifically, CircleCI:
The same setup works when run locally.
The text was updated successfully, but these errors were encountered: