Merge pull request 'feat: look at approvals or issues next if the PR queue is empty (Closes #572)' (#573) from feat/issue-572-empty-queue-fallback into master
This commit was merged in pull request #573.
This commit is contained in:
@@ -16,7 +16,7 @@ Rules:
|
|||||||
reviewing a second PR after a terminal mutation in this run.
|
reviewing a second PR after a terminal mutation in this run.
|
||||||
- After REQUEST_CHANGES: stop. After APPROVED: merge only this PR and only if
|
- After REQUEST_CHANGES: stop. After APPROVED: merge only this PR and only if
|
||||||
operator explicitly authorized merge for this PR in this run.
|
operator explicitly authorized merge for this PR in this run.
|
||||||
- Report Next suggested PR without continuing to it.
|
- Report Next suggested PR without continuing to it. If the PR queue is empty, look at approvals, or issues next.
|
||||||
|
|
||||||
Operator PR list (optional): <pr numbers or "oldest eligible from inventory">
|
Operator PR list (optional): <pr numbers or "oldest eligible from inventory">
|
||||||
Merge authorized for selected PR in this run: <true|false>
|
Merge authorized for selected PR in this run: <true|false>
|
||||||
|
|||||||
@@ -28,7 +28,8 @@ Repo name disambiguation (Gitea-Tools blind review hardening):
|
|||||||
`pr_inventory_trust_gate.status`, trust-gate reasons, corroboration,
|
`pr_inventory_trust_gate.status`, trust-gate reasons, corroboration,
|
||||||
remote/owner/repo/state filter, and the inventory MCP profile. A recent merge
|
remote/owner/repo/state filter, and the inventory MCP profile. A recent merge
|
||||||
commit is not valid corroboration. Author-bound sessions must not present
|
commit is not valid corroboration. Author-bound sessions must not present
|
||||||
reviewer queue inventory as a reviewer decision.
|
reviewer queue inventory as a reviewer decision. If the PR queue is empty,
|
||||||
|
look at approvals, or issues next.
|
||||||
|
|
||||||
Load the canonical workflow first:
|
Load the canonical workflow first:
|
||||||
`skills/llm-project-workflow/workflows/review-merge-pr.md` (task mode: review-merge-pr).
|
`skills/llm-project-workflow/workflows/review-merge-pr.md` (task mode: review-merge-pr).
|
||||||
|
|||||||
@@ -55,6 +55,10 @@ claim. Select exactly one PR according to project queue ordering rules
|
|||||||
PRs only with live per-PR proof. No multi-PR validation and no batch report
|
PRs only with live per-PR proof. No multi-PR validation and no batch report
|
||||||
may substitute for per-PR proof.
|
may substitute for per-PR proof.
|
||||||
|
|
||||||
|
If the open PR queue is empty:
|
||||||
|
* First, look at **Approvals** next: check if there are open PRs with pending/completed approvals requiring attention or merge.
|
||||||
|
* Next, look at **Issues** next: check if there are unresolved open issues requiring action/fixes.
|
||||||
|
|
||||||
## 4. Terminal mutation chain
|
## 4. Terminal mutation chain
|
||||||
|
|
||||||
`pr_queue_cleanup.resolve_cleanup_run_state` is the authority:
|
`pr_queue_cleanup.resolve_cleanup_run_state` is the authority:
|
||||||
|
|||||||
@@ -272,6 +272,10 @@ If queue ordering cannot be proven, stop and produce a recovery handoff.
|
|||||||
|
|
||||||
## 10. Select the next actionable PR using project rules
|
## 10. Select the next actionable PR using project rules
|
||||||
|
|
||||||
|
If the open PR queue is empty:
|
||||||
|
* First, look at approvals next to see if there are pending approvals or approved PRs that need attention/merge.
|
||||||
|
* Next, look at issues next to see if there are open issues requiring action/fixes.
|
||||||
|
|
||||||
Do not review your own PR.
|
Do not review your own PR.
|
||||||
|
|
||||||
Do not review stale, draft, blocked, duplicate, already-owned, dependency-blocked, already-landed, already-requested-changes work unless the rules explicitly allow it.
|
Do not review stale, draft, blocked, duplicate, already-owned, dependency-blocked, already-landed, already-requested-changes work unless the rules explicitly allow it.
|
||||||
|
|||||||
Reference in New Issue
Block a user