docs(azmcp): add Managed Remote hosting survey banner to README#2998
Draft
yunjchoi wants to merge 2 commits into
Draft
docs(azmcp): add Managed Remote hosting survey banner to README#2998yunjchoi wants to merge 2 commits into
yunjchoi wants to merge 2 commits into
Conversation
Adds a clickable image banner at the top of the Azure MCP Server README inviting users to fill out the Managed Remote hosting survey (https://aka.ms/azmcp/managed-hosting-survey). The banner is wrapped in a remove-section annotation so it only appears in the GitHub README and is excluded from nuget/vsix/npm/pypi package READMEs. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
yunjchoi
requested review from
JasonYeMSFT,
chidozieononiwu,
hallipr,
joshfree,
saikoumudi,
tmeschter and
wbreza
Contributor
There was a problem hiding this comment.
Pull request overview
Adds a GitHub-only banner at the top of the Azure MCP Server README to direct users to the “Azure MCP Managed Remote hosting” survey, while ensuring the banner is stripped from all packaged README variants via remove-section annotations.
Changes:
- Added a clickable survey banner link at the top of
servers/Azure.Mcp.Server/README.md, wrapped inremove-sectionmarkers fornuget;vsix;npm;pypi. - Added the corresponding image asset under
servers/Azure.Mcp.Server/images/.
Reviewed changes
Copilot reviewed 1 out of 2 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| servers/Azure.Mcp.Server/README.md | Inserts a GitHub-only banner link using remove-section annotations so package README outputs exclude it. |
| servers/Azure.Mcp.Server/images/managed-hosting-survey-banner.png | Adds the banner image referenced by the README. |
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
jongio
reviewed
Jul 6, 2026
jongio
left a comment
jongio
left a comment
Contributor
There was a problem hiding this comment.
Checked the remove-section annotation format against existing patterns in this README (remove_note_ga, remove_azure_logo, remove_install_links). The new remove_managed_hosting_survey identifier follows the same convention.
Verified:
- Package targets (
nuget;vsix;npm;pypi) match other GitHub-only content sections - Image path (
images/managed-hosting-survey-banner.png) resolves correctly from the README location - Start/end annotation identifiers are correctly paired
- Alt text is present for accessibility
No issues found. CI builds are still in progress.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
Adds an image banner at the top of the Azure MCP Server README inviting users to fill out the Azure MCP Managed Remote hosting survey.
servers/Azure.Mcp.Server/images/managed-hosting-survey-banner.pngremove-sectionannotation (nuget;vsix;npm;pypi) so it appears only in the GitHub README and is excluded from all package-specific READMEs.Validation
Process-PackageReadMe.ps1 -Command validatepasses.npmconfirms the banner is stripped from package READMEs.Invoke-Cspell.ps1on the README surfaces no new spelling issues from this change.Preview