-
Notifications
You must be signed in to change notification settings - Fork 1
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
Expecting Node 9 #3
Comments
Hi @MarcL I'd be really interested to know your thoughts specifically on the tests (i am very far from an expert on testing) and also the docs - e.g. would a "how it works" section be useful (will people care?). All thoughts gratefully received! Cheers |
Ah OK, so the issue i hit first off with node 6 is that ava doesn't like the
Maybe it needs an extra bit of ava config or a plugin or something. I'll try to find out what. |
Ah OK, i think i'm over-complicating it. I think supporting node < 9 means transpiling the tests too (?) - i am assuming Ava doesn't transpile the tests. |
Hmm, i've tried (with a tired brain) a few things on branch |
Clarification: Node < 8 cannot run unit tests but the pre-commit hook works fine (thanks to Babel) on Node 6, 7, 8, 9 at least. |
Ensure to update the readme to remove the note about this if it gets fixed |
Just tried installing with Node 6.14.1 and it expects Node 8 for its post install step for
|
@MarcL - ah dang it! Thanks for the report, i'll see what i can do :-) |
In
package.json
you're setting engines as>=9.0.0
and we're currently running node 6 LTS (6.12.0). We will be switching to Node 8 LTS at some point soon but haven't yet.Does it need Node 9 or can you drop it back and/or transpile?
The text was updated successfully, but these errors were encountered: