Skip to content

Conversation

@legobeat
Copy link
Contributor

@legobeat legobeat commented May 28, 2024

Explanation

Updates @metamask/nonce-tracker to remove need for @ts-expect-error around the blockTracker type.

Also updates transaction-controller tests to reset providers and block trackers between each test.

References

Changelog

@metamask/network-controller

  • CHANGED: Update @metamask/eth-block-tracker from ^9.0.2 to ^9.0.3

@metamask/transaction-controller

  • CHANGED: Update @metamask/nonce-tracker from ^5.0.0 to ^6.0.0

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

@legobeat legobeat added dependencies Pull requests that update a dependency file team-security labels May 28, 2024
@legobeat legobeat force-pushed the deps-nonce-tracker-6 branch 5 times, most recently from 0d0f46c to 3d59210 Compare May 30, 2024 23:10
@legobeat legobeat force-pushed the deps-nonce-tracker-6 branch 2 times, most recently from caa09ab to 2d19826 Compare June 6, 2024 22:32
@legobeat legobeat force-pushed the deps-nonce-tracker-6 branch 2 times, most recently from 31bf8ca to 5a7b861 Compare June 8, 2024 00:50
@legobeat legobeat marked this pull request as ready for review June 8, 2024 00:56
@legobeat legobeat requested a review from a team June 8, 2024 00:56
@legobeat legobeat force-pushed the deps-nonce-tracker-6 branch from 5a7b861 to 962fc13 Compare June 10, 2024 00:23
@legobeat legobeat force-pushed the deps-nonce-tracker-6 branch from 962fc13 to c807342 Compare June 10, 2024 09:27
mcmire
mcmire previously approved these changes Jun 10, 2024
Copy link
Contributor

@mcmire mcmire left a comment

Choose a reason for hiding this comment

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

Nice!

@legobeat legobeat force-pushed the deps-nonce-tracker-6 branch from c807342 to d4c8478 Compare June 10, 2024 17:44
@legobeat
Copy link
Contributor Author

@metamaskbot publish-preview

@legobeat legobeat force-pushed the deps-nonce-tracker-6 branch from d4c8478 to 0f65e9e Compare June 10, 2024 18:06
@legobeat

This comment was marked as resolved.

@legobeat

This comment was marked as resolved.

@legobeat
Copy link
Contributor Author

Prerelease builds:

{
  "@metamask-previews/accounts-controller": "16.0.0-preview-0f65e9e",
  "@metamask-previews/address-book-controller": "5.0.0-preview-0f65e9e",
  "@metamask-previews/announcement-controller": "7.0.0-preview-0f65e9e",
  "@metamask-previews/approval-controller": "7.0.0-preview-0f65e9e",
  "@metamask-previews/assets-controllers": "32.0.0-preview-0f65e9e",
  "@metamask-previews/base-controller": "6.0.0-preview-0f65e9e",
  "@metamask-previews/build-utils": "3.0.0-preview-0f65e9e",
  "@metamask-previews/chain-controller": "0.0.0-preview-0f65e9e",
  "@metamask-previews/composable-controller": "7.0.0-preview-0f65e9e",
  "@metamask-previews/controller-utils": "11.0.0-preview-0f65e9e",
  "@metamask-previews/ens-controller": "12.0.0-preview-0f65e9e",
  "@metamask-previews/eth-json-rpc-provider": "4.0.0-preview-0f65e9e",
  "@metamask-previews/gas-fee-controller": "17.0.0-preview-0f65e9e",
  "@metamask-previews/json-rpc-engine": "9.0.0-preview-0f65e9e",
  "@metamask-previews/json-rpc-middleware-stream": "8.0.0-preview-0f65e9e",
  "@metamask-previews/keyring-controller": "17.0.0-preview-0f65e9e",
  "@metamask-previews/logging-controller": "5.0.0-preview-0f65e9e",
  "@metamask-previews/message-manager": "10.0.0-preview-0f65e9e",
  "@metamask-previews/name-controller": "8.0.0-preview-0f65e9e",
  "@metamask-previews/network-controller": "19.0.0-preview-0f65e9e",
  "@metamask-previews/notification-controller": "6.0.0-preview-0f65e9e",
  "@metamask-previews/permission-controller": "10.0.0-preview-0f65e9e",
  "@metamask-previews/permission-log-controller": "3.0.0-preview-0f65e9e",
  "@metamask-previews/phishing-controller": "10.0.0-preview-0f65e9e",
  "@metamask-previews/polling-controller": "8.0.0-preview-0f65e9e",
  "@metamask-previews/preferences-controller": "13.0.0-preview-0f65e9e",
  "@metamask-previews/profile-sync-controller": "0.0.0-preview-0f65e9e",
  "@metamask-previews/queued-request-controller": "0.12.0-preview-0f65e9e",
  "@metamask-previews/rate-limit-controller": "6.0.0-preview-0f65e9e",
  "@metamask-previews/selected-network-controller": "15.0.1-preview-0f65e9e",
  "@metamask-previews/signature-controller": "18.0.0-preview-0f65e9e",
  "@metamask-previews/transaction-controller": "32.0.0-preview-0f65e9e",
  "@metamask-previews/user-operation-controller": "12.0.0-preview-0f65e9e"
}

kumavis
kumavis previously approved these changes Jun 11, 2024
mikesposito
mikesposito previously approved these changes Jun 11, 2024
@legobeat legobeat force-pushed the deps-nonce-tracker-6 branch from 0f65e9e to 32a2f8e Compare June 13, 2024 00:16
shanejonas added a commit that referenced this pull request Jun 19, 2024
## Explanation

Please see https://github.com/MetaMask/MetaMask-planning/issues/2668

## References

See: https://github.com/MetaMask/MetaMask-planning/issues/2668
Unblocks: #4327

## Changelog

No user facing changes

## 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

---------

Co-authored-by: Jiexi Luan <jiexiluan@gmail.com>
Co-authored-by: Derek Brans <dbrans@gmail.com>
mcmire pushed a commit that referenced this pull request Jun 20, 2024
## Explanation

Please see MetaMask/MetaMask-planning#2668

## References

See: MetaMask/MetaMask-planning#2668
Unblocks: #4327

## Changelog

No user facing changes

## 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

---------

Co-authored-by: Jiexi Luan <jiexiluan@gmail.com>
Co-authored-by: Derek Brans <dbrans@gmail.com>
@legobeat legobeat dismissed stale reviews from mikesposito, kumavis, and mcmire via 2682049 July 12, 2024 00:36
@legobeat
Copy link
Contributor Author

Superseded by #4591

@legobeat legobeat closed this Aug 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Pull requests that update a dependency file team-security

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

4 participants