chore(refactor): update so forked networks chainID is used and rename `fork-at` to fork-block-number` #372

dutterbutter · 2024-11-01T00:55:05Z

What 💻

Update so forked networks chainID is used, is overridden if explicitly set --chain-id
Rename fork-at to fork-block-number`
Updates tests accordingly

ChainID when forking a network was still 260
Renames fork-at to reflect anvil fork-block-number
Supports PR for alloy-zksync: feat: add support for forked network and fork-block-number popzxc/alloy-zksync#11

BEFORE:

AFTER:

…hain-id arg

…nd respects chainID during forking

src/config/cli.rs

dutterbutter added 3 commits October 31, 2024 12:14

feat: make use of forked network chainID unless explicitly set by --c…

96293b3

…hain-id arg

fix: update test to reflect changes

849b8d0

chore(refactor): update fork-at to reflect anvils fork-block-number a…

c8e730b

…nd respects chainID during forking

dutterbutter requested a review from a team as a code owner November 1, 2024 00:55

dutterbutter mentioned this pull request Nov 1, 2024

feat: add support for forked network and fork-block-number popzxc/alloy-zksync#11

Merged

dutterbutter requested a review from popzxc November 1, 2024 01:03

popzxc reviewed Nov 2, 2024

View reviewed changes

src/config/cli.rs Show resolved Hide resolved

fix: add alias for cli change

19b3330

dutterbutter requested a review from popzxc November 4, 2024 12:34

popzxc approved these changes Nov 4, 2024

View reviewed changes

dutterbutter merged commit 1cb9d6b into main Nov 4, 2024
10 checks passed

dutterbutter deleted the db/update-chainid-on-fork branch November 4, 2024 12:57