From 36b67fcda3e9e4fdb27bd3bd00133e0d8769ffa6 Mon Sep 17 00:00:00 2001 From: atlas Date: Wed, 2 Sep 2026 14:54:48 +0200 Subject: [PATCH] docs/ci: the local-checks block named a command that does not exist MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit `docs/scheduler/ci.md` told the reader to run `nix develop -c treefmt`. treefmt is not on the devshell PATH — `nix develop -c which treefmt` exits 1 while `which cargo` resolves, so the check discriminates and the documented command has never worked from that shell. The same line enumerated the formatter set as "rustfmt + nixfmt + taplo", which was already wrong before this branch: keep-sorted has been enabled all along, and the commit before this adds prettier. Replaced the list with a pointer to nix/treefmt.nix so it can't drift out of sync again. Also drops "+ rustfmt" from the nix-flake-check row: rustfmt is one of treefmt's formatters, not a separate thing that runs alongside it. --- docs/scheduler/ci.md | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/docs/scheduler/ci.md b/docs/scheduler/ci.md index 5ea53aaf..6d1ced37 100644 --- a/docs/scheduler/ci.md +++ b/docs/scheduler/ci.md @@ -36,11 +36,11 @@ defined in [`.forgejo/workflows/ci.yml`](../../.forgejo/workflows/ci.yml). All three are required checks (forge branch protection) — a hit on any of them blocks merge. -| Job | What it runs | -| ---------------------- | ----------------------------------------------------------------------------------------- | -| **nix flake check** | treefmt + rustfmt formatting, `cargo clippy -D warnings`, `cargo test`, module evaluation | -| **tracker-tag lint** | flags `#NNN` issue tags in source and comments (`scripts/check-issue-refs.sh`) | -| **comment-block lint** | flags contiguous comment blocks over 30 lines (`scripts/check-comment-blocks.sh`) | +| Job | What it runs | +| ---------------------- | --------------------------------------------------------------------------------- | +| **nix flake check** | treefmt formatting, `cargo clippy -D warnings`, `cargo test`, module evaluation | +| **tracker-tag lint** | flags `#NNN` issue tags in source and comments (`scripts/check-issue-refs.sh`) | +| **comment-block lint** | flags contiguous comment blocks over 30 lines (`scripts/check-comment-blocks.sh`) | `hive-forge ci-rerun --pr N` dispatches a `workflow_dispatch` retrigger without an empty commit. @@ -69,7 +69,7 @@ wastes a remote-builder slot. Use the devshell equivalents instead: ```sh nix develop -c cargo clippy --all-targets -- -D warnings nix develop -c cargo test -nix develop -c treefmt # same as nix fmt; treefmt covers rustfmt + nixfmt + taplo +nix fmt # authoritative; the formatter set is nix/treefmt.nix sh scripts/check-issue-refs.sh # tracker-tag lint sh scripts/check-comment-blocks.sh # comment-block lint ```