Skip to content

Commit 395a3f5

Browse files
committed
IOS-5364 Add explicit instruction to post review to PR
1 parent 893cc8e commit 395a3f5

File tree

1 file changed

+37
-0
lines changed

1 file changed

+37
-0
lines changed

.github/workflows/claude-code-review-prompt.md

Lines changed: 37 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -104,6 +104,43 @@ Always post a summary at the end:
104104
gh pr comment ${PR_NUMBER} --body "Review complete - see inline comments for details"
105105
```
106106

107+
## CRITICAL: Post Your Review
108+
109+
**YOU MUST POST YOUR REVIEW TO THE PR** - analysis alone is not sufficient.
110+
111+
After completing your review analysis:
112+
113+
1. **For reviews with inline comments**: Post inline comments first using the strategies above, then post a final summary
114+
2. **For reviews without inline comments**: Post your full review text as a single PR comment
115+
116+
**Command**:
117+
```bash
118+
gh pr comment ${PR_NUMBER} --repo ${REPO} --body "YOUR_REVIEW_TEXT_HERE"
119+
```
120+
121+
**Example** (clean approval):
122+
```bash
123+
gh pr comment ${PR_NUMBER} --repo ${REPO} --body "✅ **Approved** - No issues found"
124+
```
125+
126+
**Example** (review with issues):
127+
```bash
128+
gh pr comment ${PR_NUMBER} --repo ${REPO} --body "## Bugs/Issues
129+
130+
**SpaceHubToolbar.swift:109**
131+
The \`attentionDotView\` overlay positioning is incorrect...
132+
133+
---
134+
135+
⚠️ **Minor Issues** - Fix overlay positioning"
136+
```
137+
138+
**Important**:
139+
- Use single quotes to wrap multi-line review text if needed
140+
- Escape special characters appropriately for bash
141+
- Always include the status emoji summary at the end
142+
- The workflow provides ${PR_NUMBER} and ${REPO} variables
143+
107144
## Common Analysis Mistakes to Avoid
108145

109146
### Mistake: Assuming Unused Code After UI Element Removal

0 commit comments

Comments
 (0)