Skip to content
This repository has been archived by the owner on Nov 9, 2023. It is now read-only.

update to @metamask/json-rpc-engine #54

Merged
merged 5 commits into from
Sep 13, 2023

Conversation

legobeat
Copy link
Contributor

@legobeat legobeat commented Sep 4, 2023

@legobeat legobeat requested a review from a team as a code owner September 4, 2023 08:05
@legobeat legobeat changed the title fix: update to @metamask/json-rpc-engine update to @metamask/json-rpc-engine Sep 4, 2023
@legobeat legobeat changed the base branch from fb/bump-json-rpc-engine to main September 4, 2023 08:06
@legobeat legobeat marked this pull request as draft September 4, 2023 08:06
@legobeat legobeat marked this pull request as ready for review September 4, 2023 09:44
@legobeat legobeat force-pushed the bump-json-rpc-engine branch from 12bf9ba to 91cccf2 Compare September 4, 2023 22:42
@legobeat legobeat marked this pull request as draft September 4, 2023 22:42
@socket-security
Copy link

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

🚮 Removed packages: [email protected]

@legobeat legobeat force-pushed the bump-json-rpc-engine branch from e5953f3 to 0547993 Compare September 13, 2023 00:54
@legobeat
Copy link
Contributor Author

https://github.com/MetaMask/json-rpc-middleware-stream/actions/runs/6166324690/job/16735638837#step:11:75

Same uncovered lines on currently passing main, if anyone's curious about the jest coverage change.

@legobeat legobeat force-pushed the bump-json-rpc-engine branch from 8cbd01c to 0830fdc Compare September 13, 2023 00:58
@legobeat legobeat marked this pull request as ready for review September 13, 2023 01:13
@legobeat legobeat requested review from a team, kumavis and FrederikBolding September 13, 2023 01:13
@legobeat legobeat merged commit fafa136 into MetaMask:main Sep 13, 2023
3 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
dependencies Pull requests that update a dependency file
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants