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 Constantinople Meta-EIP 1013 #1366

Merged
merged 4 commits into from
Sep 19, 2018
Merged

Conversation

5chdn
Copy link
Contributor

@5chdn 5chdn commented Aug 31, 2018

  • Remove EIP-210 from list
  • Replace EIP-1087 with EIP-1283
  • Add EIP-1234

As discussed at ethereum/pm#55; see also ethereum/pm#53

@eip-automerger
Copy link

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

@axic
Copy link
Member

axic commented Sep 19, 2018

@nicksavers @holiman can you review?

@holiman
Copy link
Contributor

holiman commented Sep 19, 2018

I'm not an eip editor, not sure who usually does this @Arachnid ?

@axic
Copy link
Member

axic commented Sep 19, 2018

Nobody I guess.

But by review I meant if this matches that of the agreement on AllCoreDevs.

Copy link
Contributor

@holiman holiman left a comment

Choose a reason for hiding this comment

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

LGTM

@eip-automerger eip-automerger merged commit f32661d into ethereum:master Sep 19, 2018
@5chdn 5chdn deleted the patch-3 branch September 20, 2018 05:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants