Skip to content

Conversation

@LouisTsai-Csie
Copy link
Collaborator

πŸ—’οΈ Description

None

πŸ”— Related Issues or PRs

None

βœ… 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: Considered updating the online docs in the ./docs/ directory.
  • All: Set appropriate labels for the changes (only maintainers can apply labels).
  • Tests: Ran mkdocs serve locally and verified the auto-generated docs for new tests in the Test Case Reference are correctly formatted.
  • Tests: For PRs implementing a missed test case, update the post-mortem document to add an entry the list.
  • Ported Tests: All converted JSON/YML tests from ethereum/tests or tests/static have been assigned @ported_from marker.

Cute Animal Picture

Put a link to a cute animal picture inside the parenthesis-->

@LouisTsai-Csie LouisTsai-Csie self-assigned this Dec 22, 2025
@codecov
Copy link

codecov bot commented Dec 22, 2025

Codecov Report

βœ… All modified and coverable lines are covered by tests.
⚠️ Please upload report for BASE (forks/amsterdam@8b089c9). Learn more about missing BASE report.
⚠️ Report is 4 commits behind head on forks/amsterdam.

Additional details and impacted files
@@                Coverage Diff                 @@
##             forks/amsterdam    #1946   +/-   ##
==================================================
  Coverage                   ?   86.26%           
==================================================
  Files                      ?      538           
  Lines                      ?    34557           
  Branches                   ?     3222           
==================================================
  Hits                       ?    29809           
  Misses                     ?     4161           
  Partials                   ?      587           
Flag Coverage Ξ”
unittests 86.26% <ΓΈ> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

β˜” View full report in Codecov by Sentry.
πŸ“’ Have feedback on the report? Share it here.

πŸš€ New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • πŸ“¦ JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@LouisTsai-Csie LouisTsai-Csie added A-test-benchmark Area: execution_testing.benchmark and tests/benchmark C-refactor Category: refactor labels Dec 22, 2025
@LouisTsai-Csie LouisTsai-Csie changed the title Refactor(tests): benchmark regex matching logic Refactor(test-benchmark): benchmark regex matching logic Dec 22, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-test-benchmark Area: execution_testing.benchmark and tests/benchmark C-refactor Category: refactor

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants