Skip to content

Update Build Info to handle first GA scenario of MCP servers.#2394

Merged
g2vinay merged 2 commits intomicrosoft:mainfrom
g2vinay:add-logic-to-honor-csproj-version
Apr 13, 2026
Merged

Update Build Info to handle first GA scenario of MCP servers.#2394
g2vinay merged 2 commits intomicrosoft:mainfrom
g2vinay:add-logic-to-honor-csproj-version

Conversation

@g2vinay
Copy link
Copy Markdown
Contributor

@g2vinay g2vinay commented Apr 13, 2026

No description provided.

@g2vinay g2vinay marked this pull request as ready for review April 13, 2026 21:06
@g2vinay g2vinay requested review from a team as code owners April 13, 2026 21:06
@g2vinay g2vinay changed the title Update Build Info to handle first GA scenario. Update Build Info to handle first GA scenario of MCP servers. Apr 13, 2026
Copy link
Copy Markdown
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

Note

Copilot was unable to run its full agentic suite in this review.

Updates the build-info generation script to allow producing a stable VSIX version for an initial GA release when no Marketplace history exists for the major-version series.

Changes:

  • Adds a special-case path to use the .csproj version directly when the version is exactly 1.0.0 (stable) and no Major.0.X Marketplace versions exist.
  • Updates error handling/messages for the “no Marketplace versions found” scenario outside that special case.

@github-project-automation github-project-automation bot moved this from Untriaged to In Progress in Azure MCP Server Apr 13, 2026
@g2vinay g2vinay merged commit cc4ef95 into microsoft:main Apr 13, 2026
30 checks passed
@github-project-automation github-project-automation bot moved this from In Progress to Done in Azure MCP Server Apr 13, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

4 participants