Skip to content

Update documented Node requirement#6546

Open
vicksiyi wants to merge 2 commits into
FlowiseAI:mainfrom
vicksiyi:codex/update-node-requirement-docs
Open

Update documented Node requirement#6546
vicksiyi wants to merge 2 commits into
FlowiseAI:mainfrom
vicksiyi:codex/update-node-requirement-docs

Conversation

@vicksiyi

Copy link
Copy Markdown

Summary

  • Update the README quick start Node.js requirement to match the current package engine.
  • Document the pnpm 10.26.0 requirement in developer setup.

Fixes #4138

Tests

  • git diff --check

Notes

  • This is a documentation-only change based on the root package.json engines and CI workflow versions.

@gemini-code-assist gemini-code-assist Bot 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.

Code Review

This pull request updates the README.md to specify NodeJS 24.x and PNPM 10.26.0 as prerequisites. The feedback suggests specifying >= 10.26.0 for PNPM instead of a fixed version, as the global installation command installs the latest version and a fixed version might mislead developers.

Important

The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.

Comment thread README.md Outdated
### Prerequisite

- Install [PNPM](https://pnpm.io/installation)
- Install [PNPM](https://pnpm.io/installation) 10.26.0

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.

medium

Specifying an exact version like 10.26.0 might lead developers to believe only that exact version is supported, whereas the installation command npm i -g pnpm will install the latest version of PNPM. It is better to specify >= 10.26.0 to allow newer minor/patch versions of PNPM v10.

Suggested change
- Install [PNPM](https://pnpm.io/installation) 10.26.0
- Install [PNPM](https://pnpm.io/installation) >= 10.26.0

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Updated in the latest commit to document pnpm as >= 10.26.0, matching the engine minimum without implying only one exact patch version is supported.

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.

please update required Node version

1 participant