8000 Main updates for Prague by fselmo · Pull Request #317 · ethereum/eth-tester · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Main updates for Prague #317

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

Merged
merged 6 commits into from
Apr 23, 2025
Merged

Main updates for Prague #317

merged 6 commits into from
Apr 23, 2025

Conversation

fselmo
Copy link
Contributor
@fselmo fselmo commented Apr 11, 2025

What was wrong?

Missing Prague updates

How was it fixed?

  • Serialize requests_hash for Prague block headers
  • Serialize authorization list fields
  • Add comprehensive test for sending signed set code transaction
  • Add test for checking new block fields at fork transition
  • Add some obvious validation tests
  • Add missing Cancun field validation tests

Todo:

Cute Animal Picture

Screenshot 2025-04-23 at 12 57 53

@fselmo fselmo changed the title Main updates for Prague: Main updates for Prague Apr 11, 2025
@fselmo fselmo mentioned this pull request Apr 11, 2025
5 tasks
fselmo added 3 commits April 21, 2025 17:08
- Use ``PragueVM`` by default
- Serialize ``requests_hash`` for Prague block headers
- Serialize, validate, and normalize authorization list fields
- Add type 4 to known transaction types
- Add test for sending signed set code transaction
- Add test for checking new block fields at fork transition
- Add some validation tests for type 4 transactions / authorization lists
- Add a test for building and sending a type 4 transaction via
  py-evm backend.
- Test sending raw, "send", and "send_signed" APIs for
  set code transaction. This tests the transaction builder's
  ``new_set_code_transaction`` as well as the
  ``new_unsigned_set_code_transaction`` methods.
@fselmo fselmo marked this pull request as ready for review April 23, 2025 18:58
@fselmo fselmo requested review from pacrob, kclowes and reedsa April 23, 2025 18:58
Copy link
Contributor
@kclowes kclowes left a comment

Choose a reason for hiding this comment

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

🚀

< 8164 button aria-expanded="false" type="button" data-view-component="true" class="js-details-target btn-sm btn float-right"> Hide details View details @fselmo fselmo merged commit 92827c4 into ethereum:main Apr 23, 2025
29 checks passed
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.

2 participants
0