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

rename ConsensusFork.EIP4844 to ConsensusFork.Deneb #4692

Merged
merged 1 commit into from
Mar 4, 2023
Merged

rename ConsensusFork.EIP4844 to ConsensusFork.Deneb #4692

merged 1 commit into from
Mar 4, 2023

Conversation

tersec
Copy link
Contributor

@tersec tersec commented Mar 4, 2023

Almost minimal. Some copyright year changes, and in REST serialization, since they tend to be on the same lines of code, some instances of looking for/producing the literal string eip4844 as a fork identifier in the protocol are replaced with deneb. The tests don't need a workaround anymore, either, of seeing a deneb directory and interpreting that as eip4844.

@tersec tersec enabled auto-merge (squash) March 4, 2023 10:56
@tersec tersec merged commit 3b41e6a into unstable Mar 4, 2023
@tersec tersec deleted the P3v branch March 4, 2023 13:35
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.

1 participant