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

Jump to bottom

Add new chiado chainspec and delete denver #4531

Merged
merged 21 commits into from Oct 6, 2022
Merged

Add new chiado chainspec and delete denver #4531

merged 21 commits into from Oct 6, 2022

Conversation

jmederosalvarado
Copy link
Member

Changes:

  • Add new chiado chainspec and remove old denver

Types of changes

What types of changes does your code introduce?
Put an x in the boxes that apply

  • Bugfix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation Update
  • Code style update (formatting, renaming)
  • Refactoring (no functional changes, no api changes)
  • Build related changes
  • Other (please describe): config

Testing

Requires testing

  • Yes
  • No

LukaszRozmej
LukaszRozmej approved these changes Sep 7, 2022
View reviewed changes
Copy link
Member

@LukaszRozmej LukaszRozmej left a comment

Choose a reason for hiding this comment

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

Should we try starting it for testing?

src/Nethermind/Nethermind.Runner/configs/chiado.cfg Show resolved Hide resolved
@jmederosalvarado jmederosalvarado force-pushed the gnosis/testnets-reboot branch 2 times, most recently from 883a7c0 to 9eb5839 Compare September 27, 2022 18:36
@jmederosalvarado
Add new chiado chainspec
7eeb2ff
@jmederosalvarado
Remove old denver chainspec
f5d36c1
@jmederosalvarado
Delete denver config
8c94c3b
@jmederosalvarado
Update chiado config
b27d925
@jmederosalvarado
Fix template error in chiado spec
16e1c9a
@jmederosalvarado
Update chiado network id
d905e82
@jmederosalvarado
Add enodes and merge to chiado
92ab550
@jmederosalvarado
Update chiado config
8e0255b
@jmederosalvarado
Remove chiado bootnodes from chainspec
e2c57bd
@jmederosalvarado
Remove redundant eips from chiado chainspec
a84e56c 
Remove `eip152Transition`, `eip1108Transition` and `eip2565Transition`
from chainspec. They are inferred from precompiles in the `accounts`
field. See [ChainSpecLoader#L167-L171](https://github.com/nethermindeth/nethermind/blob/5b73a2771ed00c6be792fc46f8d5c56333a28b11/src/Nethermind/Nethermind.Specs/ChainSpecStyle/ChainSpecLoader.cs#L167-L171)
@jmederosalvarado
Move baseFeePerGas to genesis
df74521
@jmederosalvarado
new chiado again
e6d4eff
@jmederosalvarado
Update chiado chainspec
a7fb74d
@jmederosalvarado
Add chiado hf to include more validator keys
911b05c
@jmederosalvarado
Add bootnodes to chiado
e1f4f36
@jmederosalvarado
Remove change to validator set contract to avoid unexpected hardfork
6866a2c
@jmederosalvarado jmederosalvarado merged commit b15c131 into master Oct 6, 2022
@jmederosalvarado jmederosalvarado deleted the gnosis/testnets-reboot branch October 6, 2022 20:31
rubo pushed a commit that referenced this pull request Oct 13, 2022
@jmederosalvarado @rubo
Add new chiado chainspec and delete denver (#4531)
183b6d9
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@dapplion dapplion dapplion left review comments

@MarekM25 MarekM25 Awaiting requested review from MarekM25

@LukaszRozmej LukaszRozmej Awaiting requested review from LukaszRozmej

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@jmederosalvarado @LukaszRozmej @dapplion