Skip to content

Bump Snaps packages - #4689

Merged
Mrtenz merged 2 commits into
mainfrom
mrtenz/bump-snaps-packages
Sep 12, 2024
Merged

Bump Snaps packages#4689
Mrtenz merged 2 commits into
mainfrom
mrtenz/bump-snaps-packages

Conversation

@Mrtenz

Copy link
Copy Markdown
Member

Explanation

This bumps all Snaps packages used in the core repo to the latest version. This is necessary to unblock #4648.

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

@socket-security

socket-securityBot commented Sep 11, 2024

Copy link
Copy Markdown

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

🚮 Removed packages:npm/@metamask/utils@9.1.0

View full report↗︎

@Mrtenz
Mrtenzforce-pushed the mrtenz/bump-snaps-packages branch from caafa07 to cfbc2beCompareSeptember 12, 2024 14:03
@Mrtenz
Mrtenz marked this pull request as ready for review September 12, 2024 14:16
@Mrtenz
Mrtenz requested a review from a teamSeptember 12, 2024 14:16
@Mrtenz
Mrtenz merged commit 92e6472 into mainSep 12, 2024
@Mrtenz
Mrtenz deleted the mrtenz/bump-snaps-packages branch September 12, 2024 14:21
matthewwalsh0 added a commit that referenced this pull request Apr 30, 2025
## Explanation
Remove validation of `from` if `origin` is internal.
## References
Fixes [#4689](MetaMask/MetaMask-planning#4689)
## Checklist
- [x] I've updated the test suite for new or updated code as appropriate
- [x] I've updated documentation (JSDoc, Markdown, etc.) for new or
updated code as appropriate
- [x] I've communicated my changes to consumers by [updating changelogs
for packages I've
changed](https://github.com/MetaMask/core/tree/main/docs/contributing.md#updating-changelogs),
highlighting breaking changes as necessary
- [x] I've prepared draft pull requests for clients and consumer
packages to resolve any breaking changes
Sign up for freeto 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.

2 participants

@Mrtenz@GuillaumeRx