Mirrors_Framework/awesome-python
1
0
Fork 0
mirror of https://github.com/vinta/awesome-python.git synced 2026-05-31 18:56:15 +08:00
Code Issues Actions 2 Packages Projects Releases Wiki Activity

docs: improve PR review workflow command

Browse Source 
Simplify gh pr list query using --search flag to filter by label,
reorganize rejection criteria with merge conflicts first, and streamline
output format instructions.

🤖 Generated with Claude Code (https://claude.ai/code)
Co-Authored-By: Claude <noreply@anthropic.com>
This commit is contained in:
Vinta Chen
2026-01-08 15:32:52 +08:00
parent 34c29a9b84
commit 4117b23614
1 changed files with 5 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files
.claude/commands/review-pending-prs.md
+5 -5
View File
@@ -10,7 +10,7 @@ Review pending PRs against [CONTRIBUTING.md](../../CONTRIBUTING.md) acceptance c
## Instructions ## Instructions
1. Fetch open PRs (skip reviewed): `gh pr list --repo vinta/awesome-python --limit 20 --json number,title,author,url,labels --jq '[.[] | select(.labels | map(.name) | index("claude reviewed") | not)] | .[:10]'` 1. Fetch 10 open PRs (skip reviewed): `gh pr list --repo vinta/awesome-python --limit 10 --search "-label:\"claude reviewed\"" --json number,title,author,url`
2. For each PR: 2. For each PR:
- Fetch PR details: `gh pr view <number> --repo vinta/awesome-python --json title,body,author,files,url,mergeable,mergeStateStatus` - Fetch PR details: `gh pr view <number> --repo vinta/awesome-python --json title,body,author,files,url,mergeable,mergeStateStatus`
- Fetch PR diff: `gh pr diff <number> --repo vinta/awesome-python` - Fetch PR diff: `gh pr diff <number> --repo vinta/awesome-python`
@@ -24,27 +24,27 @@ Would you like me to:
1. Close the rejected PRs with comments? 1. Close the rejected PRs with comments?
2. Add "claude reviewed" label to the approved PRs? 2. Add "claude reviewed" label to the approved PRs?
3. Add comments to PRs need to be fixed? 3. Do all
``` ```
## Quick Rejection Checks ## Quick Rejection Checks
Check these rules first - if any fail, recommend rejection: Check these rules first - if any fail, recommend rejection:
- PR has merge conflicts
- Add more than one project per PR - Add more than one project per PR
- Duplicate of existing entry - Duplicate of existing entry
- Placed under an inappropriate category - Placed under an inappropriate category
- Project is archived or abandoned (no commits in 12+ months) - Project is archived or abandoned (no commits in 12+ months)
- No documentation or unclear use case - No documentation or unclear use case
- Less than 100 GitHub stars AND not justified as a hidden gem - Less than 100 GitHub stars AND not justified as a hidden gem
- PR has merge conflicts (notify author to resolve)
## Output Format ## Output Format
Provide a simple review: Provide a simple review:
1. **Rejection Check** - table with the 3 rules and PASS/FAIL 1. **Rejection Check** - table with the above rules and PASS/FAIL
2. **Recommendation** - APPROVE for further review, or REJECT with reason 2. **Recommendation** - APPROVE or REJECT
## Closing PRs ## Closing PRs