Skip to content

Apply cargo fmt across the rust crate#159

Merged
tinpotnick merged 1 commit into
babblevoice:mainfrom
PierreFouquet:chore-cargo-fmt-rust-crate
Jun 9, 2026
Merged

Apply cargo fmt across the rust crate#159
tinpotnick merged 1 commit into
babblevoice:mainfrom
PierreFouquet:chore-cargo-fmt-rust-crate

Conversation

@PierreFouquet

Copy link
Copy Markdown
Contributor

Problem

The Rust CI workflow fails at the rustfmt step:

cargo fmt --all -- --check
Diff in rust/src/tone.rs:201 ...
Process completed with exit code 1

The rust/ tree was committed without ever running rustfmt, so the drift spans the whole crate (27 files), not just tone.rs.

Fix

Run cargo fmt --all and commit the result. Changes are purely formatting — multi-line enum variants, import ordering, block wrapping — with no semantic effect. Verified with cargo fmt --all -- --check (now passes) using stable rustfmt 1.8.0, default config (no rustfmt.toml), edition 2021 — matching the @stable toolchain CI uses.

This is independent of the Docker build / libilbc submodule failure, which is handled in #158. It is large but mechanical; reviewing per-commit or with whitespace hidden is recommended.

🤖 Generated with Claude Code

The rust/ tree was committed without running rustfmt, so the rustfmt
step in Rust CI (`cargo fmt --all -- --check`) fails. Apply
`cargo fmt --all`. The changes are purely formatting — multi-line enum
variants, import ordering, block wrapping — with no semantic effect.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@PierreFouquet

Copy link
Copy Markdown
Contributor Author

This is still failing - so need to investigate why.

@tinpotnick tinpotnick merged commit abac14b into babblevoice:main Jun 9, 2026
0 of 2 checks passed
@PierreFouquet PierreFouquet deleted the chore-cargo-fmt-rust-crate branch June 9, 2026 10:33
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