Skip to content
This repository has been archived by the owner on Oct 7, 2024. It is now read-only.

deps: bump @metamask/providers to v14; bump @metamask/snaps packages to latest #209

Merged
merged 2 commits into from
Nov 22, 2023

Conversation

legobeat
Copy link
Contributor

@legobeat legobeat commented Nov 22, 2023

  • deps: @metamask/providers@^13.0.0->^14.0.1
  • deps: bump @metamask/snaps- packages to latest

This is a breaking change due to the update of @metamask/providers: https://github.com/MetaMask/providers/releases/tag/v14.0.0

Consolidating updates into a single PR in order to not create duplication with incompatible versions of extension-port-stream.


Blocking

@legobeat legobeat added the type-dependencies Pull requests that update a dependency file label Nov 22, 2023
@legobeat legobeat mentioned this pull request Nov 22, 2023
1 task
Copy link

Removed dependencies detected. Learn more about Socket for GitHub ↗︎

🚮 Removed packages: @metamask/[email protected], @metamask/[email protected]

@legobeat legobeat marked this pull request as ready for review November 22, 2023 08:48
@legobeat legobeat requested a review from a team as a code owner November 22, 2023 08:48
@legobeat legobeat requested a review from a team November 22, 2023 08:48
@legobeat legobeat requested a review from jiexi November 22, 2023 09:03
FrederikBolding pushed a commit to MetaMask/metamask-extension that referenced this pull request Nov 22, 2023
## **Description**

- deps: @metamask/providers@^13.1.0->^14.0.2
- deps: extension-port-stream@^2.1.1->^3.0.0
- deps: json-rpc-middleware-stream@^4.2.1->^5.0.1


This fully removes runtime dependency on `@metamask/[email protected]`,
except for `@metamask/keyring-api`
(MetaMask/keyring-api#209)

## **Related issues**

## **Manual testing steps**

1. Go to this page...
2.
3.

## **Screenshots/Recordings**

<!-- If applicable, add screenshots and/or recordings to visualize the
before and after of your change. -->

### **Before**

<!-- [screenshots/recordings] -->

### **After**

<!-- [screenshots/recordings] -->

## **Pre-merge author checklist**

- [ ] I’ve followed [MetaMask Coding
Standards](https://github.com/MetaMask/metamask-extension/blob/develop/.github/guidelines/CODING_GUIDELINES.md).
- [ ] I've clearly explained what problem this PR is solving and how it
is solved.
- [ ] I've linked related issues
- [ ] I've included manual testing steps
- [ ] I've included screenshots/recordings if applicable
- [ ] I’ve included tests if applicable
- [ ] I’ve documented my code using [JSDoc](https://jsdoc.app/) format
if applicable
- [ ] I’ve applied the right labels on the PR (see [labeling
guidelines](https://github.com/MetaMask/metamask-extension/blob/develop/.github/guidelines/LABELING_GUIDELINES.md)).
Not required for external contributors.
- [ ] I’ve properly set the pull request status:
  - [ ] In case it's not yet "ready for review", I've set it to "draft".
- [ ] In case it's "ready for review", I've changed it from "draft" to
"non-draft".

## **Pre-merge reviewer checklist**

- [ ] I've manually tested the PR (e.g. pull and build branch, run the
app, test code being changed).
- [ ] I confirm that this PR addresses all acceptance criteria described
in the ticket it closes and includes the necessary testing evidence such
as recordings and or screenshots.

---------

Co-authored-by: MetaMask Bot <[email protected]>
@danroc danroc added this pull request to the merge queue Nov 22, 2023
Merged via the queue into MetaMask:main with commit abf955a Nov 22, 2023
17 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
type-dependencies Pull requests that update a dependency file
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants