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

Updates difficulty calculation per EIP PR 1240 #710

Closed
wants to merge 1 commit into from

Conversation

MicahZoltu
Copy link

EIP number TBD, will update once decided.
ethereum/EIPs#1240

EIP number TBD, will update once decided.
ethereum/EIPs#1240
@nicksavers
Copy link
Contributor

@MicahZoltu Do you want to keep this around?

This would require a fork. The current master branch describes Byzantium. And as far as I'm aware, it is not planned for the constantinople branch either.

It's fine to keep this PR around for as long as the EIP is in Draft in case it gets Accepted and included in a fork, but for now I don't expect to merge it.

@MicahZoltu
Copy link
Author

I'm somewhat ambivalent to closing this or keeping it open. I believe I was outvoted for this change, so perhaps I should just close the PR as well (even though I still think it is a good idea).

@nicksavers
Copy link
Contributor

Then I'd recommend to close it for now and create a new PR into a branch for the fork in which it will be included.

@yperbasis
Copy link
Member

Closing since EIP-1240 hasn't been accepted.

@yperbasis yperbasis closed this Jun 11, 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.

3 participants