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.

Sign up for GitHub

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

Update Shanghai and add Cancun #663

Merged
merged 1 commit into from
Dec 9, 2022
Merged

Update Shanghai and add Cancun #663

merged 1 commit into from
Dec 9, 2022

Conversation

timbeiko
Copy link
Collaborator

@timbeiko timbeiko commented Dec 8, 2022

This reflects the decisions made on ACD151, namely:

  • Including the EOF EIPs into Shanghai (may be removed if not ready by January)
  • Including EIP-4844 in a newly specified Cancun upgrade, set to follow Shanghai (naming follows devcon locations chronologically)
  • Moving previously CFI'd EIPs to the Cancun spec to discuss them further in future calls.
    • Note: CFI'd should probably be moved outside of specific upgrade specs, but wanted to minimize changes in this specific PR, so just stuck to existing pattern.

@timbeiko timbeiko mentioned this pull request Dec 8, 2022
Closed
@holgerd77
Copy link
Contributor

LGTM

@timbeiko timbeiko merged commit 1b33944 into master Dec 9, 2022
@holgerd77
Copy link
Contributor

Do we want an entry in the table from the root README as well? 🤔

@holgerd77
Copy link
Contributor

(so: for Cancun)

@timbeiko
Copy link
Collaborator Author

timbeiko commented Dec 9, 2022

Good catch, @holgerd77 ! Quickly added it in a follow up commit: ccf47a3

@SamWilsn SamWilsn deleted the acd151 branch April 10, 2024 23:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@yperbasis yperbasis yperbasis approved these changes

@djrtwo djrtwo djrtwo approved these changes

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

Successfully merging this pull request may close these issues.
4 participants
@timbeiko @holgerd77 @djrtwo @yperbasis