jpmschweitzer/settled-reach
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

chore(skills): add merge workflow and heredoc workaround to review-pr

Browse Source 
Document that tea pr merge fails (405) on conflicting branches,
requiring local merge + tea pr close. Also note heredoc hanging
issue with tea comment — use temp file instead.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
This commit is contained in:
jpmschweitzer
2026-02-11 18:22:14 +01:00
co-authored by Claude Opus 4.6
parent cd10834aca
commit 8cc6167bd9
1 changed files with 21 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files
.claude/skills/review-pr/SKILL.md
+21
View File
@@ -157,6 +157,27 @@ REVIEW
Note: `tea pr reject` does not work on your own PRs. Use `tea comment` instead.
**Heredoc bodies hang** in `tea comment`. Write review to a temp file instead:
```bash
# Write review to temp file, then pass via cat
tea comment --login schweitz --repo jpmschweitzer/settled-reach <PR_NUMBER> "$(cat /tmp/review.md)"
```
## Merging approved PRs
`tea pr merge` fails (405) when branches have conflicts with main. Merge
locally instead:
```bash
git fetch --all
git merge origin/<branch> # resolve conflicts if any
git push origin main
tea pr close --login schweitz --repo jpmschweitzer/settled-reach <PR_NUMBER>
```
Gitea does **not** auto-close PRs when you push a local merge — always close
manually with `tea pr close` after pushing.
## Tips from practice
- **Vendor code**: Explicitly note vendor code in the prompt so reviewers focus