code-review

Review dotnet/runtime code changes for correctness, performance, and conventions.

18.2k|5.5k|Updated Sep 24, 2019

Search Tags:#maintainability #quality-assurance #coding-standards #code-review #pull-request #review-guidelines #dotnet-runtime

Ready to run

npx skills add https://github.com/dotnet/runtime --skill code-review-dotnet-2

💬 Prompt for ChatGPT / Claude Chat▼

Please help me install this Agent Skill.
Skill: code-review
Source: https://github.com/dotnet/runtime/tree/main/.github/skills/code-review
Command: npx skills add https://github.com/dotnet/runtime --skill code-review-dotnet-2

SYSTEM DOCUMENTATION & REQUIREMENTS

What problem does it solve?

Streamline and standardize the evaluation of dotnet/runtime code changes by providing a consistent, thorough, and maintainable review process that catches defects early.

Core Features & Use Cases

Holistic PR assessment focusing on correctness, performance, safety, and conformance to established patterns.
Clear, actionable feedback guiding contributors to fix issues with explicit references to project conventions.
Use Case: A maintainer reviews a PR before merging to ensure changes align with runtime coding standards and design goals.

Quick Start

Provide a structured PR review by following the holistic assessment steps outlined above.

Frequently Asked Questions about code-review

High-intent search queries and answers about installing and using this skill.

FAQPage Schema

How do I review a pull request for dotnet runtime code changes?▼

To review a pull request for dotnet runtime code changes, apply a holistic assessment focusing on correctness, performance, safety, and conformance to established project conventions to catch defects early.

What is the best way to check if my code aligns with dotnet runtime coding standards?▼

Checking dotnet runtime coding standards involves evaluating changes against established project conventions and guidelines, ensuring your pull request meets maintainability, safety, and architectural requirements.

Does this code review process work for pre-merge checks in the dotnet runtime repository?▼

Yes, this code review process works for pre-merge checks by applying during pull request reviews to catch style, safety, performance, and architectural issues across dotnet runtime repository changes.

How do I provide actionable feedback for dotnet runtime pull requests?▼

You provide actionable feedback for dotnet runtime pull requests by guiding contributors with explicit references to project conventions, ensuring evaluations remain consistent, thorough, and maintainable.

What are the limitations of automating dotnet runtime code critique?▼

Automating dotnet runtime code critique requires alignment with established conventions and guidelines, meaning it focuses strictly on defined checklists for style, safety, performance, and architectural correctness rather than subjective design.