Skip to content

Commit 0ffa131

Browse files
authored
ci: add testing feedback guidelines to reduce verbosity (#119)
1 parent be5162f commit 0ffa131

File tree

1 file changed

+7
-1
lines changed

1 file changed

+7
-1
lines changed

.github/workflows/claude-code-review.yml

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -111,9 +111,15 @@ jobs:
111111
6) Constructive Suggestions
112112
- Alternative approaches that might be cleaner
113113
- Opportunities to reduce complexity
114-
- Missing test coverage for critical component behavior
115114
- Opportunities to improve accessibility
116115
116+
Testing Feedback Guidelines:
117+
- **Be concise**: Avoid verbose test suggestions or lengthy code examples.
118+
- **Stay laser-focused**: Only suggest tests that directly verify the specific changes in this PR—not tangential functionality.
119+
- **Prioritize manual testing**: Brief, actionable suggestions for what to manually verify are most useful (e.g., "Test keyboard navigation on the new dropdown").
120+
- **Limit scope**: If tests are needed, mention the scenario to cover in 1-2 sentences rather than writing full test implementations.
121+
- **Skip if unnecessary**: Not every PR needs test suggestions—if the changes are straightforward or already covered, don't force test recommendations.
122+
117123
Documentation & Storybook Review:
118124
- Determine if component behavior, props, or APIs have changed.
119125
- Heuristics (run): gh pr diff ${{ github.event.pull_request.number }} --name-only

0 commit comments

Comments
 (0)