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

readiness matrix: downgrade impls to 🚧 for 7620 #111

Closed
wants to merge 1 commit into from

Conversation

chfast
Copy link
Member

@chfast chfast commented May 13, 2024

Because of the recent spec change.

@chfast chfast requested review from gumb0 and pdobacz May 13, 2024 16:23
Copy link
Member

@pdobacz pdobacz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sounds good, but if we're being pedantic, we could degrade also megaspec and EIP, because #86 - this needs rebase, some editorial changes, and potentially expanding by the "no deadsubcontainers" rule.

OTOH the - **EIP-7620** - ethereum/EIPs#8522 (Remove TXCREATE, InitcodeTransaction and the Creator Contract) line can be taken down now, and the 7702 story put in its place (or not?)

Because of the recent spec change.
@pdobacz
Copy link
Member

pdobacz commented Jun 4, 2024

IIRC, the issue was with the initcode mode validation, and it is now fully specced out (Megaspec & EIPs). Evmone impl is in review, not sure about besu. However, I'm now more inclined to think that if we're doing such feedback-addressing updates, maybe we shouldn't be so strict with downgrading, as the bulk of work is still done.

Also, we had plans to additionally mark cells which are done-done (passing comprehensive reference tests). We can be strict about that.

@pdobacz
Copy link
Member

pdobacz commented Jun 17, 2024

I'm not sure this update is up-to-date anymore. Close or refresh?

@axic
Copy link
Member

axic commented Aug 14, 2024

Ping @chfast @pdobacz

@pdobacz
Copy link
Member

pdobacz commented Aug 19, 2024

Ping @chfast @pdobacz

looks fairly out-of-date by now, closing. I'll look over the readiness tables in general shortly

@pdobacz pdobacz closed this Aug 19, 2024
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.

3 participants