Skip to content

feat(revme): add --omit-progress flag to statetest command#3419

Merged
rakita merged 1 commit intobluealloy:mainfrom
bluebluesaku:omit-progress
Feb 23, 2026
Merged

feat(revme): add --omit-progress flag to statetest command#3419
rakita merged 1 commit intobluealloy:mainfrom
bluebluesaku:omit-progress

Conversation

@bluebluesaku
Copy link
Contributor

The blockchaintest command already has --omit-progress to suppress the progress bar, but statetest was missing it. This is annoying when profiling with samply/perf since the progress bar keeps flooding the terminal. Added the same flag so both commands behave consistently.

Copy link
Member

@rakita rakita left a comment

Choose a reason for hiding this comment

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

lgtm, nice

@rakita
Copy link
Member

rakita commented Feb 16, 2026

Waiting for CI to finish

@codspeed-hq
Copy link

codspeed-hq bot commented Feb 16, 2026

Merging this PR will improve performance by 4.83%

⚡ 2 improved benchmarks
✅ 171 untouched benchmarks

Performance Changes

Benchmark BASE HEAD Efficiency
transact_commit_1000txs 2.8 ms 2.7 ms +4.83%
transact_1000tx_commit_inner_every_40 2.3 ms 2.2 ms +3.97%

Comparing bluebluesaku:omit-progress (200a83e) with main (67eecf4)

Open in CodSpeed

@bluebluesaku
Copy link
Contributor Author

Waiting for CI to finish

Looks goooood!

@rakita rakita merged commit c07925c into bluealloy:main Feb 23, 2026
31 checks passed
@github-actions github-actions bot mentioned this pull request Feb 23, 2026
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