Skip to content

change: Drop support for Node.js 18 and 20 (v14)#3138

Open
lahirumaramba wants to merge 3 commits into
v14from
lm-node-v26
Open

change: Drop support for Node.js 18 and 20 (v14)#3138
lahirumaramba wants to merge 3 commits into
v14from
lm-node-v26

Conversation

@lahirumaramba
Copy link
Copy Markdown
Member

@lahirumaramba lahirumaramba commented May 12, 2026

Drop support for Node.js 18 and 20
Add Node.js 26 to CIs
Upgrade firebase-tools to v15

Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist Bot left a comment

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 project's supported Node.js version to 22 and higher, dropping support for versions 18 and 20 in the documentation and package engine requirements. Feedback indicates that the changes are incomplete, as CI configuration files and other package manifests within the repository still need to be updated to reflect this new requirement consistently.

Comment thread package.json
"homepage": "https://firebase.google.com/",
"engines": {
"node": ">=18"
"node": ">=22"
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

The PR description mentions adding "Node.js 26 to CIs", which appears to be a typo (likely intended for Node.js 22). Additionally, this PR is missing updates to CI configuration files (e.g., GitHub Actions workflows) and other package manifests in the repository (for instance, test/integration/postcheck/package.json still references @types/node: >=14.0.0). Please ensure all references to supported Node.js versions are updated consistently across the project.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

release:stage Stage a release candidate release-note

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants