Skip to content

ci: polish claude code review prompts#5537

Merged
Xuanwo merged 2 commits intomainfrom
xuanwo/valletta
Dec 18, 2025
Merged

ci: polish claude code review prompts#5537
Xuanwo merged 2 commits intomainfrom
xuanwo/valletta

Conversation

@Xuanwo
Copy link
Copy Markdown
Collaborator

@Xuanwo Xuanwo commented Dec 18, 2025

Polish claude code's review behavior.

@github-actions github-actions Bot added the ci Github Action or Test issues label Dec 18, 2025
Comment thread .github/workflows/claude-code-review.yml Outdated
- Your review output SHOULD be concise and clear.
- You SHOULD only highlight P0 and P1 level issues, such as severe bugs, performance degradation, or security concerns.
- You MUST not reiterate detailed changes in your review.
- You MUST not repeat aspects of the PR that are already well done.
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yeah... we really don't need kudos from AI

@Xuanwo Xuanwo merged commit f0ee26a into main Dec 18, 2025
9 checks passed
@Xuanwo Xuanwo deleted the xuanwo/valletta branch December 18, 2025 15:38
@claude
Copy link
Copy Markdown

claude Bot commented Dec 18, 2025

PR Review

Summary: This PR improves the Claude Code review workflow prompts to encourage more focused, actionable reviews.

Assessment: ✅ LGTM

The changes are straightforward and well-reasoned:

  • Emphasizes reviewer attention as a scarce resource
  • Focuses on high-priority issues (P0/P1)
  • Reduces noise by discouraging redundant commentary

No concerns with this change.

wjones127 pushed a commit to wjones127/lance that referenced this pull request Dec 30, 2025
jackye1995 pushed a commit to jackye1995/lance that referenced this pull request Jan 21, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci Github Action or Test issues

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants