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.

Sign up for GitHub

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

Jump to bottom

Make h5web compatible with newer node versions. #1716

Closed
wants to merge 1 commit into from
Closed

Make h5web compatible with newer node versions. #1716

wants to merge 1 commit into from

Conversation

NAThompson
Copy link

No description provided.

@NAThompson
Copy link
Author

@axelboc : This is in response to the following error message I got while trying the CONTRIBUTING.md instructions:

~/h5web ❯❯❯ pnpm install                                                                                                                                                        main
 ERR_PNPM_UNSUPPORTED_ENGINE  Unsupported environment (bad pnpm and/or Node.js version)

Your Node version is incompatible with "~/h5web".

Expected version: 20.x
Got: v23.1.0

This is happening because the package's manifest has an engines.node field specified.
To fix this issue, install the required Node version.

@axelboc
Copy link
Contributor

axelboc commented Oct 31, 2024

Hi @NAThompson, this is expected. The development environment is pinned to Node 20.x for stability. I recommend using a Node version manager like Volta to switch version easily.

That being said, we might move to Node 22 soon since it is now the LTS version. More info on Node versioning here.

@axelboc axelboc closed this Oct 31, 2024
@axelboc axelboc mentioned this pull request Oct 31, 2024
Open
@NAThompson NAThompson deleted the NAThompson-patch-1 branch October 31, 2024 15:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@NAThompson @axelboc