Skip to content

Update code review skill to push model to not repeat same feedback if previously resolved#84348

Open
akhera99 wants to merge 2 commits into
dotnet:mainfrom
akhera99:update_code_review_skill
Open

Update code review skill to push model to not repeat same feedback if previously resolved#84348
akhera99 wants to merge 2 commits into
dotnet:mainfrom
akhera99:update_code_review_skill

Conversation

@akhera99

@akhera99 akhera99 commented Jun 30, 2026

Copy link
Copy Markdown
Member
Microsoft Reviewers: Open in CodeFlow
Copilot AI review requested due to automatic review settings June 30, 2026 21:51
@akhera99 akhera99 requested a review from a team as a code owner June 30, 2026 21:51

Copilot AI left a comment

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.

Pull request overview

Updates the .github/skills/code-review guidance to more strongly require reviewers to consult prior review iterations and avoid duplicating feedback across iterations.

Changes:

  • Expands “Existing review comments” guidance to include resolved/outdated/addressed comments and discourage re-posting previously raised feedback.
  • Extends the “Don’t pile on” guidance to also discourage repeating the same feedback across review iterations.
Show a summary per file
File Description
.github/skills/code-review/SKILL.md Strengthens review-process guidance around reading prior comments and avoiding duplicate feedback across iterations.

Copilot's findings

  • Files reviewed: 1/1 changed files
  • Comments generated: 2
Comment thread .github/skills/code-review/SKILL.md Outdated
Comment thread .github/skills/code-review/SKILL.md Outdated
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

4 participants