Skip to content

chain_config: Add nil check to blobschedule configs#13573

Merged
somnergy merged 1 commit intomainfrom
som/blobschedule_nil
Jan 27, 2025
Merged

chain_config: Add nil check to blobschedule configs#13573
somnergy merged 1 commit intomainfrom
som/blobschedule_nil

Conversation

@somnergy
Copy link
Copy Markdown
Member

Fixes issues where config files have some values and not all, or empty values for forks.

Note: This config is picked up for custom (unlisted) networks after init cmd is run using a correctly formatted genesis.json file (see pectra-devnet-5 gensis file for example)

@somnergy somnergy requested a review from yperbasis January 27, 2025 10:12
@somnergy somnergy requested a review from taratorio January 27, 2025 10:12
@somnergy somnergy changed the title Add nil check to blobschedule configs chain_config: Add nil check to blobschedule configs Jan 27, 2025
@somnergy somnergy merged commit ad35609 into main Jan 27, 2025
@somnergy somnergy deleted the som/blobschedule_nil branch January 27, 2025 13:57
somnergy added a commit that referenced this pull request Jan 28, 2025
Fixes issues where config files have some values and not all, or empty
values for forks.

Note: This config is picked up for custom (unlisted) networks after
`init` cmd is run using a correctly formatted `genesis.json` file (see
[pectra-devnet-5
gensis](https://github.com/ethpandaops/pectra-devnets/blob/master/network-configs/devnet-5/metadata/genesis.json)
file for example)
somnergy added a commit that referenced this pull request Jan 28, 2025
Fixes issues where config files have some values and not all, or empty
values for forks.

Note: This config is picked up for custom (unlisted) networks after
`init` cmd is run using a correctly formatted `genesis.json` file (see
[pectra-devnet-5
gensis](https://github.com/ethpandaops/pectra-devnets/blob/master/network-configs/devnet-5/metadata/genesis.json)
file for example)
@VBulikov VBulikov mentioned this pull request Feb 5, 2025
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