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

Use Node.js version 18 #595

Closed
wants to merge 1 commit into from
Closed

Use Node.js version 18 #595

wants to merge 1 commit into from

Conversation

victorlin
Copy link
Member

@victorlin victorlin commented Sep 27, 2022

Description of proposed changes

Although the current version (14) is still in active LTS status, the latest LTS version comes with performance improvements [1]. The performance improvements from version 16 should also apply [2].

Also update the npm version to a minimum of 8.6.0 since it corresponds with Node.js 18.0.0 per https://nodejs.org/en/download/releases/.

Related issue(s)

N/A

Tasks

Testing

  • Checks pass
  • Manually test preview build

Although the current version (14) is still in active LTS status, the
latest LTS version comes with performance improvements [1].
The performance improvements from version 16 should also apply [2].

Also update the npm version to a minimum of 8.6.0 since it corresponds
with Node.js 18.0.0 per https://nodejs.org/en/download/releases/.

[1]: https://nodejs.org/en/blog/release/v18.0.0/
[2]: https://nodejs.org/en/blog/release/v16.0.0/
@victorlin victorlin self-assigned this Sep 27, 2022
@nextstrain-bot nextstrain-bot temporarily deployed to nextstrain-s-victorlin--a96e4k September 27, 2022 18:19 Inactive
@@ -8,8 +8,8 @@
"repository": "github:nextstrain/nextstrain.org",
"homepage": "https://nextstrain.org",
"engines": {
"node": "^14",
"npm": "^6.14"
"node": "^18",
Copy link
Member Author

Choose a reason for hiding this comment

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

This error from npm run build:

error UNHANDLED REJECTION error:0308010C:digital envelope routines::unsupported

was also observed in nextstrain/auspice#1520. Webpack, an Auspice dependency, is the culprit. This will be fixed once that PR is merged, released, and the Auspice version is bumped for this project.

@victorlin
Copy link
Member Author

This was premature and incomplete. Closing in favor of #635.

@victorlin victorlin closed this Nov 29, 2022
@victorlin victorlin deleted the victorlin/use-node-18 branch November 29, 2022 01:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
No open projects
Development

Successfully merging this pull request may close these issues.

2 participants