-
Notifications
You must be signed in to change notification settings - Fork 326
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
MAINT - Bump node version (22.x) #1993
Conversation
Note this PR needs rebasing against main as I opened it before the fix to add FA to our build assets. Will do next week. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do you want this pre 0.16.0 ?
Personally I would wait |
So we are in agreement then and will leave it until 0.16.1 or so. |
May I suggest we may also use milestones for what for which version ? |
Made a 0.16.1 it's always easy to remove a milestone. |
I don't really expect this to break anything, but still I think it would be wise to be cautious, and wait a few days. |
A few days, have now been a few weeks, merging. |
We currently use
node.js 16
for PST, which has long been out of LTS.This PR bumps node to v22, which is the Current version and will soon be moved to Active.
Ref: https://nodejs.org/en/about/previous-releases