-
Notifications
You must be signed in to change notification settings - Fork 106
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
Bump NodeJS to version 20 #1513
Conversation
Also bumps the version of `node-gyp` transitive dependency to address a security vulnerability.
Bumps dependency versions, as well as remove manual caching.
Pull Request Test Coverage Report for Build 7593133001
💛 - Coveralls |
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.
Looks good! Let's have this merged and ready ASAP so that it becomes easier to setup with the same node version as the frontend.
* Update Node version * Remove duplicate header * Change H1s to H2s for better hierarchy
* Remove soft breaks * Add syntax highlighting for commands * Fix license line break
Great, thanks! I've updated (and reformatted) the README as well, will merge after the CI passes. |
Node 16 is EOL and Node 18 has build problems, but Node 20 seems to work fine and all tests pass locally on my MacBook.
TODOs: