Skip to content

Remove duplicate call to setupInfuraProvider - #270

Merged
whymarrh merged 1 commit into
MetaMask:developfrom
whymarrh:tidy-network-controller-initialize
Sep 9, 2020
Merged

whymarrh merged 1 commit into
MetaMask:developfrom
whymarrh:tidy-network-controller-initialize

Conversation

@whymarrh

@whymarrh whymarrh commented Sep 9, 2020

Copy link
Copy Markdown
Contributor

This PR removes a duplicate call to setupInfuraProvider and merges all of the Infura cases into the single call.

@whymarrh
whymarrh requested a review from a team as a code owner September 9, 2020 16:34

@Gudahtt Gudahtt left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM!

@whymarrh
whymarrh merged commit 6056048 into MetaMask:develop Sep 9, 2020
@whymarrh
whymarrh deleted the tidy-network-controller-initialize branch September 9, 2020 16:39
Mrtenz pushed a commit that referenced this pull request Oct 16, 2025
Mrtenz pushed a commit to Mrtenz/core that referenced this pull request Jun 17, 2026
Naz-Ovh pushed a commit to 0x-fork/metamask-core that referenced this pull request Sep 13, 2026
…etaMask#270)

<!--
Thanks for your contribution! Take a moment to answer these questions so
that reviewers have the information they need to properly understand
your changes:

* What is the current state of things and why does it need to change?
* What is the solution your changes offer and how does it work?

Are there any issues or other links reviewers should consult to
understand this pull request better? For instance:

* Fixes #12345
* See: #67890
-->
The current use of the `is` function from the `superstruct` package is
drastically slowing down caip type checks. This replaces those function
calls with much faster regex checks.
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