-
-
Notifications
You must be signed in to change notification settings - Fork 525
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
sequelize-cli non-functional @node7.0.0 #383
Comments
getting this too. |
It looks like this is tied to the version of Gulp being used, and Gulp 4.0 seems suited based on version number of the package vinyl-fs. That being said, it's also not a fully released build yet, and so would certainly require testing to ensure it will work as expected in sequelize-cli. |
And the 'npm ls graceful-fs' result is:
Gulp is using graceful-fs < v4. It needs to update before this can be used in node v7.. |
Looks like another reason for #344 |
I'm having pretty much the same issue with node 7.7.1 on windows 10, except that i'm not getting any errors, but the cli does not execute any command, not even
Switching to node 6.7.0 via nvm makes it functional again. I've just test it on ubuntu 16.04 with the same version of node and it seems to be working |
Has anyone found a workaround for this other than switching to node 6.7.0 via nvm? |
I can confirm this is an issue in Node 8 as well. @janmeier do you have any thoughts on how we can fix this? Node 8 is going into LTS soon so this will become a bigger problem. It looks like maybe a Gulp upgrade would fix it? nodejs/node#9355 (comment) |
Same issue in Node 8 as well |
CLI doesn't depend on Gulp any more, so this should be fixed now |
Using sequelize with the latest version of node throws the following error:
This seems to be related to a bad version of graceful-fs, as noted in this npm issue
Needless to say, the expected behavior was that updating the package would allow for it to work inside node7.0.0. It seems that the graceful-fs package being used by sequelize-cli needs to be updated.
The text was updated successfully, but these errors were encountered: