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-6492: Move to Final #7640

Merged
merged 33 commits into from
Sep 5, 2023
Merged

Update EIP-6492: Move to Final #7640

merged 33 commits into from
Sep 5, 2023

Conversation

Ivshti
Copy link
Contributor

@Ivshti Ivshti commented Sep 5, 2023

The last call deadline has expired, so the status is being changed to Final

…ually Deployed Contracts

* Error fixes
* Readability improvements
* Better definitions
* Moved counterfactual check to the beginning
* Propose a validation implementation that does not require pre-deploying singletons
* Consistency in using verification/validation

* Suggest a library to use
* Specify off-chain usage

* Change example contract to something a bit more compiler optimizable
* Remove ERC-4337 link so that the EIP can be considered stable
@Ivshti Ivshti requested a review from eth-bot as a code owner September 5, 2023 07:52
@github-actions github-actions bot added c-status Changes a proposal's status s-final This EIP is Final t-erc labels Sep 5, 2023
@eth-bot
Copy link
Collaborator

eth-bot commented Sep 5, 2023

✅ All reviewers have approved.

@eth-bot eth-bot changed the title Update EIP-6492: change status to Final Update EIP-6492: Move to Final Sep 5, 2023
@eth-bot eth-bot added the e-review Waiting on editor to review label Sep 5, 2023
@eth-bot eth-bot enabled auto-merge (squash) September 5, 2023 14:29
Copy link
Collaborator

@eth-bot eth-bot left a 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...

@eth-bot eth-bot merged commit d98a0a3 into ethereum:master Sep 5, 2023
15 of 16 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
c-status Changes a proposal's status e-review Waiting on editor to review s-final This EIP is Final t-erc
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants