Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

Remove EIP-3074 #3582

Merged
merged 4 commits into from
Aug 13, 2024
Merged

Remove EIP-3074 #3582

merged 4 commits into from
Aug 13, 2024

Conversation

jochem-brouwer
Copy link
Member

Removes EIP 3074 (AUTH / AUTHCALL opcdoes), since it is superseded by EIP 7702 (EOA code delegation transactions)

@codecov Codecov
Copy link

codecov bot commented Aug 13, 2024

Codecov Report

Attention: Patch coverage is 50.00000% with 3 lines in your changes missing coverage. Please review.

Project coverage is 57.81%. Comparing base (4470cc3) to head (1b064f7).
Report is 2 commits behind head on master.

Additional details and impacted files

Impacted file tree graph

Flag Coverage Δ
block ?
client ?
evm 64.33% <50.00%> (?)
tx ?
wallet 0.00% <ø> (ø)

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

acolytec3
acolytec3 approved these changes Aug 13, 2024
View reviewed changes
Copy link
Contributor

@acolytec3 acolytec3 left a comment

Choose a reason for hiding this comment

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

LGTM

@acolytec3 acolytec3 merged commit 0686310 into master Aug 13, 2024
34 checks passed
@acolytec3 acolytec3 deleted the remove-3074 branch August 13, 2024 18:17
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers

@acolytec3 acolytec3 acolytec3 approved these changes

Assignees
No one assigned
Labels
package: evm PR state: needs review
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@jochem-brouwer @acolytec3 @holgerd77