Skip to content

[DOCS] Align Node.js requirement in docs README with package.json eng… - #1397

Merged
Aryan-Verma-999 merged 1 commit into
RocketChat:developfrom
swarnendusahu10-dotcom:docs/update-docs-package-node-version
Oct 23, 2026
Merged

Aryan-Verma-999 merged 1 commit into
RocketChat:developfrom
swarnendusahu10-dotcom:docs/update-docs-package-node-version

Conversation

@swarnendusahu10-dotcom

Copy link
Copy Markdown
Contributor

Proposed Changes

Update the Node.js requirement in packages/docs/README.md from v22 to v24 (LTS) to reflect the "engines": { "node": ">=24.0.0" } specification defined in packages/docs/package.json.

Motivation

While the root repository was recently updated to require Node 24 via .nvmrc and scripts/node-check.js, packages/docs/README.md still instructed contributors to install Node 22 (nvm install 22). Because packages/docs/package.json enforces "node": ">=24.0.0", following the existing docs instructions leads to installation errors when working on the documentation site.

Testing

  • Verified engines field in packages/docs/package.json.
  • Confirmed docs README instructions now align with root .nvmrc (v24.0.0).

@CLAassistant

CLAassistant commented Oct 21, 2026 •

Copy link
Copy Markdown

CLA assistant check
All committers have signed the CLA.

@Aryan-Verma-999
Aryan-Verma-999 merged commit a1238d3 into RocketChat:develop Oct 23, 2026
4 checks passed
github-actions Bot added a commit that referenced this pull request Oct 23, 2026
@Aryan-Verma-999

Copy link
Copy Markdown
Collaborator

LGTM, Thanks for the contribution :)

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.

3 participants