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

Update EIP-7702: add instruction to remove code at end of tx #8869

Closed
wants to merge 1 commit into from

Conversation

NiloCK
Copy link
Contributor

@NiloCK NiloCK commented Sep 6, 2024

Original version of the EIP had:

At the end of the transaction, set the contract_code of each signer back to empty.

But currently no similar instruction is present. This PR replaces it.

This instruction was explicit in earlier versions, but was lost to edits.
@NiloCK NiloCK requested a review from eth-bot as a code owner September 6, 2024 14:16
@github-actions github-actions bot added c-update Modifies an existing proposal s-review This EIP is in Review t-core labels Sep 6, 2024
@eth-bot
Copy link
Collaborator

eth-bot commented Sep 6, 2024

File EIPS/eip-7702.md

Requires 1 more reviewers from @adietrichs, @lightclient, @SamWilsn, @vbuterin

@eth-bot eth-bot added the a-review Waiting on author to review label Sep 6, 2024
@eth-bot eth-bot changed the title Update 7702: add instruction to remove code at end of tx Update EIP-7702: add instruction to remove code at end of tx Sep 6, 2024
@lightclient
Copy link
Member

This is specifically no longer a goal of 7702

@NiloCK
Copy link
Contributor Author

NiloCK commented Sep 6, 2024

This is specifically no longer a goal of 7702

Interesting, thanks.

In that case, I expect the point

Once this is implemented, allowing EOAs to migrate permanently is "only one line of code": just add a flag to not set the code back to empty at the end.

should be removed from the Forward-compatibility section.

@NiloCK
Copy link
Contributor Author

NiloCK commented Sep 6, 2024

Closing - opened #8871.

@NiloCK NiloCK closed this Sep 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
a-review Waiting on author to review c-update Modifies an existing proposal s-review This EIP is in Review t-core
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants