-
-
Notifications
You must be signed in to change notification settings - Fork 9.3k
chore: add support for node 24 #24586
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
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
innerdvations
left a comment
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.
Some things I remember from past updates:
- there may be a grid in a readme of recommended versions to update (can't remember)
- there is some code that runs on startup and reinforces the node engines at runtime (or logs a warning?) that should be updated
Ok, I checked this and it's built so that it doesn't need to be manually updated anymore. We may want to set the .nvmrc to |
|
Also, we might need to:
|
|
We need to wait for Cloud to update before we can merge this or it'll break any builds on 24. |
ea2dafc to
c3f10a5
Compare
|
Size Change: -242 kB (-2.04%) Total Size: 11.6 MB
ℹ️ View Unchanged
|
What does it do?
Why is it needed?
How to test it?
0.0.0-experimental.1827b02a6ce14d67bfd14ebe164a8039ef6ca2f3resolves #24731