Skip to content

New chain deploys#41

Open
JasperTimm wants to merge 3 commits intomainfrom
new-chain-deploys
Open

New chain deploys#41
JasperTimm wants to merge 3 commits intomainfrom
new-chain-deploys

Conversation

@JasperTimm
Copy link
Collaborator

Bunch of config changes and script changes for the new chains: Base and zkSync Era

@codecov-commenter
Copy link

Codecov Report

Patch coverage: 68.75% and project coverage change: -1.32 ⚠️

Comparison is base (1d65b7f) 97.01% compared to head (ff0bfde) 95.70%.

❗ Current head ff0bfde differs from pull request most recent head 87ef694. Consider uploading reports for the commit 87ef694 to get more accurate results

❗ Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #41      +/-   ##
==========================================
- Coverage   97.01%   95.70%   -1.32%     
==========================================
  Files           4        4              
  Lines         335      349      +14     
  Branches       88       92       +4     
==========================================
+ Hits          325      334       +9     
- Misses         10       15       +5     
Impacted Files Coverage Δ
ethereum/kycdao-ntnft/contracts/PriceFeed.sol 86.84% <68.75%> (-13.16%) ⬇️

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

@JasperTimm JasperTimm requested a review from blackghost1987 May 19, 2023 11:50
accounts: {
mnemonic: test_mnemonic(),
},
ethNetwork: "goerli", // The Ethereum Web3 RPC URL, or the identifier of the network (e.g. `mainnet` or `goerli`)
Copy link
Member

Choose a reason for hiding this comment

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

why does this one have ethNetwork and Base doesn't, while Base has chainID and this one doesn't?

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.

3 participants