-
Notifications
You must be signed in to change notification settings - Fork 375
WIP) Upgrade deps to latest ( prettier, ethers ) #912
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
base: master
Are you sure you want to change the base?
Conversation
|
Warning Review the following alerts detected in dependencies. According to your organization's Security Policy, it is recommended to resolve "Warn" alerts. Learn more about Socket for GitHub.
|
fixes #911
Note that this PR should be replaced with commits to maintain ethers-v5 or starknet.js properly,
But as we don't have resources to fix including those atm would leave as is until requested explicitly.
This PR removes unnecessary deps and services params to rely on nodejs's promise API which is available since 18.x
To try out I would recommend using the following