Skip to content

docs: clarify accepted input formats for merge-queue skill#889

Open
ralphbean wants to merge 1 commit into
mainfrom
fix/merge-queue-skill-docs
Open

docs: clarify accepted input formats for merge-queue skill#889
ralphbean wants to merge 1 commit into
mainfrom
fix/merge-queue-skill-docs

Conversation

@ralphbean
Copy link
Copy Markdown
Contributor

Summary

  • Documents the accepted argument formats for the merge-queue skill (PR_NUMBER, PR_URL, or omitted)
  • Explicitly notes that owner/repo#number format is not supported

This prevents failed invocations when an LLM passes the wrong format to the enqueue script.

Test plan

  • Verify the skill doc renders correctly
  • Invoke the merge-queue skill and confirm the agent passes a supported format

🤖 Generated with Claude Code

The skill doc didn't specify which argument formats the script accepts,
leading to failed invocations with owner/repo#number (which gh pr view
does not understand). List the supported formats explicitly.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@github-actions
Copy link
Copy Markdown

fullsend review is working on this — view logs

@github-actions
Copy link
Copy Markdown

Site preview

Preview: https://f720a9d7-site.fullsend-ai.workers.dev

Commit: 9a351ca5e21254c4a31dbabf8521b938bb13cc50

@fullsend-ai-review
Copy link
Copy Markdown

Review: #889

Head SHA: 9a351ca
Timestamp: 2026-05-13T00:00:00Z
Outcome: approve

Summary

Clean documentation-only change that adds an "Accepted input formats" section to the merge-queue skill. The documented formats (PR number, PR URL, omitted) accurately match the three code paths in enqueue-pr.sh, and the note that owner/repo#number is unsupported is correct — gh pr view does not parse that format. No findings across any review dimension.

Findings

No findings.

Footer

Outcome: approve
This review applies to SHA 9a351ca5e21254c4a31dbabf8521b938bb13cc50. Any push to the PR head clears this review and requires a new evaluation.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant