Skip to content

Updated To Final#2449

Merged
nicksavers merged 1 commit intoethereum:masterfrom
MadeofTin:patch-9
Jan 27, 2020
Merged

Updated To Final#2449
nicksavers merged 1 commit intoethereum:masterfrom
MadeofTin:patch-9

Conversation

@MadeofTin
Copy link
Copy Markdown
Contributor

Updated to clarify there is no change in block reward, as requested in ACD Call 77. https://github.com/ethereum/pm

Notes for the call have not yet been published but, call 77 can be found on https://github.com/ethereum/pm

The condition is that block reward issuance is clarified that there is no change to the block reward. The last call review period has completed and so it is ready to be merged as Final.
@eip-automerger
Copy link
Copy Markdown

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

  • Trying to change EIP 2387 state from Last Call to Final

Copy link
Copy Markdown
Member

@axic axic left a comment

Choose a reason for hiding this comment

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

Can you please address #2448 (comment) ?

@axic
Copy link
Copy Markdown
Member

axic commented Dec 27, 2019

This is also broken: #2447 (comment)

@nicksavers nicksavers merged commit 26442c9 into ethereum:master Jan 27, 2020
Comment thread EIPS/eip-2387.md
type: Meta
status: Last Call
status: Final
review-period-end: 2019-12-12
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Leaving this in is wrong.

pizzarob pushed a commit to pizzarob/EIPs that referenced this pull request Jun 12, 2020
tkstanczak pushed a commit to tkstanczak/EIPs that referenced this pull request Nov 7, 2020
Arachnid pushed a commit to Arachnid/EIPs that referenced this pull request Mar 6, 2021
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.

4 participants