Conversation
There was a problem hiding this comment.
Pull request overview
This PR introduces a Claude “review” skill intended to standardize PR reviews for the ClickHouse .NET driver repo, and updates the contributor/agent guidance to point reviewers to that skill.
Changes:
- Updates PR review guidance in
AGENTS.mdto reference using a review skill. - Adds a new
.claude/skills/review/SKILL.mdfile defining a structured PR review process and expected output format.
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 4 comments.
| File | Description |
|---|---|
| AGENTS.md | Replaces the prior reference to .github/copilot-instructions.md with a short directive to use the review skill. |
| .claude/skills/review/SKILL.md | Adds the new “review” skill definition and detailed review/output instructions. |
| ## PR Review Guidelines | ||
|
|
||
| Use the guidelines in .github/copilot-instructions.md | ||
| Use review skill. |
There was a problem hiding this comment.
The instruction "Use review skill." is too vague for someone reading AGENTS.md without prior context. Consider linking to the actual skill file (e.g., .claude/skills/review/SKILL.md) and/or briefly stating how to invoke the review skill, otherwise this section no longer provides actionable guidance compared to the previous reference to .github/copilot-instructions.md.
| Use review skill. | |
| Use the `review` skill defined in [.claude/skills/review/SKILL.md](.claude/skills/review/SKILL.md) to perform structured PR reviews (see that file for invocation details and prompts). |
Codecov Report✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! |
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
No description provided.