Skip to content

Conversation

@hallipr
Copy link
Member

@hallipr hallipr commented Sep 2, 2025

What does this PR do?

Add missing parameter from build.yml down to release.yml

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 September 2, 2025 22:45
@hallipr hallipr requested a review from a team as a code owner September 2, 2025 22:45
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 adds support for including native builds in the Azure MCP Server pipeline by propagating the IncludeNative parameter through the build configuration files.

  • Adds IncludeNative: true parameter to the Azure MCP Server build configuration
  • Updates the common pipeline template to accept and pass through the IncludeNative parameter

Reviewed Changes

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

File Description
servers/Azure.Mcp.Server/build.yml Adds IncludeNative: true to enable native builds for this server
eng/pipelines/templates/common.yml Defines the IncludeNative parameter and passes it to the release template

@hallipr hallipr merged commit 146a438 into main Sep 2, 2025
9 of 25 checks passed
@hallipr hallipr deleted the users/pahallis/buildNative branch September 2, 2025 22:57
colbytimm pushed a commit to colbytimm/microsoft-mcp that referenced this pull request Sep 27, 2025
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