Skip to content
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

feat: add recommended Node.js version #1012

Merged
merged 4 commits into from
Dec 21, 2024

Conversation

easymikey
Copy link
Contributor

Add recommended Node.js version for version manager supports .node_version and for Volta add package.json field.

I have prepared a draft for support .node_version file for volta

volta-cli/volta#1973

Fixes #995

  • Tests pass
  • Appropriate changes to README are included in PR

"license": "Apache-2.0"
"license": "Apache-2.0",
"volta": {
"node": "22.12.0"
Copy link
Collaborator

Choose a reason for hiding this comment

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

can we use just 22 here?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

No

I will be uploading an issue soon.

Screenshot 2024-12-20 at 18 01 47

* Switch to the recommended version of Node.js
* If you are using [Node.js version manager](https://github.com/shadowspawn/node-version-usage) supported `.node_version` using it.
* If you are using Volta, Node.js version will be set automatically from `package.json`.
* In another case manually install recommended version from `.node_version` file.
Copy link
Collaborator

Choose a reason for hiding this comment

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

  • Install manually Node.js >= 22
  • Delegate the routine to any version manager, that supports .node_version config
  • Use Volta, the target version will be set automatically from the package.json

Copy link
Contributor Author

Choose a reason for hiding this comment

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

D1

Copy link
Collaborator

@antongolub antongolub left a comment

Choose a reason for hiding this comment

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

Accepted. Thanks!

@antongolub antongolub merged commit c4e4902 into google:main Dec 21, 2024
19 checks passed
@easymikey easymikey deleted the feat-recommended_node_version branch December 21, 2024 08:48
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.

feat: add recommended node.js version for development
2 participants