-
Notifications
You must be signed in to change notification settings - Fork 458
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
Genesis, ChainID, and Context Configuration #18
Conversation
case 6: | ||
genesis = core.MainnetZoneGenesisBlock(params.MainnetZoneOneChainConfig) | ||
case 7: | ||
genesis = core.MainnetZoneGenesisBlock(params.MainnetZoneOneChainConfig) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
please fix the whitespace
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why is it all zone 1?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@jdowning100 Fixed in #23
}, | ||
Threshold: 2, | ||
} | ||
MainnetCheckpointOracle = &CheckpointOracleConfig{} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
please add a TODO here saying that we need to add checkpoint keys back in.
@@ -316,6 +255,7 @@ type CheckpointOracleConfig struct { | |||
type ChainConfig struct { | |||
ChainID *big.Int `json:"chainId"` // chainId identifies the current chain and is used for replay protection | |||
|
|||
Context int // Context defines the index in which the chain operates at | |||
HomesteadBlock *big.Int `json:"homesteadBlock,omitempty"` // Homestead switch block (nil = no fork, 0 = already homestead) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We don't need to do this here, but we should make an issue to remove all of the forks that exist in ETH and make this a clean slate
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@mechanikalk yeah it'll be an issue that we need to flag
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
would be best to do it all in one go
WIP for writing genesis block based off of context.