-
Notifications
You must be signed in to change notification settings - Fork 2.7k
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
Yarn 0.23.2 uses Array.include which is not present in node 4 #3138
Comments
I face the same problem, my node version is v5.10.1 |
Also having this issue with nodejs 4.8.2 and yarn 0.23.2. |
Seeing the same, here is a Travis build with node 4: https://travis-ci.org/kaliber5/ember-bootstrap/jobs/221796901#L225 |
You can have a look at #3139 that fixes the problem |
It seems originally release with this error happened because this specific line doesn't have any tests, since otherwise CI would pick it up? I think it would be a good idea not only to fix it but add tests for it as well |
Do you want to request a feature or report a bug?
What is the current behavior?
yarn tries to use Array.include which is not present in node 4.
If the current behavior is a bug, please provide the steps to reproduce.
What is the expected behavior?
It should install the dependencies and avoid depending on language features/apis not available on node 4.
Please mention your node.js, yarn and operating system version.
Full log
The text was updated successfully, but these errors were encountered: