Skip to content

bump min node version to 22.12.0#6559

Merged
masenf merged 1 commit into
mainfrom
bump-min-node-version
May 22, 2026
Merged

bump min node version to 22.12.0#6559
masenf merged 1 commit into
mainfrom
bump-min-node-version

Conversation

@adhami3310
Copy link
Copy Markdown
Member

fixes #6054

@adhami3310 adhami3310 requested a review from a team as a code owner May 22, 2026 18:36
@greptile-apps
Copy link
Copy Markdown
Contributor

greptile-apps Bot commented May 22, 2026

Greptile Summary

This PR bumps the minimum required Node.js version from 20.19.0 to 22.12.0, addressing issue #6054. Node 20 reached end-of-life in April 2026, making Node 22 (Active LTS) the appropriate new baseline.

  • The change is a single-line update to the Node.MIN_VERSION constant in installer.py, which is already the sole source of truth — all version checks and error messages across js_runtimes.py and exec.py consume it via constants.Node.MIN_VERSION, so no other files need updating.
  • The existing CI workflow (check_node_latest.yml) tests against node: \"node\" (latest), which will continue to cover compatibility going forward.

Confidence Score: 5/5

Safe to merge — a focused one-line constant bump with no ancillary files requiring updates.

The change touches a single centralized constant. All downstream version checks already reference this constant, there are no hardcoded version strings elsewhere that need to be kept in sync, and the CI pipeline tests against the latest Node release, so any runtime incompatibility would surface in CI before merge.

No files require special attention.

Important Files Changed

Filename Overview
packages/reflex-base/src/reflex_base/constants/installer.py Bumps Node.js minimum required version from 20.19.0 to 22.12.0; the constant is already the single source of truth consumed throughout the codebase via constants.Node.MIN_VERSION.

Reviews (1): Last reviewed commit: "bump min node version to 22.12.0" | Re-trigger Greptile

@codspeed-hq
Copy link
Copy Markdown

codspeed-hq Bot commented May 22, 2026

Merging this PR will not alter performance

✅ 24 untouched benchmarks


Comparing bump-min-node-version (2fac61a) with main (d611a5d)

Open in CodSpeed

@masenf masenf merged commit dc8a23c into main May 22, 2026
70 checks passed
@masenf masenf deleted the bump-min-node-version branch May 22, 2026 23:39
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.

reflex export fails with rx.code_block due to ESM/CJS interop in react-syntax-highlighter

2 participants