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

Arrow Glacier hard fork / EIP-4345 support #2045

Merged
merged 2 commits into from Dec 15, 2021
Merged

Conversation

fselmo
Copy link
Collaborator

@fselmo fselmo commented Nov 19, 2021

What was wrong?

  • Arrow Glacier hard fork is just around the corner [here] and we need to support it.

How was it fixed?

  • Added support for EIP-4345 which is the only change we need to support Arrow Glacier.
  • Updated ethereum/tests to version 10.2
  • Added /json-fixtures/test_difficulty.py to test against the new ethereum/tests/DifficultyTests

To-Do

  • Clean up commit history
  • Add entry to the release notes
  • Add test fillers + tests for new ethereum/tests difficulty tests

Cute Animal Picture

20211213_091312

@fselmo fselmo force-pushed the eip-4345 branch 3 times, most recently from f1f56ec to 1ca990d Compare November 19, 2021 22:11
@fselmo fselmo force-pushed the eip-4345 branch 7 times, most recently from 9805314 to 13e6869 Compare December 14, 2021 20:53
@fselmo fselmo force-pushed the eip-4345 branch 5 times, most recently from ba8e31f to 016ff1d Compare December 15, 2021 03:13
@fselmo fselmo marked this pull request as ready for review December 15, 2021 03:13
@fselmo fselmo force-pushed the eip-4345 branch 2 times, most recently from 9cd7423 to 26302ee Compare December 15, 2021 17:37
@fselmo fselmo requested a review from wolovim December 15, 2021 17:44
Also updated test fixture to v10.2 to pull in the new `/DifficultyTests` and update the test suite. This had to be done by commit hash as updating the submodule was giving some issues. The commit hash can be found in the actual release notes for the new `ethereum/tests` here: https://github.com/ethereum/tests/releases/tag/v10.2
- Add a new `/json-fixtures` test for testing the newly added `ethereum/tests/DifficultyTests`. This includes tests for EIP-4345 for the new difficulty bomb changes.
Copy link
Member

@wolovim wolovim left a comment

Choose a reason for hiding this comment

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

LGTM! appreciate the pair review. difficulty tests look like a solid starting point.

@fselmo fselmo merged commit eef377c into ethereum:master Dec 15, 2021
@fselmo fselmo deleted the eip-4345 branch December 15, 2021 18:28
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.

None yet

2 participants