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-1898: Move to draft #5980

Merged
merged 6 commits into from
Nov 20, 2022
Merged

Conversation

macfarla
Copy link
Contributor

@macfarla macfarla commented Nov 16, 2022

Signed-off-by: Sally MacFarlane [email protected]

It is supported by Geth 1.9.6 (PR).
It is supported by Besu 20.10.4 (PR).

When opening a pull request to submit a new EIP, please use the suggested template: https://github.com/ethereum/EIPs/blob/master/eip-template.md

We have a GitHub bot that automatically merges some PRs. It will merge yours immediately if certain criteria are met:

  • The PR edits only existing draft PRs.
  • The build passes.
  • Your GitHub username or email address is listed in the 'author' header of all affected PRs, inside .
  • If matching on email address, the email address is the one publicly listed on your GitHub profile.

Signed-off-by: Sally MacFarlane <[email protected]>
@macfarla macfarla requested a review from eth-bot as a code owner November 16, 2022 04:09
@github-actions github-actions bot added c-status Changes a proposal's status s-draft This EIP is a Draft t-interface labels Nov 16, 2022
@eth-bot
Copy link
Collaborator

eth-bot commented Nov 16, 2022

A critical exception has occurred:
Message: pr 5980 is already merged; quitting
(cc @alita-moore, @mryalamanchi)

Signed-off-by: Sally MacFarlane <[email protected]>
Signed-off-by: Sally MacFarlane <[email protected]>
charles-cooper
charles-cooper previously approved these changes Nov 16, 2022
@charles-cooper
Copy link
Contributor

This could probably be moved to review or final stage. It has been implemented and stable in clients for years now.

@macfarla
Copy link
Contributor Author

Details

Thanks @charles-cooper - changed to Review

Signed-off-by: Sally MacFarlane <[email protected]>
@github-actions github-actions bot added s-review This EIP is in Review and removed s-draft This EIP is a Draft labels Nov 16, 2022
@macfarla macfarla changed the title EIP-1898 - change back to draft EIP-1898 - change to review status Nov 16, 2022
@Pandapip1 Pandapip1 changed the title EIP-1898 - change to review status Update EIP-1898: Move to review Nov 16, 2022
EIPS/eip-1898.md Outdated Show resolved Hide resolved
Signed-off-by: Sally MacFarlane <[email protected]>
@github-actions github-actions bot added s-draft This EIP is a Draft and removed s-review This EIP is in Review labels Nov 16, 2022
@macfarla macfarla changed the title Update EIP-1898: Move to review Update EIP-1898: Move to draft Nov 16, 2022
@macfarla
Copy link
Contributor Author

opened a separate PR for moving EIP-1474 back to draft also #5986

@xinbenlv
Copy link
Contributor

xinbenlv commented Nov 20, 2022

LGTM: the author approved and the update seems reasonable, agree to move back to Draft.

In terms of moving to Review -> Final, I love to see if we can accelerate that process if this EIP have been stable and implemented by clients. Editors to discuss

@eth-bot eth-bot enabled auto-merge (squash) November 20, 2022 17:11
@eth-bot eth-bot merged commit a9cc9b9 into ethereum:master Nov 20, 2022
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 s-draft This EIP is a Draft t-interface
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants