Skip to content
This repository was archived by the owner on Feb 6, 2026. It is now read-only.

Update docs to use node 20#628

Merged
KarishmaGhiya merged 1 commit into
Azure:mainfrom
chidozieononiwu:update-readme
Jul 17, 2025
Merged

Update docs to use node 20#628
KarishmaGhiya merged 1 commit into
Azure:mainfrom
chidozieononiwu:update-readme

Conversation

@chidozieononiwu

@chidozieononiwu chidozieononiwu commented Jul 16, 2025

Copy link
Copy Markdown
Member

What does this PR do?

Change @azure/mcp node version to >=20.0, to align with JS Azure SDK.

GitHub issue number?

Copilot AI review requested due to automatic review settings July 16, 2025 17:07
@chidozieononiwu chidozieononiwu requested a review from a team as a code owner July 16, 2025 17:07
@chidozieononiwu chidozieononiwu requested a review from hallipr July 16, 2025 17:07

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull Request Overview

Updates documentation to reflect the Node.js version requirement bump from 20 to 22. This change ensures users are aware of the updated runtime requirement for the Azure MCP Server.

  • Updated README.md installation instructions to specify Node.js 22 or later
  • Added changelog entry documenting the Node.js version requirement change

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
README.md Updated installation instruction to specify Node.js 22 instead of 20
CHANGELOG.md Added entry documenting the Node.js version bump and mentions package.json engine update

Comment thread CHANGELOG.md Outdated
@chidozieononiwu chidozieononiwu changed the title Update README.md instruction to use node 22 Update docs to use node 22 Jul 16, 2025
@chidozieononiwu chidozieononiwu force-pushed the update-readme branch 2 times, most recently from d419116 to afa1997 Compare July 16, 2025 19:56
@chidozieononiwu chidozieononiwu requested a review from a team as a code owner July 16, 2025 19:56
@chidozieononiwu chidozieononiwu changed the title Update docs to use node 22 Update docs to use node 20 Jul 16, 2025

@joshfree joshfree left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

There's an "npx 20" reference in our readme with a grammatical error, could you fix that at the same time and do a grep for NPX too to double check there's no wrong NPX version references too

@chidozieononiwu

Copy link
Copy Markdown
Member Author

npx 20

I have fixed the error.

Comment thread CHANGELOG.md
@chidozieononiwu

Copy link
Copy Markdown
Member Author

/check-enforcer override

@KarishmaGhiya KarishmaGhiya merged commit df3a60b into Azure:main Jul 17, 2025
2 checks passed
@github-project-automation github-project-automation Bot moved this from Untriaged to Done in Azure MCP Server (OLD) Jul 17, 2025
srnagar pushed a commit to srnagar/azure-mcp that referenced this pull request Jul 22, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

5 participants