Skip to content

Colocate skills and add project file skill - #12964

Merged
gewarren merged 1 commit into
dotnet:mainfrom
gewarren:project-skill
Aug 11, 2026
Merged

Colocate skills and add project file skill#12964
gewarren merged 1 commit into
dotnet:mainfrom
gewarren:project-skill

Conversation

@gewarren

Copy link
Copy Markdown
Collaborator

No description provided.

Copilot AI lite review requested due to automatic review settings August 10, 2026 21:54
@gewarren
gewarren requested a review from a team as a code owner August 10, 2026 21:54

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

This PR adds two GitHub Copilot Agent skill definitions under .github/skills/ to guide consistent maintenance of .NET API reference snippets—one focused on consolidating snippet project files and one focused on modernizing C# snippets while preserving behavior and intent.

Changes:

  • Adds a snippet-project-files skill with guidance for ensuring each snippet directory builds as a single project.
  • Adds a C# snippet modernization skill describing preferred syntax updates, behavioral safeguards, and validation steps.

Reviewed changes

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

File Description
.github/skills/snippet-project-files/SKILL.md Introduces guidance for consolidating/maintaining snippet project files so snippet directories build reliably.
.github/skills/csharp-snippet-modernization/SKILL.md Introduces modernization guidance for C# snippets, emphasizing behavior preservation and validation.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@gewarren
gewarren enabled auto-merge (squash) August 10, 2026 22:00
@wadepickett

Copy link
Copy Markdown
Contributor

@gewarren, this file was renamed:
.github/skills/csharp-snippet-modernization/SKILL.md

The name in the file needs to change too from name: dotnet-snippet-modernization to
name: csharp-snippet-modernization?

@wadepickett wadepickett 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.

@gewarren Looks good. Approved. See my note on the internal name frontmater matching the file name.

@gewarren
gewarren merged commit 54ad241 into dotnet:main Aug 11, 2026
9 checks passed
@wadepickett

Copy link
Copy Markdown
Contributor

Whoops, sorry, I forgot wee are using auto-merge on approval here and I simultaniously made a suggestion and approved which resulted of course in automerge as it should. @gewarren: See my note on the internal name frontmater matching the file name.

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