docs: use pr status's positional form in the two remaining --pr examples
#4184 gave 'pr status' a positional PR-number arg (--pr <n> still works, just no longer the recommended form). docs/tools/forge.md's examples and the prose section still showed the old --pr form in four places; same one-line hint baked into every agent's own system prompt (hive-agent/prompts/ system.md). Neither file was touched by #4184 itself -- leftover from my own #4182 branch that lost the collision to atlas's #4184 (docs/tools/forge.md and hive-agent/prompts/system.md were the two genuinely non-overlapping bits of that branch, tracked as a follow-up rather than dropped). docs/tools/forge-cli.md is clap-generated and doesn't need regenerating -- prose-only doc changes don't touch the clap tree it's derived from. refs #4182
This commit is contained in:
parent
7b9d8e4c4c
commit
2f792a868c
2 changed files with 14 additions and 13 deletions
|
|
@ -24,7 +24,7 @@ markdown-docs > docs/tools/forge-cli.md`.
|
|||
|
||||
**Kind-namespaced commands (preferred):** hive-forge groups issue/PR operations
|
||||
under `issue` and `pr` parent commands — `hive-forge pr close 42`,
|
||||
`hive-forge issue create --title …`, `hive-forge pr status --pr 42`. The
|
||||
`hive-forge issue create --title …`, `hive-forge pr status 42`. The
|
||||
`pr <verb>` / `issue <verb>` forms validate the number's kind (for example `pr close`
|
||||
refuses an issue number, which the old generic `close` couldn't). Run
|
||||
`hive-forge pr --help` / `hive-forge issue --help` for the full subcommand
|
||||
|
|
@ -49,7 +49,7 @@ hive-forge pr close 42 # close a PR (kind-validated)
|
|||
hive-forge pr reopen 42 # reopen a closed PR (kind-validated)
|
||||
hive-forge issue close 42 # close an issue (kind-validated)
|
||||
hive-forge issue reopen 42 # reopen a closed issue (kind-validated)
|
||||
hive-forge pr status --pr 42 # PR health (mergeable / CI / reviews)
|
||||
hive-forge pr status 42 # PR health (mergeable / CI / reviews)
|
||||
hive-forge issue create --title "..." --body "..."
|
||||
# --- flat aliases below remain valid (hidden) ---
|
||||
hive-forge view 42 # title + body + comments (marks thread read)
|
||||
|
|
@ -105,7 +105,7 @@ hive-forge lint no-reviewer --reviewer argus # PRs where argus specifically isn
|
|||
hive-forge lint stale-branches --days 14 # branches with no recent activity, each with its merge outcome (PR #n merged / closed unmerged / no PR)
|
||||
hive-forge lint assignments # per-assignee open item count
|
||||
hive-forge lint unlabeled --scope type # open issues/PRs with no exclusive type/* label (any scope works, e.g. --scope area)
|
||||
hive-forge pr status --pr 42 # PR health: mergeable, CI, reviews, last comment (exit 0 = ready)
|
||||
hive-forge pr status 42 # PR health: mergeable, CI, reviews, last comment (exit 0 = ready)
|
||||
hive-forge pr status --sha <sha> # CI-only fast path for an explicit commit sha
|
||||
hive-forge pr merge 42 # merge (refuses unless mergeable + CI not red + no changes-requested); deletes head branch
|
||||
hive-forge pr merge 42 --method rebase # rebase-merge instead of a merge commit (no squash option)
|
||||
|
|
@ -145,7 +145,7 @@ distinct output shape — pick by what you need:
|
|||
| One specific comment by its id | `comment-show <id>` | single comment |
|
||||
| The audit trail — closes, label/assignee changes, commit refs, merges (no comment bodies) | `timeline <n>` | event list |
|
||||
| The code change | `pr diff <n>` | unified diff |
|
||||
| Is this PR ready to merge — mergeable + CI + reviews | `pr status --pr <n>` | health view; exit code = merge-readiness |
|
||||
| Is this PR ready to merge — mergeable + CI + reviews | `pr status <n>` | health view; exit code = merge-readiness |
|
||||
| The review verdicts + inline review comments | `pr reviews <n>` | review list |
|
||||
|
||||
Rule of thumb: **`view` first** when you're about to act on a thread (it
|
||||
|
|
@ -189,19 +189,20 @@ commits straight to `main`.
|
|||
|
||||
### `pr status`
|
||||
|
||||
One-stop PR health view (`--pr <n>`): mergeable state, CI checks,
|
||||
requested reviewers + review verdicts, and the last-comment timestamp —
|
||||
the things you need to know whether a PR is ready to merge (CI must pass
|
||||
before merge). `--sha <sha>` is a CI-only fast path for a raw commit.
|
||||
One-stop PR health view (`<n>`, positional — `--pr <n>` also still works):
|
||||
mergeable state, CI checks, requested reviewers + review verdicts, and the
|
||||
last-comment timestamp — the things you need to know whether a PR is ready
|
||||
to merge (CI must pass before merge). `--sha <sha>` is a CI-only fast path
|
||||
for a raw commit.
|
||||
|
||||
A failing/erroring CI context prints its job link. The process exit code
|
||||
is a **merge-readiness** verdict for `--pr` (0 only when CI is green AND
|
||||
the PR is mergeable AND no review requests changes), so it composes:
|
||||
`hive-forge pr status --pr 42 && echo ready`. `--sha` mirrors the CI
|
||||
is a **merge-readiness** verdict for a PR number (0 only when CI is green
|
||||
AND the PR is mergeable AND no review requests changes), so it composes:
|
||||
`hive-forge pr status 42 && echo ready`. `--sha` mirrors the CI
|
||||
verdict alone.
|
||||
|
||||
```
|
||||
hive-forge pr status --pr 42
|
||||
hive-forge pr status 42
|
||||
# PR #42: feat(...): ... # lint:allow: sample CLI output, not a real tag
|
||||
# state: open (mergeable: yes)
|
||||
# CI: e39a87ea3949: ✅ success (1 context(s))
|
||||
|
|
|
|||
Loading…
Reference in a new issue