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

chore: bump accounts dependencies #5177

Merged
merged 4 commits into from
Jan 22, 2025
Merged

chore: bump accounts dependencies #5177

merged 4 commits into from
Jan 22, 2025

Conversation

ccharly
Copy link
Contributor

@ccharly ccharly commented Jan 21, 2025

Explanation

Bump accounts-related dependencies.

References

N/A

Changelog

@metamask/accounts-controller

  • CHANGED: Bump @metamask/keyring-api from ^13.0.0 to ^14.0.0
  • CHANGED: Bump @metamask/keyring-internal-api from ^2.0.0 to ^2.0.1
  • CHANGED: Bump @metamask/eth-snap-keyring from ^8.1.0 to ^8.1.1

@metamask/assets-controller

  • CHANGED: Bump @metamask/keyring-api from ^13.0.0 to ^14.0.0
  • CHANGED: Bump @metamask/keyring-internal-api from ^2.0.0 to ^2.0.1
  • CHANGED: Bump @metamask/keyring-snap-client from ^2.0.0 to ^3.0.0

@metamask/accounts-controller

  • CHANGED: Bump @metamask/keyring-api from ^13.0.0 to ^14.0.0
  • CHANGED: Bump @metamask/keyring-internal-api from ^2.0.0 to ^2.0.1

@metamask/accounts-controller

  • CHANGED: Bump @metamask/keyring-api from ^13.0.0 to ^14.0.0
  • CHANGED: Bump @metamask/keyring-internal-api from ^2.0.0 to ^2.0.1

Checklist

  • I've updated the test suite for new or updated code as appropriate
  • I've updated documentation (JSDoc, Markdown, etc.) for new or updated code as appropriate
  • I've highlighted breaking changes using the "BREAKING" category above as appropriate
  • I've prepared draft pull requests for clients and consumer packages to resolve any breaking changes

@ccharly ccharly marked this pull request as ready for review January 22, 2025 08:55
@ccharly ccharly requested review from a team as code owners January 22, 2025 08:55
mathieuartu
mathieuartu previously approved these changes Jan 22, 2025
Copy link
Contributor

@mathieuartu mathieuartu left a comment

Choose a reason for hiding this comment

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

Owned files LGTM!

sahar-fehri
sahar-fehri previously approved these changes Jan 22, 2025
@ccharly ccharly dismissed stale reviews from sahar-fehri and mathieuartu via 5e1fd1b January 22, 2025 09:10
@sahar-fehri
Copy link
Contributor

LGTM, can re-approve once green 🙏

Copy link

Removed dependencies detected. Learn more about Socket for GitHub ↗︎

🚮 Removed packages: npm/@metamask/[email protected]

View full report↗︎

@ccharly ccharly merged commit 7c8c892 into main Jan 22, 2025
123 checks passed
@ccharly ccharly deleted the chore/update-accounts-deps branch January 22, 2025 09:31
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