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-1153.md #6132

Closed
wants to merge 1 commit into from
Closed

Update eip-1153.md #6132

wants to merge 1 commit into from

Conversation

acegilz
Copy link

@acegilz acegilz commented Dec 14, 2022

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.

@github-actions github-actions bot added c-update Modifies an existing proposal t-core labels Dec 14, 2022
@eth-bot
Copy link
Collaborator

eth-bot commented Dec 14, 2022

Hi! I'm a bot, and I wanted to automerge your PR, but couldn't because of the following issue(s):


(fail) eip-1153.md

classification
updateEIP

@moodysalem
Copy link
Contributor

Please provide an explanation. The EIP is already implemented in 3/5 clients with pending PRs in the last 2, there doesn't seem to be a good reason to change the opcode values.

@acegilz
Copy link
Author

acegilz commented Dec 16, 2022

Nevermid, closing

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
c-update Modifies an existing proposal t-core
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants