-
Notifications
You must be signed in to change notification settings - Fork 5.5k
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
Update EIP-4881: Move to Review #6812
Conversation
f8c8d90
to
29e6380
Compare
✅ All reviewers have approved. |
The commit 29e6380 (as a parent of a63de0c) contains errors. |
As discussed in EIP Editing Office Hour #14, the blocker for this PR to be merged is the mention of EIP-4444 in "Security consideration". However, the proposal does not seem to be depending on EIP-4444 in any way. |
29e6380
to
a39c517
Compare
Sorry I got a bit sidetracked last week with other work but I believe I've reworded it now without references to 4444 :) |
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.
All Reviewers Have Approved; Performing Automatic Merge...
Moving 4881 to Review Stage