Skip to content

Conversation

@hallipr
Copy link
Member

@hallipr hallipr commented Aug 21, 2025

What does this PR do?

Temporarily adds users to CODEOWNERS to fill in for the missing @Azure/azure-mcp team
Fix links in README.md and update to point to the new repo

Pre-merge Checklist

  • Required for All PRs
    • Read contribution guidelines
    • PR title clearly describes the change
    • Commit history is clean with descriptive messages (cleanup guide)
    • Added comprehensive tests for new/modified functionality
    • Updated CHANGELOG.md for product changes (features, bug fixes, UI/UX, updated dependencies)
    • Spelling check passes: .\eng\common\spelling\Invoke-Cspell.ps1
  • For MCP tool changes:
    • One tool per PR: This PR adds or modifies only one MCP tool for faster review cycles
    • Updated README.md documentation
    • Updated command list in /docs/azmcp-commands.md
    • Updated test prompts in /docs/e2eTestPrompts.md
    • For new or modified tool descriptions, ran ToolDescriptionEvaluator and obtained a score of 0.4 or more and a top 3 ranking for all related test prompts
  • 👉 For Community (non-Azure team member) PRs:
    • Security review: Reviewed code for security vulnerabilities, malicious code, or suspicious activities before running tests (crypto mining, spam, data exfiltration, etc.)
    • Manual tests run: added comment /azp run azure - mcp to run Live Test Pipeline

Copilot AI review requested due to automatic review settings August 21, 2025 23:50
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

This PR updates repository references from the old Azure/azure-mcp repository to the new microsoft/mcp repository across documentation and configuration files. Additionally, it temporarily adds individual users to CODEOWNERS to fill in for the missing @Azure/azure-mcp team.

  • Updates all GitHub repository links from Azure/azure-mcp to microsoft/mcp
  • Adds individual users to CODEOWNERS as temporary replacements for the missing team
  • Fixes various documentation links to point to the correct repository locations

Reviewed Changes

Copilot reviewed 11 out of 12 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
servers/Azure.Mcp.Server/README.md Updates repository links and documentation references to new location
eng/vscode/README.md Updates image links and repository references to new location
eng/tools/ToolDescriptionEvaluator/Quickstart.md Updates documentation link to new repository
docs/Authentication.md Updates troubleshooting and issue links to new repository
TROUBLESHOOTING.md Updates all repository links and adds table of contents entries
SUPPORT.md Updates issue and documentation links to new repository
README.md Updates main repository reference and Microsoft trademark link
CONTRIBUTING.md Updates all repository links and references throughout
.github/copilot-instructions.md Updates contribution guidelines link
.github/PULL_REQUEST_TEMPLATE.md Updates contribution guidelines and tool evaluator links
.github/CODEOWNERS Adds individual users temporarily and updates file paths from areas/ to tools/

@hallipr hallipr requested review from jongio, joshfree and vcolin7 August 22, 2025 00:21
@hallipr hallipr enabled auto-merge August 22, 2025 06:59
@hallipr hallipr disabled auto-merge August 22, 2025 06:59
@@ -1,7 +1,7 @@
# Contributing to Azure MCP
Copy link
Member

Choose a reason for hiding this comment

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

@jongio @joshfree Should this move into the Azure.Mcp.Server directory?

@weshaggard weshaggard merged commit a8629a7 into main Aug 22, 2025
11 of 25 checks passed
@weshaggard weshaggard deleted the users/pahallis/readme-links branch August 22, 2025 15:57
feiskyer pushed a commit to feiskyer/microsoft-mcp that referenced this pull request Sep 8, 2025
* Add xunit3 and sort/cleanup dependencies

* Migrate existing test

* Switch to SkipWhen

* Update tests/Client/ClientToolTests.cs

Co-authored-by: Christopher Scott <[email protected]>

* Fix after rebase

* Remove no longer needed comment

---------

Co-authored-by: Daniel Marbach <[email protected]>
Co-authored-by: Christopher Scott <[email protected]>
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.

4 participants