From 8de85729cc8e061a6bb71a54d818a6ba5c883c88 Mon Sep 17 00:00:00 2001 From: atlas Date: Mon, 21 Sep 2026 20:47:54 +0200 Subject: [PATCH] hive-forge: make ci-rerun --pr help text match the refusal it prints The doc comment still described the old (never-shipped) behavior -- exactly the lie this whole PR exists to remove, just in --help instead of the module doc or forge.md. Say what the flag actually does: refuse, because a workflow_dispatch run writes no commit status and can't clear a red (pull_request) check. Regenerated docs/tools/forge-cli.md (derived from clap help via 'hive-forge markdown-docs'). --- docs/tools/forge-cli.md | 2 +- hive-forge/src/verbs/ci_rerun.rs | 5 +++-- 2 files changed, 4 insertions(+), 3 deletions(-) diff --git a/docs/tools/forge-cli.md b/docs/tools/forge-cli.md index 7e544401..13b21a3f 100644 --- a/docs/tools/forge-cli.md +++ b/docs/tools/forge-cli.md @@ -1370,7 +1370,7 @@ Re-run CI without an empty commit. Pass one of `--pr `, `--run `, or `--br ###### **Options:** -* `--pr ` — Re-run CI for this PR (its head branch). Mutually exclusive with `--run` / `--branch` +* `--pr ` — Doesn't work: a `workflow_dispatch` run writes no commit status, so it can't clear a red `(pull_request)` check — re-run from the web UI instead. Mutually exclusive with `--run` / `--branch` * `--run ` — Re-run the same workflow on the same branch this run used. The run number is the `runs/` in the run-page URL. Mutually exclusive with `--pr` / `--branch` * `--branch ` — Re-run `--workflow` on this branch. Mutually exclusive with `--pr` / `--run` * `--workflow ` — Workflow file to run (default `ci.yml`). Ignored for `--run`, which uses the run's own workflow diff --git a/hive-forge/src/verbs/ci_rerun.rs b/hive-forge/src/verbs/ci_rerun.rs index 7964bb05..8f666276 100644 --- a/hive-forge/src/verbs/ci_rerun.rs +++ b/hive-forge/src/verbs/ci_rerun.rs @@ -34,8 +34,9 @@ use crate::client::{Client, index}; #[derive(ClapArgs)] pub struct Args { - /// Re-run CI for this PR (its head branch). Mutually exclusive with - /// `--run` / `--branch`. + /// Doesn't work: a `workflow_dispatch` run writes no commit status, so + /// it can't clear a red `(pull_request)` check — re-run from the web UI + /// instead. Mutually exclusive with `--run` / `--branch`. #[arg(long, conflicts_with_all = ["run", "branch"])] pr: Option, /// Re-run the same workflow on the same branch this run used. The run