fix(build): remove broken cargo verify alias; ci: gate rc branch #212

2026-07-29T16:35:52Z

KofTwentyTwo commented

2026-07-29 16:35:52 +00:00

(Migrated from github.com)

Summary

Remove the unfixable verify alias from .cargo/config.toml — Cargo aliases are a single command's argv and cannot chain subcommands, so cargo verify = ["fmt --all -- --check", "check-all", "test-all"] always errored trying to run a literal subcommand named "fmt --all -- --check".
Delete the matching RustRover run configuration (.idea/runConfigurations/Cargo_Verify.xml) and reword every doc that told readers to run cargo verify to instead run the three checks separately (cargo fmt --all -- --check, cargo check-all, cargo test-all) — CLAUDE.md, README.md, docs/DEVELOPING-IN-RUSTROVER.md, docs/HANDOFF.md, docs/STORY-WORKFLOW.md, docs/BACKLOG.md, docs/ROADMAP.md.
The .githooks/pre-commit hook (tracked, core.hooksPath already points at it) already ran the three checks as separate commands, so it needed no fix.
Add rc to the push/pull_request branch triggers in ci.yml so the dev -> rc -> main release flow is CI-gated on rc too.

Test plan

cargo fmt --all -- --check passes
cargo check-all (clippy --all-targets --workspace -D warnings) passes
cargo test-all passes (all crates, 0 failed)
Local pre-commit hook ran clean on both commits
CI required checks green on this PR (Code Formatting Check, Clippy Linter Audit, Test Suite Execution, Cross-Platform Build Matrix x3)

## Summary - Remove the unfixable `verify` alias from `.cargo/config.toml` — Cargo aliases are a single command's argv and cannot chain subcommands, so `cargo verify = ["fmt --all -- --check", "check-all", "test-all"]` always errored trying to run a literal subcommand named `"fmt --all -- --check"`. - Delete the matching RustRover run configuration (`.idea/runConfigurations/Cargo_Verify.xml`) and reword every doc that told readers to run `cargo verify` to instead run the three checks separately (`cargo fmt --all -- --check`, `cargo check-all`, `cargo test-all`) — CLAUDE.md, README.md, docs/DEVELOPING-IN-RUSTROVER.md, docs/HANDOFF.md, docs/STORY-WORKFLOW.md, docs/BACKLOG.md, docs/ROADMAP.md. - The `.githooks/pre-commit` hook (tracked, `core.hooksPath` already points at it) already ran the three checks as separate commands, so it needed no fix. - Add `rc` to the `push`/`pull_request` branch triggers in `ci.yml` so the dev -> rc -> main release flow is CI-gated on `rc` too. ## Test plan - [x] `cargo fmt --all -- --check` passes - [x] `cargo check-all` (clippy `--all-targets --workspace -D warnings`) passes - [x] `cargo test-all` passes (all crates, 0 failed) - [x] Local pre-commit hook ran clean on both commits - [ ] CI required checks green on this PR (Code Formatting Check, Clippy Linter Audit, Test Suite Execution, Cross-Platform Build Matrix x3)

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: KofTwentyTwo/nuncio#212