Skip to content

Add local networks#30

Merged
liamzebedee merged 13 commits intomasterfrom
local-ovm/base
Oct 8, 2021
Merged

Add local networks#30
liamzebedee merged 13 commits intomasterfrom
local-ovm/base

Conversation

@liamzebedee
Copy link
Copy Markdown
Contributor

@liamzebedee liamzebedee commented Jun 22, 2021

  • adds watch mode to perform rebuild upon file change
  • adds local networks for hardhat (31337) and ovm (420)

Epic: Synthetixio/issues#209

@liamzebedee liamzebedee requested a review from 0xclem June 22, 2021 07:11
'https://optimism.io/images/metamask_icon.png',
],
},
420: {
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

TODO: had a chat with Clem. We will probably put this in a guide - apparently Metamask's API complains if the rpc url is http, which it will be for the local optimism ops node.

this uses chokidar, which is used by other big companies. It's relatively simple, and I'd like to use typescript's built-in watch mode, but we have a slightly more complex setup with node and webpacked versions.
This was really confusing to read.
@liamzebedee liamzebedee requested a review from etn0m October 7, 2021 02:26
Copy link
Copy Markdown
Contributor

@etn0m etn0m left a comment

Choose a reason for hiding this comment

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

glorious

@liamzebedee
Copy link
Copy Markdown
Contributor Author

@JChiaramonte7 glory be

looks like there's a timeout error here:

● @synthetixio/data tests › debtSnapshots query › should return debtSnapshots data from l1
thrown: "Exceeded timeout of 5000 ms for a test.

Is this related or extraneous (eg. intermittent)?

@liamzebedee liamzebedee merged commit 7d49509 into master Oct 8, 2021
noisekit pushed a commit that referenced this pull request Aug 10, 2022
chore(webpack): improving caching with webpack
@noisekit noisekit deleted the local-ovm/base branch August 24, 2022 02:57
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