Browse Source

docs: state the dependent check's contract as zero-versus-non-zero

Verified against gh: --base filters server-side and exactly, so any
open dependent guarantees a non-zero print, but --jq length counts the
fetched page and gh's default --limit is 30 (--limit 5 prints 5 when
the true count is 20). The printed number is min(actual, limit), so the
skill no longer claims it as the auto-close count; only 0 clears a
delete.
Tianyi Cui 2 months ago
parent
commit
89e7005cfd
1 changed files with 1 additions and 1 deletions
  1. 1 1
      .agents/skills/dsh-merging-stacked-prs/SKILL.md

+ 1 - 1
.agents/skills/dsh-merging-stacked-prs/SKILL.md

@@ -38,7 +38,7 @@ Before deleting a branch, ask GitHub directly whether any open PR still bases on
 gh pr list --state open --base <branch> --json number --jq length
 ```
 
-Anything other than `0` means deleting `<branch>` auto-closes that many open PRs — do not delete it. The `--base` filter is applied server-side, so the answer is exact no matter how many PRs are open, and the output is a single number. Default to merging without `--delete-branch` throughout, and do the deletions as a separate final pass once every branch you're about to delete reports `0`.
+Anything other than `0` means open PRs still base on `<branch>` and deleting it would auto-close them — do not delete it. The `--base` filter is applied server-side, so zero-versus-non-zero is exact no matter how many PRs are open; the printed number itself saturates at `gh`'s `--limit` (default 30), which never matters here because only `0` clears a delete. Default to merging without `--delete-branch` throughout, and do the deletions as a separate final pass once every branch you're about to delete reports `0`.
 
 ## Longer chains