Skip to content

fix #2595 : update Node.js version requirement in README to match package.json#2596

Merged
bgravenorst merged 2 commits intoMetaMask:mainfrom
subhamkumarr:fix/nodejs-version-docs-mismatch
Feb 10, 2026
Merged

fix #2595 : update Node.js version requirement in README to match package.json#2596
bgravenorst merged 2 commits intoMetaMask:mainfrom
subhamkumarr:fix/nodejs-version-docs-mismatch

Conversation

@subhamkumarr
Copy link
Contributor

@subhamkumarr subhamkumarr commented Dec 26, 2025

Fix Node.js version requirement documentation mismatch

Problem

The README.md incorrectly states "Node.js version 18+" while package.json specifies "node": "20.x" in the engines field. This inconsistency can mislead contributors who may install an incompatible Node.js version.

Solution

Updated README.md to specify "Node.js version 20.x" to match the actual requirement in package.json.

Changes

  • Updated README.md line 14: Changed "Node.js version 18+" to "Node.js version 20.x"

Testing

  • Verified that the change aligns with the package.json engines field
  • No code changes, documentation only

Impact

  • ✅ Prevents confusion for new contributors
  • ✅ Ensures accurate setup instructions
  • ✅ Aligns documentation with actual project requirements

Fixes #2595


Note

Low Risk
Documentation-only change that updates local build prerequisites; no runtime or build logic is modified.

Overview
Updates README.md to require Node.js 20.x (instead of 18+) when building the docs locally, aligning the setup instructions with the package.json engines.node requirement.

Written by Cursor Bugbot for commit 5b17bfb. This will update automatically on new commits. Configure here.

@subhamkumarr subhamkumarr requested review from a team as code owners December 26, 2025 19:34
@vercel
Copy link

vercel bot commented Dec 26, 2025

@subhamkumarr is attempting to deploy a commit to the Consensys Team on Vercel.

A member of the Team first needs to authorize it.

@subhamkumarr
Copy link
Contributor Author

subhamkumarr commented Dec 26, 2025

@AyushBherwani1998 could you please review this and merge it when you get a chance? Thanks!
Issue #2595

@subhamkumarr
Copy link
Contributor Author

@alexandratran could you please review this and merge it when you get a chance? Thanks!
Issue #2595

Copy link
Contributor

@bgravenorst bgravenorst left a comment

Choose a reason for hiding this comment

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

Thanks for your contribution 🚀
lgtm

@vercel
Copy link

vercel bot commented Feb 10, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
metamask-docs Ready Ready Preview, Comment Feb 10, 2026 11:37pm

Request Review

@bgravenorst bgravenorst merged commit 8cde483 into MetaMask:main Feb 10, 2026
18 checks passed
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.

README lists incorrect Node.js version (18+) — project requires Node 20.x

2 participants