Skip to content

Release/417.0.0 - #5888

Merged
ccharly merged 6 commits into
mainfrom
release/417.0.0
May 30, 2025
Merged

Release/417.0.0#5888
ccharly merged 6 commits into
mainfrom
release/417.0.0

Conversation

@ccharly

Copy link
Copy Markdown
Contributor

Release to mostly aligns latest accounts and snaps packages.

The accounts change are required because of a major bump of the eth-snap-keyring package for the KeyringRequest.origin feature.

Comment on lines -25 to -26
- **BREAKING:** Bump `@metamask/providers` peer dependency from `^21.0.0` to `^22.0.0` ([#5871](https://github.com/MetaMask/core/pull/5871))
- **BREAKING:** Bump `@metamask/snaps-controllers` peer dependency from `^11.0.0` to `^12.0.0` ([#5871](https://github.com/MetaMask/core/pull/5871))

Copy link
Copy Markdown
ContributorAuthor

Choose a reason for hiding this comment

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

This was a mistake from my initial PR, I updated the wrong section 🙈


### Changed

- Bump `@metamask/keyring-api` peer dependency from `^17.4.0` to `^18.0.0` ([#5871](https://github.com/MetaMask/core/pull/5871))

@ccharlyccharlyMay 30, 2025

Copy link
Copy Markdown
ContributorAuthor

Choose a reason for hiding this comment

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

Also a mistake from my other PR, this is not a peer dep.

Comment on lines +15 to +16
- **BREAKING:** bump `@metamask/snaps-controllers` peer dependency to `^12.0.0` ([#5871](https://github.com/MetaMask/core/pull/5871))
- **BREAKING:** bump `@metamask/providers` peer dependency to `^22.0.0` ([#5871](https://github.com/MetaMask/core/pull/5871))

@ccharlyccharlyMay 30, 2025

Copy link
Copy Markdown
ContributorAuthor

Choose a reason for hiding this comment

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

Those were also missing on my previous PR.

@ccharly
ccharly marked this pull request as ready for review May 30, 2025 15:47
@ccharly
ccharly requested review from a team as code ownersMay 30, 2025 15:47

@cryptodev-2scryptodev-2s left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM!

@ccharly
ccharly merged commit 7ff8785 into mainMay 30, 2025
@ccharly
ccharly deleted the release/417.0.0 branch May 30, 2025 16:39
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@ccharly@cryptodev-2s