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

(Partially) update Cancun PR table #769

Closed
wants to merge 1 commit into from

Conversation

haltman-at
Copy link
Contributor

I figured I'd update the Cancun PR table. I removed the column for EIP-6475 and updated the remaining three columns. Well, partly -- I actually largely left the column for 4844 alone, because that one's complicated and I didn't want to take the time right now to figure out everything going on there. But I updated the other two columns, and updated one entry in the 4844 column.

I haven't added columns for the ones that are merely CFI for now. (One of which still hasn't even been merged as a proper EIP yet!)

@codecov-commenter
Copy link

Codecov Report

Patch and project coverage have no change.

Comparison is base (aadc2a4) 74.01% compared to head (536f387) 74.01%.

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #769   +/-   ##
=======================================
  Coverage   74.01%   74.01%           
=======================================
  Files         570      570           
  Lines       31557    31557           
=======================================
  Hits        23356    23356           
  Misses       8201     8201           
Flag Coverage Δ
unittests 74.01% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

@timbeiko
Copy link
Contributor

timbeiko commented Jun 8, 2023

Sorry, I missed this! Will close given it's now superseded by #792

@timbeiko timbeiko closed this Jun 8, 2023
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