Skip to content

Release/401.0.0#5818

Merged
adonesky1 merged 7 commits intomainfrom
release/401.0.0
May 15, 2025
Merged

Release/401.0.0#5818
adonesky1 merged 7 commits intomainfrom
release/401.0.0

Conversation

@adonesky1
Copy link
Copy Markdown
Contributor

@adonesky1 adonesky1 commented May 15, 2025 •
edited
Loading

@metamask/chain-agnostic-permission

[0.7.0]

Changed

  • Bump @metamask/api-specs to ^0.14.0 (#5817)
  • Bump @metamask/network-controller to ^23.5.0 (#5765, #5812)
  • Bump @metamask/controller-utils to ^11.8.0 (#5765, #5812)

@metamask/multichain-api-middleware

[0.3.0]

Changed

  • feat: Add more chain-agnostic-permission utility functions from sip-26 usage (#5609)
  • Bump @metamask/chain-agnostic-permission to ^0.6.0 (#5715,#5760)
  • Bump @metamask/api-specs to ^0.14.0 (#5817)
  • Bump @metamask/controller-utils to ^11.9.0 (#5765, #5812)
  • Bump @metamask/network-controller to ^23.5.0 (#5765, #5812)

@metamask/multichain

[4.1.0]

Changed

  • Bump @metamask/api-specs to ^0.14.0 (#5817)
  • Bump @metamask/controller-utils to ^11.9.0 (#5583, #5765, #5812)

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 communicated my changes to consumers by updating changelogs for packages I've changed, highlighting breaking changes as necessary
  • I've prepared draft pull requests for clients and consumer packages to resolve any breaking changes

@adonesky1 adonesky1 requested review from a team as code owners May 15, 2025 18:44
cryptodev-2s
cryptodev-2s reviewed May 15, 2025
View reviewed changes
packages/eip1193-permission-middleware/CHANGELOG.md Outdated

### Changed

- Bump `@metamask/chain-agnostic-permission` to `^0.7.0` ([#5818](https://github.com/MetaMask/core/pull/5818))
Copy link
Copy Markdown
Contributor

@cryptodev-2s cryptodev-2s May 15, 2025 •
edited
Loading

Choose a reason for hiding this comment

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

Nit: What do you think about combining the three lines for the chain-agnostic permission bumps into a single entry, listing the PRs comma-separated? Also, I noticed a gap in the bump sequence—could it be that some PRs are missing?

Suggested change
- Bump `@metamask/chain-agnostic-permission` to `^0.7.0` ([#5818](https://github.com/MetaMask/core/pull/5818))
- Bump `@metamask/chain-agnostic-permission` to `^0.7.0` ([#5518](https://github.com/MetaMask/core/pull/5518), [#5674](https://github.com/MetaMask/core/pull/5674), [#5818](https://github.com/MetaMask/core/pull/5818))

Copy link
Copy Markdown
Contributor Author

@adonesky1 adonesky1 May 15, 2025

Choose a reason for hiding this comment

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

cleaned up here: e967dd7

@adonesky1
Copy link
Copy Markdown
Contributor Author

adonesky1 commented May 15, 2025

I believe the failing check-changelog process is incorrect...

Changelog check failed for package multichain-api-middleware: This PR contains changes that might require documentation in the changelog. If these changes aren't user-facing, consider adding the 'no-changelog' label instead.

I've made all the relevant changelog entries for multichain-api-middleware

@adonesky1 adonesky1 enabled auto-merge (squash) May 15, 2025 21:26
@adonesky1 adonesky1 requested a review from cryptodev-2s May 15, 2025 21:26
cryptodev-2s
cryptodev-2s reviewed May 15, 2025
View reviewed changes
packages/multichain-api-middleware/CHANGELOG.md Outdated

### Changed

- feat: Add more chain-agnostic-permission utility functions from sip-26 usage ([#5609](https://github.com/MetaMask/core/pull/5609))
Copy link
Copy Markdown
Contributor

@cryptodev-2s cryptodev-2s May 15, 2025

Choose a reason for hiding this comment

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

Nit: can we remove this entry from here now ?

Copy link
Copy Markdown
Contributor Author

@adonesky1 adonesky1 May 15, 2025

Choose a reason for hiding this comment

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

🙏 fixed here: 2abe4bc

cryptodev-2s
cryptodev-2s approved these changes May 15, 2025
View reviewed changes
Copy link
Copy Markdown
Contributor

@cryptodev-2s cryptodev-2s left a comment

Choose a reason for hiding this comment

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

LGTM!

@adonesky1 adonesky1 merged commit e3372e3 into main May 15, 2025
201 of 202 checks passed
@adonesky1 adonesky1 deleted the release/401.0.0 branch May 15, 2025 21:41
@adonesky1 adonesky1 mentioned this pull request Jun 20, 2025
Merged
adonesky1 added a commit that referenced this pull request Jun 20, 2025
@adonesky1
Release/447.0.0 (#6013)
b9d7622 
## @metamask/chain-agnostic-permission

## [1.0.0]

### Changed

- This package is now considered stable

## @metamask/eip1193-permission-middleware

## [1.0.0]

### Changed

- Bump `@metamask/chain-agnostic-permission` to `^0.8.0`
([#5518](#5518),
[#5674](#5674),
[#5818](https://github.com/MetaMask/core/pull/5818)[#5583](https://github.com/MetaMask/core/pull/5583),
[#5982](https://github.com/MetaMask/core/pull/5982),[#6004](https://github.com/MetaMask/core/pull/6004))
- Bump `@metamask/chain-agnostic-permission` to `^0.8.0`
([#5550](#5550),
[#5518](#5518),
[#5674](#5674),
[#5715](#5715),
[#5760](#5760),
[#5818](https://github.com/MetaMask/core/pull/5818)[#5583](https://github.com/MetaMask/core/pull/5583),
[#5982](https://github.com/MetaMask/core/pull/5982),[#6004](https://github.com/MetaMask/core/pull/6004))
- Bump `@metamask/controller-utils` to `^11.10.0`
([#5935](#5935),
[#5583](#5583),
[#5765](#5765),
[#5812](#5812))


## @metamask/multichain-api-middleware

## [1.0.0]

### Changed

- Bump `@metamask/chain-agnostic-permission` to `^0.8.0`
([#5518](#5518),
[#5674](#5674),
[#5818](https://github.com/MetaMask/core/pull/5818)[#5583](https://github.com/MetaMask/core/pull/5583),
[#5982](https://github.com/MetaMask/core/pull/5982),[#6004](https://github.com/MetaMask/core/pull/6004))
- Bump `@metamask/chain-agnostic-permission` to `^0.8.0`
([#5550](#5550),
[#5518](#5518),
[#5674](#5674),
[#5715](#5715),
[#5760](#5760),
[#5818](https://github.com/MetaMask/core/pull/5818)[#5583](https://github.com/MetaMask/core/pull/5583),
[#5982](https://github.com/MetaMask/core/pull/5982),[#6004](https://github.com/MetaMask/core/pull/6004))
- Bump `@metamask/controller-utils` to `^11.10.0`
([#5935](#5935),
[#5583](#5583),
[#5765](#5765),
[#5812](#5812))
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@cryptodev-2s cryptodev-2s cryptodev-2s approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@adonesky1 @cryptodev-2s