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

fix: explicitly add web3-utils dependency #191

Open
wants to merge 2 commits into
base: dev
Choose a base branch
from

Conversation

ThunderDeliverer
Copy link

Description
Testing with Smock failed due to the missing web3-utils dependency.

The dependency is now explicitly added to the package.json file.

Metadata

Testing with Smock failed due to the missing web3-utils dependency.

The dependecy is now explicitly added to the package.json file.
@ThunderDeliverer ThunderDeliverer requested a review from a team as a code owner September 5, 2023 08:05
package.json Outdated
@@ -109,7 +109,8 @@
"ts-node": "10.0.0",
"tsconfig-paths": "^3.9.0",
"typechain": "5.1.1",
"typescript": "4.5.2"
"typescript": "4.5.2",
"web3-utils": "^4.0.5"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

blz explicit version 🙏

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sure 😁

Should be ok now ✅

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

getting this error on gh tests, any clue?

error @noble/[email protected]: The engine "node" is incompatible with this module. Expected version ">= 16". Got "14.21.3"

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Huh, let me try them locally

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm getting no issues on my end (I'm using yarn 1.22.19 and node v16.15.0)

wei3erHase
wei3erHase previously approved these changes Sep 10, 2023
Copy link
Member

@wei3erHase wei3erHase left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm! @0xGorilla wdyt?

@wei3erHase wei3erHase dismissed their stale review September 10, 2023 19:53

Tests not running

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.

Missing dependency web3-utils
2 participants