-
Notifications
You must be signed in to change notification settings - Fork 79
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
Support node v12 #461
Comments
@0xGabi We've been able to upgrade ganache-core; could we retest this to see if node 12 now works (and run CI on node 12 as well)? |
I just tested it and the install worked 🥳 anyways I got some build errors with node-gyp. I'll test it a bit more and also try to install in a linux vm before close this one. Install is going to be much better once we separate ganache-core to his own package |
Looks like those build errors are for the native modules, and keccak and secp256k1 specifically. They have fallback JS implementations so it's not going to break, but the situation is not ideal. Let's add node12 to the CI but keep advertising that users use node 10. |
Check with a Linux vm and it worked as well, I'll close this issue |
This issue was funded with 60 DAI, see more issues on our dao projects app.
See the travis builds:
https://travis-ci.org/aragon/aragon-cli/builds/532301139?utm_source=github_status&utm_medium=notification
Specifically this error:
We could try after updating
ganache-core
.Related: trufflesuite/ganache#425
The text was updated successfully, but these errors were encountered: