Skip to content

Conversation

matthewwalsh0
Copy link
Member

@matthewwalsh0 matthewwalsh0 commented Feb 14, 2025

Explanation

Align @ethereumjs/* package versions to avoid any risk of incompatibility and simplify future maintenance.

Upgrade originally required by @metamask/transaction-controller to support EIP-7702 transactions.

References

  • @metamask/accounts PR

Changelog

See changelogs.

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

@matthewwalsh0 matthewwalsh0 marked this pull request as ready for review February 15, 2025 00:47
@matthewwalsh0 matthewwalsh0 requested review from a team as code owners February 15, 2025 00:47
@matthewwalsh0 matthewwalsh0 requested review from a team as code owners February 27, 2025 17:23
@matthewwalsh0 matthewwalsh0 changed the base branch from main to mikesposito/bump-keyrings February 27, 2025 17:25
@matthewwalsh0
Copy link
Member Author

@metamaskbot publish-preview

Copy link
Contributor

Preview builds have been published. See these instructions for more information about preview builds.

Expand for full list of packages and versions.
{
  "@metamask-previews/accounts-controller": "24.1.0-preview-4a9b4a93",
  "@metamask-previews/address-book-controller": "6.0.3-preview-4a9b4a93",
  "@metamask-previews/announcement-controller": "7.0.3-preview-4a9b4a93",
  "@metamask-previews/approval-controller": "7.1.3-preview-4a9b4a93",
  "@metamask-previews/assets-controllers": "51.0.2-preview-4a9b4a93",
  "@metamask-previews/base-controller": "8.0.0-preview-4a9b4a93",
  "@metamask-previews/bridge-controller": "1.0.0-preview-4a9b4a93",
  "@metamask-previews/bridge-status-controller": "1.0.0-preview-4a9b4a93",
  "@metamask-previews/build-utils": "3.0.3-preview-4a9b4a93",
  "@metamask-previews/composable-controller": "11.0.0-preview-4a9b4a93",
  "@metamask-previews/controller-utils": "11.5.0-preview-4a9b4a93",
  "@metamask-previews/earn-controller": "0.5.0-preview-4a9b4a93",
  "@metamask-previews/ens-controller": "15.0.2-preview-4a9b4a93",
  "@metamask-previews/eth-json-rpc-provider": "4.1.8-preview-4a9b4a93",
  "@metamask-previews/gas-fee-controller": "22.0.3-preview-4a9b4a93",
  "@metamask-previews/json-rpc-engine": "10.0.3-preview-4a9b4a93",
  "@metamask-previews/json-rpc-middleware-stream": "8.0.7-preview-4a9b4a93",
  "@metamask-previews/keyring-controller": "19.2.1-preview-4a9b4a93",
  "@metamask-previews/logging-controller": "6.0.4-preview-4a9b4a93",
  "@metamask-previews/message-manager": "12.0.1-preview-4a9b4a93",
  "@metamask-previews/multichain": "3.0.0-preview-4a9b4a93",
  "@metamask-previews/multichain-network-controller": "0.1.2-preview-4a9b4a93",
  "@metamask-previews/multichain-transactions-controller": "0.5.0-preview-4a9b4a93",
  "@metamask-previews/name-controller": "8.0.3-preview-4a9b4a93",
  "@metamask-previews/network-controller": "22.2.1-preview-4a9b4a93",
  "@metamask-previews/notification-services-controller": "1.0.0-preview-4a9b4a93",
  "@metamask-previews/permission-controller": "11.0.6-preview-4a9b4a93",
  "@metamask-previews/permission-log-controller": "3.0.3-preview-4a9b4a93",
  "@metamask-previews/phishing-controller": "12.4.0-preview-4a9b4a93",
  "@metamask-previews/polling-controller": "12.0.3-preview-4a9b4a93",
  "@metamask-previews/preferences-controller": "15.0.2-preview-4a9b4a93",
  "@metamask-previews/profile-sync-controller": "8.1.1-preview-4a9b4a93",
  "@metamask-previews/queued-request-controller": "9.0.1-preview-4a9b4a93",
  "@metamask-previews/rate-limit-controller": "6.0.3-preview-4a9b4a93",
  "@metamask-previews/remote-feature-flag-controller": "1.5.0-preview-4a9b4a93",
  "@metamask-previews/selected-network-controller": "21.0.1-preview-4a9b4a93",
  "@metamask-previews/signature-controller": "23.2.1-preview-4a9b4a93",
  "@metamask-previews/token-search-discovery-controller": "2.1.0-preview-4a9b4a93",
  "@metamask-previews/transaction-controller": "46.0.0-preview-4a9b4a93",
  "@metamask-previews/user-operation-controller": "25.0.0-preview-4a9b4a93"
}

@matthewwalsh0 matthewwalsh0 requested review from a team as code owners March 3, 2025 09:46
@matthewwalsh0 matthewwalsh0 requested a review from a team March 3, 2025 09:46
@matthewwalsh0
Copy link
Member Author

@metamaskbot publish-preview

Copy link

socket-security bot commented Mar 3, 2025

Base automatically changed from mikesposito/bump-keyrings to main March 3, 2025 14:28
@matthewwalsh0
Copy link
Member Author

@metamaskbot publish-preview

Copy link
Contributor

github-actions bot commented Mar 3, 2025

Preview builds have been published. See these instructions for more information about preview builds.

Expand for full list of packages and versions.
{
  "@metamask-previews/accounts-controller": "24.1.0-preview-4b0cb25d",
  "@metamask-previews/address-book-controller": "6.0.3-preview-4b0cb25d",
  "@metamask-previews/announcement-controller": "7.0.3-preview-4b0cb25d",
  "@metamask-previews/approval-controller": "7.1.3-preview-4b0cb25d",
  "@metamask-previews/assets-controllers": "51.0.2-preview-4b0cb25d",
  "@metamask-previews/base-controller": "8.0.0-preview-4b0cb25d",
  "@metamask-previews/bridge-controller": "2.0.0-preview-4b0cb25d",
  "@metamask-previews/bridge-status-controller": "2.0.0-preview-4b0cb25d",
  "@metamask-previews/build-utils": "3.0.3-preview-4b0cb25d",
  "@metamask-previews/composable-controller": "11.0.0-preview-4b0cb25d",
  "@metamask-previews/controller-utils": "11.5.0-preview-4b0cb25d",
  "@metamask-previews/earn-controller": "0.5.0-preview-4b0cb25d",
  "@metamask-previews/ens-controller": "15.0.2-preview-4b0cb25d",
  "@metamask-previews/eth-json-rpc-provider": "4.1.8-preview-4b0cb25d",
  "@metamask-previews/gas-fee-controller": "22.0.3-preview-4b0cb25d",
  "@metamask-previews/json-rpc-engine": "10.0.3-preview-4b0cb25d",
  "@metamask-previews/json-rpc-middleware-stream": "8.0.7-preview-4b0cb25d",
  "@metamask-previews/keyring-controller": "19.2.1-preview-4b0cb25d",
  "@metamask-previews/logging-controller": "6.0.4-preview-4b0cb25d",
  "@metamask-previews/message-manager": "12.0.1-preview-4b0cb25d",
  "@metamask-previews/multichain": "3.0.0-preview-4b0cb25d",
  "@metamask-previews/multichain-network-controller": "0.1.2-preview-4b0cb25d",
  "@metamask-previews/multichain-transactions-controller": "0.5.0-preview-4b0cb25d",
  "@metamask-previews/name-controller": "8.0.3-preview-4b0cb25d",
  "@metamask-previews/network-controller": "22.2.1-preview-4b0cb25d",
  "@metamask-previews/notification-services-controller": "1.0.0-preview-4b0cb25d",
  "@metamask-previews/permission-controller": "11.0.6-preview-4b0cb25d",
  "@metamask-previews/permission-log-controller": "3.0.3-preview-4b0cb25d",
  "@metamask-previews/phishing-controller": "12.4.0-preview-4b0cb25d",
  "@metamask-previews/polling-controller": "12.0.3-preview-4b0cb25d",
  "@metamask-previews/preferences-controller": "15.0.2-preview-4b0cb25d",
  "@metamask-previews/profile-sync-controller": "8.1.1-preview-4b0cb25d",
  "@metamask-previews/queued-request-controller": "9.0.1-preview-4b0cb25d",
  "@metamask-previews/rate-limit-controller": "6.0.3-preview-4b0cb25d",
  "@metamask-previews/remote-feature-flag-controller": "1.6.0-preview-4b0cb25d",
  "@metamask-previews/selected-network-controller": "21.0.1-preview-4b0cb25d",
  "@metamask-previews/signature-controller": "23.2.1-preview-4b0cb25d",
  "@metamask-previews/token-search-discovery-controller": "2.1.0-preview-4b0cb25d",
  "@metamask-previews/transaction-controller": "46.0.0-preview-4b0cb25d",
  "@metamask-previews/user-operation-controller": "25.0.0-preview-4b0cb25d"
}

Copy link
Contributor

@ccharly ccharly left a comment

Choose a reason for hiding this comment

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

LGTM for accounts-related files.

@matthewwalsh0 matthewwalsh0 enabled auto-merge (squash) March 5, 2025 17:08
Copy link
Member

@mikesposito mikesposito left a comment

Choose a reason for hiding this comment

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

LGTM!

@matthewwalsh0 matthewwalsh0 merged commit 1c9b6ca into main Mar 5, 2025
136 checks passed
@matthewwalsh0 matthewwalsh0 deleted the chore/upgrade-ethereumjs branch March 5, 2025 17:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants