Skip to content

Conversation

@pdobacz
Copy link
Contributor

@pdobacz pdobacz commented Oct 27, 2025

πŸ—’οΈ Description

Add missing tests for two precompiles to ensure full coverage in evmone

πŸ”— Related Issues or PRs

N/A.

βœ… Checklist

  • All: Ran fast tox checks to avoid unnecessary CI fails, see also Code Standards and Enabling Pre-commit Checks:
    uvx tox -e static
  • All: PR title adheres to the repo standard - it will be used as the squash commit message and should start type(scope):.
  • All: Considered adding an entry to CHANGELOG.md.
  • All: Set appropriate labels for the changes (only maintainers can apply labels).

@pdobacz pdobacz force-pushed the tests/modexp-ripemd branch from 4cea394 to e3fe333 Compare October 27, 2025 11:54
Anuoluwapo25 pushed a commit to Anuoluwapo25/execution-specs that referenced this pull request Oct 27, 2025
Copy link
Contributor

@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.

LGTM!! Thanks for adding ripemd tests :D

@spencer-tb spencer-tb added C-enhance Category: an improvement or new feature A-test-tests Area: tests for execution spec tests labels Oct 29, 2025
@pdobacz pdobacz force-pushed the tests/modexp-ripemd branch from e3fe333 to 2c55972 Compare October 29, 2025 15:05
@spencer-tb spencer-tb merged commit e1f2df6 into ethereum:forks/osaka Oct 31, 2025
8 of 9 checks passed
@pdobacz pdobacz deleted the tests/modexp-ripemd branch October 31, 2025 19:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-test-tests Area: tests for execution spec tests C-enhance Category: an improvement or new feature

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants