Skip to content
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

build(ganache-cli): bump version to v6.12.0 to fix node 14 errors #14

Closed
wants to merge 1 commit into from

Conversation

ashutoshvarma
Copy link

version 6.12.0 includes fixes for Node 14 - trufflesuite/ganache#732 (comment)

In node 14, while trying to test deployment script on fork using yarn ganache-fork wss://kovan.infura.io/ws/v3/<PROJECT_ID>
kept getting Error: Callback was already called.

version 6.12.0 includes fixes for Node 14

Signed-off-by: Ashutosh Varma <ashutoshvarma11@live.com>
@fulminmaxi fulminmaxi requested review from a team, daywiss, smb2796, mrice32 and pemulis and removed request for a team, smb2796 and daywiss May 4, 2021 12:51
@mrice32
Copy link
Member

mrice32 commented May 4, 2021

@ashutoshvarma The reason we can't upgrade to a node 14 compatible version is trufflesuite/ganache#797. Please let us know if you are able to actually deploy an EMP on a fork with your PR. Last time I tried with 6.12.0+, I got a false re-entrancy error.

@pemulis
Copy link
Contributor

pemulis commented Jun 29, 2021

@ashutoshvarma @mrice32 I'm going to close this one for now! We can re-open if there is a later upgrade to Ganache that solves the node v14 error.

@pemulis pemulis closed this Jun 29, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants