Skip to content

Add testing feedback guidelines to reduce verbosity #119

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
Cahllagerfeld merged 1 commit into from
Nov 25, 2025
Merged

Add testing feedback guidelines to reduce verbosity #119

Cahllagerfeld merged 1 commit into from
Nov 25, 2025

Conversation

@strickvl
Copy link
Contributor

@strickvl strickvl commented Nov 25, 2025 •
edited
Loading

Summary

  • Adds a dedicated "Testing Feedback Guidelines" section to the Claude Code review prompt
  • Addresses user feedback that test suggestions were sometimes too verbose or not laser-focused on specific PR changes
  • Prioritizes manual testing suggestions (which users found useful) over lengthy code examples

Changes

The new section includes guidance to:

  • Be concise: Avoid verbose test suggestions or lengthy code examples
  • Stay laser-focused: Only suggest tests that directly verify the specific changes in the PR
  • Prioritize manual testing: Brief, actionable suggestions for what to manually verify
  • Limit scope: Mention scenarios in 1-2 sentences rather than full test implementations
  • Skip if unnecessary: Don't force test recommendations when changes are straightforward

@strickvl
ci: add testing feedback guidelines to reduce verbosity
ea7b333 
Adds explicit guidance for Claude Code reviews to keep test
suggestions focused and concise. Addresses user feedback that test
sections were sometimes verbose or not laser-focused on the specific
PR changes.

Key changes:
- Add dedicated "Testing Feedback Guidelines" section
- Prioritize manual testing suggestions over code examples
- Limit scope to 1-2 sentence scenarios, not full implementations
- Allow skipping test suggestions when not necessary
@strickvl strickvl changed the title ci: add testing feedback guidelines to reduce verbosity Add testing feedback guidelines to reduce verbosity Nov 25, 2025
@strickvl strickvl added enhancement New feature or request snack labels Nov 25, 2025
@Cahllagerfeld Cahllagerfeld merged commit 0ffa131 into main Nov 25, 2025
3 checks passed
@Cahllagerfeld Cahllagerfeld deleted the ci/minimise-testing-verbosity branch November 25, 2025 09:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@Cahllagerfeld Cahllagerfeld Cahllagerfeld approved these changes

Assignees

No one assigned

Labels

enhancement New feature or request snack

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@strickvl @Cahllagerfeld