Skip to content

Split CLI benchmark args#264

Merged
yvette-carlisle merged 1 commit into
mainfrom
xy/split-cli-benchmark-args
Jun 29, 2026
Merged

Split CLI benchmark args#264
yvette-carlisle merged 1 commit into
mainfrom
xy/split-cli-benchmark-args

Conversation

@yvette-carlisle

Copy link
Copy Markdown
Member

Summary

  • move ELF CLI benchmark clap argument models into a private child module
  • keep crate-local crate::args::{BenchmarkArgs, BenchmarkCommand, BenchmarkReportArgs, BenchmarkRunArgs} imports working
  • keep benchmark run/report task mapping behavior unchanged

Validation

  • cargo make fmt-rust
  • cargo make check-rust
  • cargo make lint-vstyle
  • cargo make lint-rust
  • cargo make test-rust: 370 passed / 92 skipped
  • cargo make check: 370 passed / 92 skipped
  • cargo make test-rust-integration: 92 passed / 370 skipped
  • cargo run -p elf -- benchmark --help
  • cargo run -p elf -- benchmark run --dry-run --pretty
  • cargo run -p elf -- benchmark report --dry-run --pretty
  • cargo make check-trace-gate
  • git diff --cached --check

Skeptic review found no blockers and recommended commit.

…ent models into a private child module after local validation.","authority":"manual"}
@yvette-carlisle yvette-carlisle merged commit 0cd459f into main Jun 29, 2026
12 checks passed
@yvette-carlisle yvette-carlisle deleted the xy/split-cli-benchmark-args branch June 29, 2026 13:09
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.

1 participant