Skip to content

Add azure-sdk-mgmt-pr-review Copilot skill to repository#56171

Merged
live1206 merged 4 commits intomainfrom
copilot/add-azure-sdk-pr-review-skill
Feb 12, 2026
Merged

Add azure-sdk-mgmt-pr-review Copilot skill to repository#56171
live1206 merged 4 commits intomainfrom
copilot/add-azure-sdk-pr-review-skill

Conversation

@live1206
Copy link
Copy Markdown
Member

@live1206 live1206 commented Feb 11, 2026

Description

Adds the azure-sdk-mgmt-pr-review Copilot skill to .github/skills/ so it is available to all contributors in this repository.

This skill is drafted based on the .NET Mgmt. SDK API Review Guide.

It provides guidelines for reviewing Azure SDK for .NET management-plane library pull requests, including:

  • Naming convention checks (suffixes, resource naming, property naming)
  • Operation body parameter naming rules
  • Type formatting expectations (ResourceIdentifier, ETag, AzureLocation, etc.)
  • Acronym casing rules
  • Enum naming conventions
  • CheckNameAvailability operation patterns

Move the azure-sdk-pr-review Copilot skill from local user skills to the
repository .github/skills/ directory so it is available to all contributors.
- Rename skill to azure-sdk-mgmt-pr-review to reflect management-plane scope
- Clarify packages are under Azure.ResourceManager.*
- Reference TypeSpec (client.tsp, tspconfig.yaml) instead of only autorest.md
- Update output format to add comments directly to PR via GitHub MCP tools
@live1206 live1206 force-pushed the copilot/add-azure-sdk-pr-review-skill branch from 8778083 to 7559ccf Compare February 11, 2026 03:25
@live1206 live1206 marked this pull request as ready for review February 11, 2026 03:26
Copilot AI review requested due to automatic review settings February 11, 2026 03:26
Copy link
Copy Markdown
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 GitHub Copilot skill under .github/skills/ to guide reviews of Azure SDK for .NET management-plane (Azure.ResourceManager*) pull requests, based on internal API review guidance.

Changes:

  • Added a new Copilot skill definition: azure-sdk-mgmt-pr-review.
  • Documented a management-plane PR review checklist (naming, parameters, acronyms, enums, type expectations, and common API patterns).

- Include Azure.ResourceManager (base package) in scope
- Clarify Type Formatting table for .NET API surface (Guid not uuid)
- Add TypeSpec note for UUID mapping
@live1206 live1206 changed the title Add azure-sdk-pr-review Copilot skill to repository Add azure-sdk-mgmt-pr-review Copilot skill to repository Feb 11, 2026
@live1206 live1206 merged commit 633612f into main Feb 12, 2026
12 checks passed
@live1206 live1206 deleted the copilot/add-azure-sdk-pr-review-skill branch February 12, 2026 02:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants