Skip to content

Conversation

@github-actions
Copy link
Contributor

This is the release candidate for version 11.5.0.

@MajorLift MajorLift self-assigned this Aug 26, 2025
@MajorLift MajorLift marked this pull request as ready for review August 26, 2025 11:31
cryptodev-2s
cryptodev-2s previously approved these changes Aug 26, 2025
Copy link
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! Just left one minor suggestion.

Co-authored-by: cryptodev-2s <109512101+cryptodev-2s@users.noreply.github.com>
CHANGELOG.md Outdated
Comment on lines 15 to 18

### Changed

- Add `@types/lodash` as dependency ([#257](https://github.com/MetaMask/utils/pull/257))
Copy link
Contributor

Choose a reason for hiding this comment

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

I believe these may have been left unintentionally

Suggested change
### Changed
- Add `@types/lodash` as dependency ([#257](https://github.com/MetaMask/utils/pull/257))

Copy link
Contributor

@MajorLift MajorLift Aug 26, 2025

Choose a reason for hiding this comment

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

I'm a bit fuzzy on this, but the combined entry should still go under Changed, right?

It is important to remember that consumers care about capabilities first and interfaces second, so these would not count as an addition:

Adding a new dependency ❌

-- https://github.com/MetaMask/core/blob/main/docs/reviewing-release-prs.md#additions

Copy link
Contributor

Choose a reason for hiding this comment

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

Removed duplicate entry here: 1aa30fc

Copy link
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!

@MajorLift MajorLift merged commit 66f825d into main Aug 26, 2025
24 checks passed
@MajorLift MajorLift deleted the release/11.5.0 branch August 26, 2025 12:42
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.

2 participants