mirror of
https://github.com/openclaw/openclaw.git
synced 2026-09-28 05:54:09 +08:00
docs: prefer native PR workflow without requiring it (#159859)
* docs: keep unfinished handoff pushes separate from merge preparation Co-authored-by: VACInc <3279061+VACInc@users.noreply.github.com> * docs: reduce handoff guidance to the scope distinction Co-authored-by: VACInc <3279061+VACInc@users.noreply.github.com> * docs: simplify unfinished-work handoff clarification Co-authored-by: VACInc <3279061+VACInc@users.noreply.github.com> * docs: scope workflow requirements to the requested operation Do not treat one failed route as a task-wide blocker when another supported, authorized route exists. This is not limited to unfinished drafts or handoffs. Co-authored-by: VACInc <3279061+VACInc@users.noreply.github.com> * docs: prefer native workflow without requiring it exclusively Co-authored-by: VACInc <3279061+VACInc@users.noreply.github.com> * docs: keep maintainer workflow guidance discretionary Co-authored-by: VACInc <3279061+VACInc@users.noreply.github.com> --------- Co-authored-by: roboclaw-bot <309084314+roboclaw-bot@users.noreply.github.com> Co-authored-by: VACInc <3279061+VACInc@users.noreply.github.com>
This commit is contained in:
co-authored by
VACInc
roboclaw-bot
parent
9726512367
commit
74b88a509d
@@ -30,9 +30,10 @@ Use the current template and a real body file. Preserve human credit and keep
|
||||
branches editable by maintainers when safe. For a fork, consider GitHub's
|
||||
Actions/secrets warning before enabling edits.
|
||||
|
||||
Create as draft, wait for non-null `mergeable`, then mark ready. Confirm CI
|
||||
attached to the pushed head. A merge-ref startup failure cannot be rerun; the
|
||||
hourly PR CI sweeper can re-fire it, or use an authorized close/reopen after
|
||||
Create as draft; when merge readiness is requested, wait for non-null `mergeable`,
|
||||
then mark ready and confirm CI attached to the pushed head. A merge-ref startup
|
||||
failure cannot be rerun; the hourly PR CI sweeper can re-fire it, or use an
|
||||
authorized close/reopen after
|
||||
verifying the missing attachment. Do not rebase merely because main advanced.
|
||||
Refresh only for a conflict, failing guard, explicit request, or material stale
|
||||
base risk. An explicitly requested landing of one's own draft includes marking
|
||||
@@ -69,7 +70,7 @@ Do not disclose private desktop content, identifiers, model routes, or secrets.
|
||||
|
||||
## Review, prepare, merge
|
||||
|
||||
For main-targeted PRs, use only the native sequence:
|
||||
For main-targeted PRs, prefer the native sequence; adapt as needed.
|
||||
|
||||
```bash
|
||||
scripts/pr review-init <pr>
|
||||
|
||||
Reference in New Issue
Block a user