Skip to content

Upgrade Node to 22#1749

Merged
axelboc merged 1 commit intomainfrom
up-node
Feb 3, 2025
Merged

Upgrade Node to 22#1749
axelboc merged 1 commit intomainfrom
up-node

Conversation

@axelboc
Copy link
Contributor

@axelboc axelboc commented Jan 30, 2025

Upgrade Node to v22, which is the current LTS version, and fix #1717 by mentioning the prerequisite of installing Node and pnpm in the Getting started section of the CONTRIBUTING guide.

@axelboc axelboc requested a review from loichuder January 30, 2025 13:31
> If you work on multiple projects that require different versions of Node and
> pnpm, we recommend installing them in an isolated environement (e.g. with
> [Conda](https://docs.conda.io/projects/conda/en/stable/)) or with a
> specialised tool like [Volta](https://docs.volta.sh/).
Copy link
Member

Choose a reason for hiding this comment

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

Is nvm still a thing ?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yeah though I didn't mention it because I don't think it allows installing pnpm. 🤷

Copy link
Member

Choose a reason for hiding this comment

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

Indeed. I am still using it but with a global pnpm installation which is not great. Yeah, this is not something we want to encourage.

@axelboc axelboc merged commit 68d8d10 into main Feb 3, 2025
8 checks passed
@axelboc axelboc deleted the up-node branch February 3, 2025 08:42
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.

Mention Node in CONTRIBUTING.md

2 participants