Skip to content

Refactor CLI out-format tests into modules#2084

Merged
oferchen merged 1 commit into
masterfrom
implement-pure-rust-rsync-3.4.1-with-feature-parity
Nov 8, 2025
Merged

Refactor CLI out-format tests into modules#2084
oferchen merged 1 commit into
masterfrom
implement-pure-rust-rsync-3.4.1-with-feature-parity

Conversation

@oferchen
Copy link
Copy Markdown
Owner

@oferchen oferchen commented Nov 8, 2025

Summary

  • split the CLI out-format test suite into smaller modules to stay within the enforced line limits
  • point the frontend test harness at the new module tree

Testing

  • cargo test --test cli_binaries

Codex Task

@oferchen oferchen merged commit b741107 into master Nov 8, 2025
@oferchen oferchen deleted the implement-pure-rust-rsync-3.4.1-with-feature-parity branch November 8, 2025 19:54
oferchen added a commit that referenced this pull request May 16, 2026
Single-doc audit covering three follow-ups to the zsync-inspired
matching work. All three checks pass: benchmark scripts confine
destructive operations to mktemp or allowlisted /tmp paths, the CLI
and daemon expose no zsync-internal tunables, and the protocol crate
has zero compile-time or wire-level coupling to the matching crate.
oferchen added a commit that referenced this pull request May 18, 2026
Single-doc audit covering three follow-ups to the zsync-inspired
matching work. All three checks pass: benchmark scripts confine
destructive operations to mktemp or allowlisted /tmp paths, the CLI
and daemon expose no zsync-internal tunables, and the protocol crate
has zero compile-time or wire-level coupling to the matching crate.
oferchen added a commit that referenced this pull request May 18, 2026
Single-doc audit covering three follow-ups to the zsync-inspired
matching work. All three checks pass: benchmark scripts confine
destructive operations to mktemp or allowlisted /tmp paths, the CLI
and daemon expose no zsync-internal tunables, and the protocol crate
has zero compile-time or wire-level coupling to the matching crate.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant