-
Notifications
You must be signed in to change notification settings - Fork 44
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
Mocha and babel compiler warnings #144
Comments
Seems like |
Culprit: PR #135 The The |
This fixes 2 issues, both of which are vaguely related to babel transpiling during tests: * The new mocha version uses the `--require` flag instead of `--compilers`. * nyc 12.0.0 causes mysterious "Transformation errors" with transpiling during tests. This PR addresses the first issue by switching the flag as required. This PR addresses the second issue by downgrading nyc, as it's not clear if nyc or our project is the real culprit. Fixes #144
The answer to the nyc issue is in the changelog. nyc@11 is the last release with node v4/v5 support. |
This fixes 2 issues, both of which are vaguely related to babel transpiling during tests: * The new mocha version uses the `--require` flag instead of `--compilers`. * nyc 12.0.0 causes mysterious "Transformation errors" with transpiling during tests. This PR addresses the first issue by switching the flag as required. This PR addresses the second issue by downgrading nyc, as v11 is the last release which supports node v4. Fixes #144
I see the following output when I run
npm test
on node v4.8.7 and npm 5.8.0:We should find a way forward without the
--compilers
flag. The deprecation notice says this is only a warning, so we may have a different problem related to the "Transformation errors".This blocks #142.
The text was updated successfully, but these errors were encountered: