Skip to content

Fix CLI build contract issues#2

Merged
sionsmith merged 2 commits into
mainfrom
fix/build-and-cli-contract
May 23, 2026
Merged

Fix CLI build contract issues#2
sionsmith merged 2 commits into
mainfrom
fix/build-and-cli-contract

Conversation

@sionsmith
Copy link
Copy Markdown
Contributor

Summary

  • make semrush the default Cargo run target
  • allow dry-run and batch estimate without requiring an API key
  • accept README-style underscore batch command names
  • align documented exit codes and README command examples with the implemented CLI
  • add CLI regression tests for the fixed behavior

Verification

  • cargo fmt --all -- --check
  • cargo check --all-targets
  • cargo test --all-targets
  • cargo clippy --all-targets -- -D warnings
  • cargo build --release
  • RUSTFLAGS='-D warnings' cargo test --all-targets\n\n## Notes\n- Live Semrush endpoint correctness is still not covered by mock HTTP contract tests; this PR focuses on build, CLI, and documentation-contract fixes.

@sionsmith sionsmith merged commit 5438096 into main May 23, 2026
6 checks passed
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