-
Notifications
You must be signed in to change notification settings - Fork 8
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
chore: update dependencies #64
Conversation
Have you tried to bump to latest? Would prefer to go forward then backward 😅 |
As I remember I also set node to Looks like the issue was fixed in Hardaht v2.16.1. In our codex-contracts-eth/package.json Line 21 in b4e4897
|
Seems quite old, we should definitely bump it. Ot does look like it might be the root cause of the failed builds of the multiarch image? |
Oh. sorry about the issue during the Demo :) I've watched it and added some comments. |
And more notes about upgrading node and modules and results in the branch - bumb-hardhat-and-hardhat-deploy
|
@AuHau added a draft PR and we can consider that only after Now, we should probably go wit the changes proposed above. |
|
I am bit confused. Is it with the changes from this PR or from Slava's branch?
Agree, can you create PR and we can close this one in favor of this new PR? |
Probably I will commit my changes in Ben's branch. Or it make sense to create a new PR? |
I don't mind either ;-) |
3e58447
to
972949c
Compare
18.16 doesn't work reliably in the dist-test cloud environment. It works fine when the docker image is used on my local machine though. Very annoying.