Skip to content

Conversation

@victorlin
Copy link
Member

@victorlin victorlin commented Sep 17, 2025

Description of proposed changes

I did some local testing with Node.js 24 and it worked fine without any code changes.

Related issue(s)

Follow-up to #1975 (comment)

Checklist

I did some local testing with Node.js 24 and it worked fine without any
code changes.
@victorlin victorlin self-assigned this Sep 17, 2025
@nextstrain-bot nextstrain-bot temporarily deployed to auspice-victorlin-node--4s5bq5 September 17, 2025 17:19 Inactive
@victorlin victorlin temporarily deployed to auspice-victorlin-node--4s5bq5 September 17, 2025 17:25 Inactive
@victorlin victorlin mentioned this pull request Sep 17, 2025
Copy link
Member

@jameshadfield jameshadfield left a comment

Choose a reason for hiding this comment

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

Node 24 allows us to write TS on the (Auspice) server which is nice. It won't be coming anytime soon here since v20 (and perhaps v22?) don't support it, but for the nextstrain.org server this would be a big win.

@victorlin victorlin merged commit 4908811 into master Sep 17, 2025
27 of 29 checks passed
@victorlin victorlin deleted the victorlin/node-24 branch September 17, 2025 21:14
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.

4 participants