Skip to content

merge #1406

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
ankushbindlish2 merged 6 commits into from
Dec 18, 2025
Merged

merge #1406

ankushbindlish2 merged 6 commits into from
Dec 18, 2025

Conversation

@ankushbindlish2
Copy link
Member

@ankushbindlish2 ankushbindlish2 commented Dec 18, 2025

What does this PR do?

[Provide a clear, concise description of the changes]

[Any additional context, screenshots, or information that helps reviewers]

GitHub issue number?

[Link to the GitHub issue this PR addresses]

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 servers/Azure.Mcp.Server/CHANGELOG.md and/or servers/Fabric.Mcp.Server/CHANGELOG.md for product changes (features, bug fixes, UI/UX, updated dependencies)
  • For MCP tool changes:
    • One tool per PR: This PR adds or modifies only one MCP tool for faster review cycles
    • Updated servers/Azure.Mcp.Server/README.md and/or servers/Fabric.Mcp.Server/README.md documentation
    • Validate README.md changes using script at eng/scripts/Process-PackageReadMe.ps1. See Package README
    • Updated command list in /servers/Azure.Mcp.Server/docs/azmcp-commands.md and/or /docs/fabric-commands.md
    • Run .\eng\scripts\Update-AzCommandsMetadata.ps1 to update tool metadata in azmcp-commands.md (required for CI)
    • 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 tools with new names, including new tools or renamed tools, update consolidated-tools.json
    • For new tools associated with Azure services or publicly available tools/APIs/products, add URL to documentation in the PR description
  • Extra steps for Azure MCP Server tool changes:
    • Updated test prompts in /servers/Azure.Mcp.Server/docs/e2eTestPrompts.md
    • 👉 For Community (non-Microsoft 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 mcp - pullrequest - live to run Live Test Pipeline

hallipr and others added 6 commits December 17, 2025 11:13
@hallipr
Fix versions in Fabric changelog and add debug text to script (#1391)
11de04a
@hallipr
Remove unnecessary build targets and resources (#1375)
5f8e1b8 
* Move azure icon to images folder
* Remove duplicate resources and unnecessary build targets
* Remove unnecessary reference to eng/dnx from pack-nuget
@hallipr
Prevent HashTable.Count from affecting result count check (#1397)
bcb092d
@alzimmermsft
Migrate Authorization to recordings (#1399)
ac5092b
@scbedd
Fix execution of parallel testclasses within testassembly (#1393)
90feea8 
* assets.json longer optional
* changes to prevent multiple proxy instances from restoring simultaneously
@AmosHersch
Fabric MCP: Add OneLake namespace to VSCode options (#1398)
3b7d4b5 
* Fabric MCP: Add OneLake namespace to VSCode options

* Update descriptions
@ankushbindlish2 ankushbindlish2 marked this pull request as ready for review December 18, 2025 08:13
@ankushbindlish2 ankushbindlish2 requested review from chidozieononiwu, fanyang-mono, g2vinay, saikoumudi and tmeschter and removed request for a team December 18, 2025 08:13
@ankushbindlish2 ankushbindlish2 merged commit f956d65 into storage-sync-onboarding Dec 18, 2025
1 check passed
@github-project-automation github-project-automation bot moved this from Untriaged to Done in Azure MCP Server Dec 18, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@vurhanau vurhanau Awaiting requested review from vurhanau vurhanau is a code owner

@jongio jongio Awaiting requested review from jongio jongio is a code owner

@xiangyan99 xiangyan99 Awaiting requested review from xiangyan99 xiangyan99 is a code owner

@g2vinay g2vinay Awaiting requested review from g2vinay g2vinay is a code owner automatically assigned from microsoft/azure-mcp

@tmeschter tmeschter Awaiting requested review from tmeschter tmeschter is a code owner automatically assigned from microsoft/azure-mcp

@saikoumudi saikoumudi Awaiting requested review from saikoumudi saikoumudi is a code owner automatically assigned from microsoft/azure-mcp

@chidozieononiwu chidozieononiwu Awaiting requested review from chidozieononiwu chidozieononiwu is a code owner automatically assigned from microsoft/azure-mcp

@fanyang-mono fanyang-mono Awaiting requested review from fanyang-mono fanyang-mono is a code owner automatically assigned from microsoft/azure-mcp

Assignees

No one assigned

Labels

None yet

Projects

Azure MCP Server
Status: Done

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

6 participants

@ankushbindlish2 @hallipr @alzimmermsft @scbedd @AmosHersch