docs: point PR landing at maintainer workflow (#93494) · ...
fuller-stack
·
2026-06-16
·
via Recent Commits to openclaw:main
| Original file line number | Diff line number | Diff line change |
|---|
@@ -284,7 +284,7 @@ gh search issues --repo openclaw/openclaw --match title,body --limit 50 \
|
284 | 284 | - If bot review conversations exist on your PR, address them and resolve them yourself once fixed. |
285 | 285 | - Leave a review conversation unresolved only when reviewer or maintainer judgment is still needed. |
286 | 286 | - Before landing any PR with non-trivial code changes, run `$autoreview` until no accepted/actionable findings remain, unless equivalent manual review already covered it, the change is trivial/docs-only, or the user opts out. |
287 | | -- When landing or merging any PR, follow the global `/landpr` process. |
| 287 | +- When an agent is landing or merging a PR targeting `main`, use only the repo-native `scripts/pr` wrapper: run `scripts/pr review-init <PR>`, follow its emitted checkout/guard guidance, initialize and complete review artifacts with `scripts/pr review-artifacts-init <PR>`, validate them with `scripts/pr review-validate-artifacts <PR>`, then run `scripts/pr prepare-run <PR>` and `scripts/pr merge-run <PR>`. |
288 | 288 | - Use `scripts/committer "<msg>" <file...>` for scoped commits instead of manual `git add` and `git commit`. |
289 | 289 | - Keep commit messages concise and action-oriented. |
290 | 290 | - Group related changes; avoid bundling unrelated refactors. |
|
| Original file line number | Diff line number | Diff line change |
|---|
@@ -172,7 +172,7 @@ Skills own workflows; root owns hard policy and routing.
|
172 | 172 | - PR artifacts/screenshots: attach to PR/comment/external artifact store. Never push screenshots, videos, proof images, or proof assets to OpenClaw or any product repo branch, including temp artifact branches. Use Crabbox artifact publishing plus the manifest URL. Do not commit `.github/pr-assets`. |
173 | 173 | - CI polling: exact SHA, relevant checks only, minimal fields. Skip routine noise (`Auto response`, `Labeler`, docs agents, performance/stale). Logs only after failure/completion or concrete need. |
174 | 174 | - OpenClaw write-access maintainers may skip `Real behavior proof` when local tests or Crabbox verified behavior; record proof in PR verification. |
175 | | -- `/landpr`: use `~/.codex/prompts/landpr.md`; do not idle on `auto-response` or `check-docs`. |
| 175 | +- Agent PR landing to `main`: use only the repo-native `scripts/pr` wrapper: run `scripts/pr review-init <PR>`, follow its emitted checkout/guard guidance, initialize and complete review artifacts with `scripts/pr review-artifacts-init <PR>`, validate them with `scripts/pr review-validate-artifacts <PR>`, then run `scripts/pr prepare-run <PR>` and `scripts/pr merge-run <PR>`; do not idle on `auto-response` or `check-docs`. |
176 | 176 | |
177 | 177 | ## Code |
178 | 178 | |
|
此内容由惯性聚合(RSS阅读器)自动聚合整理，仅供阅读参考。 原文来自 — 版权归原作者所有。
