Skip to content

Commit 8479b9d

Browse files
committed
guide: add rule #44 — resolve CHANGES_REQUESTED before opening new PRs in same repo
1 parent 91c2e2f commit 8479b9d

1 file changed

Lines changed: 2 additions & 0 deletions

File tree

guide.md

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -193,3 +193,5 @@ PR 被维护者 supersede(关闭后自己重写)是最大的时间浪费。
193193
43. **Read comment history before closing any PR**stagehand#2026 教训:pirate 明确说 "please stop closing this, I'm leaving it open as a placeholder",但 stale-close 逻辑直接关了 PR 没检查最近评论。被 maintainer 当面纠正后道歉。原则:**关闭任何 PR 前(包括自己的),先读最近 5 条 comment**。如果有 maintainer 说 "keep open" / "placeholder" / "don't close" → 绝对不碰。流程:(1) `gh pr view OWNER/REPO#N --json comments --jq '.comments[-5:] | .[].body'` 快速扫最近评论 (2) 搜索 "keep open", "placeholder", "don't close", "leave this" 等关键词 (3) 有 maintainer 明确要求保留 → 跳过,不关。判断方法:如果你上次关了被 maintainer 要求 reopen → 那个 PR 永久 hands-off。推广:自动化/批量操作(stale close、bulk rebase、批量 ping)都应该先检查每个 target 的最近 context,不要盲目执行
194194

195195
42. **Cross-verify PR coverage — gogetajob sync is not exhaustive**qwen-code#4474 教训:3 个 PR 有 CHANGES_REQUESTED 连续多天未处理,因为 qwen-code 不在 gogetajob 关注列表里,sync 完全看不到。followup 只靠 gogetajob sync 一个数据源 = 单点故障。修法:每轮 followup 在 gogetajob sync 之后,用 `gh search prs --author=kagura-agent --state=open` 查全量 open PR 交叉验证。发现不在 sync 里的 → 立即 `gogetajob scan` 加入关注列表。原则:**提了 PR 的 repo 必须在关注列表里**,否则后续 followup 全盲。
196+
197+
44. **Resolve CHANGES_REQUESTED before opening new PRs in same repo** — qwen-code#4456/#4474/#4461 教训:同 repo 3+ PRs 同时处于 CHANGES_REQUESTED 状态,每轮 followup 列出来但多天未 push 修复(认知过载 + 维护者印象差——看到你不断开新 PR 但旧 PR 的 review feedback 一直不处理)。原则:**同一个 repo 有未处理的 CHANGES_REQUESTED → 先修再开新 PR**。流程:(1) 开工前 `gh search prs --author=kagura-agent --state=open --repo OWNER/REPO --review=changes_requested` 检查该 repo 是否有待处理 review (2) 有 → 本轮优先 push fix,不开新 PR (3) 修完后 re-request review (4) 只有该 repo 所有 CHANGES_REQUESTED 都处理完(push 了新 commit 或确认无需改动并回复 reviewer),才能开新 PR。判断方法:如果同 repo ≥2 个 PR 同时 CHANGES_REQUESTED 且都超过 48h 未处理,这已经是维护者信任危机——每多拖一天,后续 PR 被认真 review 的概率都在下降。推广:维护者视角看你的 PR 列表是你的「信用报告」——一堆未回应的 review = 不靠谱的贡献者。清理存量 review debt 的 ROI 远高于开新 PR

0 commit comments

Comments
 (0)