Skip to content

Add a code review skill based on past PR interactions#124148

Merged
stephentoub merged 3 commits intodotnet:mainfrom
stephentoub:codereviewskill
Feb 8, 2026
Merged

Add a code review skill based on past PR interactions#124148
stephentoub merged 3 commits intodotnet:mainfrom
stephentoub:codereviewskill

Conversation

@stephentoub
Copy link
Member

No description provided.

@dotnet-policy-service
Copy link
Contributor

Tagging subscribers to this area: @dotnet/runtime-infrastructure
See info in area-owners.md if you want to be subscribed.

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Adds a new code-review skill document intended to guide reviews in dotnet/runtime using patterns distilled from prior maintainer feedback, and wires that skill into the repo’s Copilot instructions so it’s used before completing work.

Changes:

  • Add a new .github/skills/code-review/SKILL.md with detailed code review guidance across correctness, performance, API design, testing, etc.
  • Update .github/copilot-instructions.md to instruct using the new code-review skill (and performance-benchmark when relevant).

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File Description
.github/skills/code-review/SKILL.md Introduces the new code review skill content and review process guidance.
.github/copilot-instructions.md Requires running the code-review skill (and optionally performance-benchmark) before completing changes.

Copilot AI review requested due to automatic review settings February 8, 2026 19:04
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated no new comments.

@lewing lewing added area-skills Agent Skills and removed area-Infrastructure labels Feb 8, 2026
Copy link
Member

@jkotas jkotas left a comment

Choose a reason for hiding this comment

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

Nice!

@stephentoub stephentoub merged commit 72fe13b into dotnet:main Feb 8, 2026
20 checks passed
@stephentoub stephentoub deleted the codereviewskill branch February 8, 2026 19:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area-skills Agent Skills

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

4 participants