Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(genesisapplier): create genesisapplier package #44

Merged
merged 1 commit into from
Jul 11, 2024

Conversation

tremarkley
Copy link
Contributor

@tremarkley tremarkley commented Jul 10, 2024

Part of #41

  • Create the genesisapplier package, which will be used for making any modifications that are needed to genesis files.
  • Adds a UpdateGenesisWithConfig function inside genesisapplier, which given a genesis and a config, returns a genesis with the updated config values.
    • This allows us to workaround the foundry bug that prevents anvil from overriding the chain id with the chain id supplied on the cli whenever a genesis file is passed into the --init flag.

@tremarkley
Copy link
Contributor Author

tremarkley commented Jul 10, 2024

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @tremarkley and the rest of your teammates on Graphite Graphite

go.mod Show resolved Hide resolved
@tremarkley tremarkley force-pushed the harry/genesis_applier branch 3 times, most recently from de7f15d to cd6e254 Compare July 10, 2024 23:19
@tremarkley tremarkley marked this pull request as ready for review July 10, 2024 23:24
@tremarkley tremarkley requested a review from a team as a code owner July 10, 2024 23:24
supersim_test.go Outdated Show resolved Hide resolved
genesis-applier/genesis_applier.go Outdated Show resolved Hide resolved
genesis-applier/genesis_applier.go Outdated Show resolved Hide resolved
@tremarkley tremarkley force-pushed the harry/orchestrator branch from 42706a0 to dcf29b3 Compare July 11, 2024 17:29
@tremarkley tremarkley force-pushed the harry/genesis_applier branch from cd6e254 to fc54c94 Compare July 11, 2024 17:29
@tremarkley tremarkley force-pushed the harry/orchestrator branch from dcf29b3 to c33a48e Compare July 11, 2024 17:36
@tremarkley tremarkley force-pushed the harry/genesis_applier branch from fc54c94 to ce0259d Compare July 11, 2024 17:36
@tremarkley tremarkley force-pushed the harry/orchestrator branch from c33a48e to e556e6e Compare July 11, 2024 17:55
@tremarkley tremarkley force-pushed the harry/genesis_applier branch 2 times, most recently from f8bda9f to 88dc62b Compare July 11, 2024 18:05
@tremarkley tremarkley changed the base branch from harry/orchestrator to main July 11, 2024 18:05
@tremarkley tremarkley force-pushed the harry/genesis_applier branch from 88dc62b to 34867fc Compare July 11, 2024 18:12
@tremarkley tremarkley force-pushed the harry/genesis_applier branch from 34867fc to 749d3ce Compare July 11, 2024 18:23
@tremarkley tremarkley changed the title feat(genesis-applier): create genesis_applier package feat(genesis-applier): create genesisapplier package Jul 11, 2024
@tremarkley tremarkley merged commit 68bcec2 into main Jul 11, 2024
3 checks passed
@tremarkley tremarkley changed the title feat(genesis-applier): create genesisapplier package feat(genesisapplier): create genesisapplier package Jul 11, 2024
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