-
-
Notifications
You must be signed in to change notification settings - Fork 189
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
bump @metamask/utils to 9.3.0
#4814
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Whoops, looks like there are conflicts to resolve. Note that despite the long list of code owners here, only a review from the wallet framework team is required. Once this has been updated, please tag that team for a re-review and we can get this merged fairly without more conflicts |
fce8d69
Removed dependencies detected. Learn more about Socket for GitHub ↗︎ 🚮 Removed packages: npm/@metamask/utils@9.2.1 |
"@metamask/rpc-errors": "^7.0.0", | ||
"@metamask/utils": "^9.1.0", | ||
"@metamask/json-rpc-engine": "^9.0.3", | ||
"@metamask/rpc-errors": "^6.3.1", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why are these two packages being downgraded here? It could be related to the failing test?
Same for @metamask/rate-limit-controller
and @metamask/transaction-controller
.
Failing lint should be a straightforward fix by ensuring version alignment as the error message tells us.
Closing in favor of #4831 |
Bumping utils to
9.3.0
, pre-cursor to #4809