Skip to content

Conversation

@raxhvl
Copy link
Member

@raxhvl raxhvl commented Oct 13, 2025

🗒️ Description

Adds new EIP-7928 test case targeting the coinbase.

  • All: Ran fast tox checks to avoid unnecessary CI fails, see also Code Standards and Enabling Pre-commit Checks:
    uvx --with=tox-uv tox -e lint,typecheck,spellcheck,markdownlint
  • All: PR title adheres to the repo standard - it will be used as the squash commit message and should start type(scope):.

Copy link
Collaborator

@LouisTsai-Csie LouisTsai-Csie left a comment

Choose a reason for hiding this comment

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

LGTM but i think @fselmo would be the best reviewer!

Copy link
Collaborator

@spencer-tb spencer-tb left a comment

Choose a reason for hiding this comment

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

Yep! @fselmo for the merge :)

@fselmo
Copy link
Collaborator

fselmo commented Oct 13, 2025

Lgtm!

@fselmo fselmo merged commit 9dff8d6 into main Oct 13, 2025
16 checks passed
@fselmo fselmo deleted the feat/eip-7928/test-cases branch October 13, 2025 13:28
kclowes pushed a commit to kclowes/execution-spec-tests that referenced this pull request Oct 20, 2025
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.

5 participants