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

E upgrade config #1135

Merged
merged 18 commits into from
May 1, 2024
Merged

E upgrade config #1135

merged 18 commits into from
May 1, 2024

Conversation

ceyonur
Copy link
Collaborator

@ceyonur ceyonur commented Mar 28, 2024

Why this should be merged

  • Adds E Upgrade Config
  • Maps Cancun activation to EUpgrade activation
  • Changes default activation timestamps for non-production chains as well
    • 0 and nil timestamps default to Avalanche activations, however non-production chains were able to skip this behaviour. Now they will also default when 0 or nil provided.

How this was tested

UTs + local testing

How is this documented

Tooling and docs should reflect that genesis timestamps must be non-zero

@ceyonur ceyonur marked this pull request as ready for review April 27, 2024 11:07
@ceyonur ceyonur requested a review from darioush as a code owner April 27, 2024 11:07
core/state_transition.go Outdated Show resolved Hide resolved
core/genesis.go Outdated Show resolved Hide resolved
plugin/evm/vm_upgrade_bytes_test.go Outdated Show resolved Hide resolved
@ceyonur ceyonur merged commit d02c4cb into master May 1, 2024
12 checks passed
@ceyonur ceyonur deleted the e-upgrade-config branch May 1, 2024 12:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

2 participants