Skip to content

Conversation

KarishmaGhiya
Copy link
Member

This PR adds a custom readme for Azure MCP Npm package

@KarishmaGhiya KarishmaGhiya requested review from a team as code owners September 16, 2025 00:44
@Copilot Copilot AI review requested due to automatic review settings September 16, 2025 00:44
@KarishmaGhiya KarishmaGhiya requested review from joshfree, vukelich, conniey and xiangyan99 and removed request for Copilot September 16, 2025 00:44
Copy link
Member

@joshfree joshfree left a comment

Choose a reason for hiding this comment

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

Left comments

@github-project-automation github-project-automation bot moved this from Untriaged to In Progress in Azure MCP Server Sep 16, 2025
@Copilot Copilot AI review requested due to automatic review settings September 16, 2025 01:18
Copy link
Contributor

@Copilot 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 comprehensive documentation for the Azure MCP NPM package, including a detailed README file and contribution guidelines. The purpose is to provide clear setup instructions, usage examples, and service documentation for users wanting to integrate Azure MCP Server with their projects.

  • Added comprehensive README.md with installation instructions, usage examples, and service documentation
  • Created CONTRIBUTING.md file with guidelines for contributors
  • Added wrapperBinariesArchitecture.md explaining the package architecture

Reviewed Changes

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

File Description
eng/npm/README.md Comprehensive documentation covering installation, setup, usage examples, and complete list of 26+ supported Azure services
eng/npm/CONTRIBUTING.md Contributor guidelines including submission requirements and legal requirements
eng/npm/wrapperBinariesArchitecture.md Technical documentation explaining the wrapper package architecture for cross-platform support

@KarishmaGhiya
Copy link
Member Author

The broken links will be present once in main. If everything else looks good, can I get an approval?

@chidozieononiwu
Copy link
Member

Can you use azure logo from https://github.com/microsoft/mcp/tree/main/eng/images

@KarishmaGhiya
Copy link
Member Author

Can you use azure logo from https://github.com/microsoft/mcp/tree/main/eng/images

Removed it from my PR, since I wasn't even using it anywhere

@KarishmaGhiya KarishmaGhiya dismissed joshfree’s stale review September 17, 2025 23:01

addressed the feedback

@KarishmaGhiya KarishmaGhiya merged commit 9e56469 into microsoft:main Sep 17, 2025
25 checks passed
@github-project-automation github-project-automation bot moved this from In Progress to Done in Azure MCP Server Sep 17, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

4 participants