-
-
Notifications
You must be signed in to change notification settings - Fork 124
Update node to v20 #509
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Update node to v20 #509
Conversation
WalkthroughThe project’s required Node.js version has been updated from 18.x (or 16.x in some places) to 20.x. This change is reflected in documentation, CI workflow configurations, and the package.json engines field to ensure consistency across development, deployment, and automated checks. Changes
Poem
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
Visit the preview URL for this PR (updated for commit 8364ad4): https://game-ci-5559f--pr509-node20-xrxs3mi9.web.app (expires Fri, 13 Jun 2025 17:00:45 GMT) 🔥 via Firebase Hosting GitHub Action 🌎 Sign: 1f0574f15f83e11bfc148eae8646486a6d0e078b |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 0
🧹 Nitpick comments (2)
CONTRIBUTING.md (1)
21-21
: Use correct spelling "Node.js" and version formatting.The official spelling is “Node.js”, and it’s clearer to use a lowercase ‘x’ for version wildcards. For example:
- - [NodeJS](https://nodejs.org/) installed at v20.X. + - [Node.js](https://nodejs.org/) installed at v20.x.🧰 Tools
🪛 LanguageTool
[uncategorized] ~21-~21: The official spelling of this programming framework is “Node.js”.
Context: ...he following tools to be installed. - NodeJS installed at v20....(NODE_JS)
README.md (1)
38-43
: Standardize "Node.js" spelling and version format.Update references to use “Node.js” and lowercase ‘x’ in version wildcards. For example:
- This project has a hard dependency on Node20. You will need to make sure that you have the proper - version of node installed to avoid errors. + This project has a hard dependency on Node.js v20.x. You will need to make sure that you have the proper + version of Node.js installed to avoid errors. - [Install NodeJS v20 on Ubuntu](https://computingforgeeks.com/how-to-install-node-js-20-on-ubuntu/) - [Install NodeJS v20 using brew](https://apple.stackexchange.com/a/207883) + [Install Node.js v20.x on Ubuntu](https://computingforgeeks.com/how-to-install-node-js-20-on-ubuntu/) + [Install Node.js v20.x using brew](https://apple.stackexchange.com/a/207883)🧰 Tools
🪛 LanguageTool
[uncategorized] ~41-~41: The official spelling of this programming framework is “Node.js”.
Context: ... installed to avoid errors. - [Install NodeJS v20 on Ubuntu](https://computingforgeek...(NODE_JS)
[uncategorized] ~42-~42: The official spelling of this programming framework is “Node.js”.
Context: ...stall-node-js-20-on-ubuntu/) - [Install NodeJS v20 using brew](https://apple.stackexch...(NODE_JS)
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (6)
.github/workflows/checks.yml
(4 hunks).github/workflows/firebase-hosting-merge.yml
(1 hunks).github/workflows/firebase-hosting-pull-request.yml
(1 hunks)CONTRIBUTING.md
(1 hunks)README.md
(1 hunks)package.json
(1 hunks)
🧰 Additional context used
🪛 LanguageTool
CONTRIBUTING.md
[uncategorized] ~21-~21: The official spelling of this programming framework is “Node.js”.
Context: ...he following tools to be installed. - NodeJS installed at v20....
(NODE_JS)
README.md
[uncategorized] ~41-~41: The official spelling of this programming framework is “Node.js”.
Context: ... installed to avoid errors. - [Install NodeJS v20 on Ubuntu](https://computingforgeek...
(NODE_JS)
[uncategorized] ~42-~42: The official spelling of this programming framework is “Node.js”.
Context: ...stall-node-js-20-on-ubuntu/) - [Install NodeJS v20 using brew](https://apple.stackexch...
(NODE_JS)
⏰ Context from checks skipped due to timeout of 90000ms (1)
- GitHub Check: build_and_preview
🔇 Additional comments (4)
package.json (1)
9-9
: Engines field updated correctly.The
node
engine requirement has been bumped to>= 20.x
to align with the rest of the project. This change is consistent and ready to ship..github/workflows/firebase-hosting-pull-request.yml (1)
17-17
: Node.js version for CI updated.The
actions/setup-node@v4
step now uses Node.js 20, matching the project-wide version bump. No further changes needed here..github/workflows/firebase-hosting-merge.yml (1)
19-19
: Node.js version for CI updated.The
actions/setup-node@v4
step now uses Node.js 20, aligning with other workflows. This is good to merge..github/workflows/checks.yml (1)
15-15
: Bump Node.js runtime to v20
The Node.js version has been updated to v20 across all workflow jobs to align with the PR objective and ensure CI consistency.Also applies to: 26-26, 37-37, 48-48
Changes
Checklist
code of conduct
Summary by CodeRabbit
Chores
Documentation