Skip to content

Revert "Added C# MCP Server development skills"#74

Merged
adityamandaleeka merged 1 commit into
mainfrom
revert-36-lerich/csharp-mcp-skills
Feb 21, 2026
Merged

Revert "Added C# MCP Server development skills"#74
adityamandaleeka merged 1 commit into
mainfrom
revert-36-lerich/csharp-mcp-skills

Conversation

@leslierichardson95

Copy link
Copy Markdown
Contributor

Reverts #36

Copilot AI review requested due to automatic review settings February 20, 2026 23:37

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 pull request reverts PR #36, which added C# MCP Server development skills to the repository. The revert removes an entire skills collection that was added in the wrong directory structure.

Changes:

  • Removes the complete skills/mcp-csharp-skills/ directory and all its contents
  • Deletes 5 skill modules (mcp-csharp, mcp-csharp-create, mcp-csharp-debug, mcp-csharp-test, mcp-csharp-publish)
  • Removes all associated documentation, reference materials, and license files

Reviewed changes

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

Show a summary per file
File Description
skills/mcp-csharp-skills/README.md Removes the overview documentation for the MCP C# skills collection
skills/mcp-csharp-skills/mcp-csharp/ Removes the main orchestrator skill for C# MCP server development
skills/mcp-csharp-skills/mcp-csharp-create/ Removes the project creation and scaffolding skill
skills/mcp-csharp-skills/mcp-csharp-debug/ Removes the debugging and local testing skill
skills/mcp-csharp-skills/mcp-csharp-test/ Removes the testing and evaluation skill
skills/mcp-csharp-skills/mcp-csharp-publish/ Removes the publishing and deployment skill

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@jeffschwMSFT jeffschwMSFT reopened this Feb 21, 2026
@jeffschwMSFT

Copy link
Copy Markdown
Member

I am sure there was a conversation somewhere, but we should consider reverting as these tools are not in the right place. has the skill validator been run? I would like to see the output prior to merging.

@ViktorHofer

Copy link
Copy Markdown
Member

Yes we should revert.

@adityamandaleeka adityamandaleeka merged commit 86dc027 into main Feb 21, 2026
4 of 5 checks passed
@adityamandaleeka adityamandaleeka deleted the revert-36-lerich/csharp-mcp-skills branch February 21, 2026 16:26
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.

5 participants