-
Notifications
You must be signed in to change notification settings - Fork 5k
chore: set minimum Node.js requirement to 20 #38952
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
Test results for "tests 1"4 flaky34669 passed, 695 skipped Merge workflow run. |
Test results for "MCP"96 failed 3350 passed, 116 skipped Merge workflow run. |
|
I don't think we can afford this, there are still too many users on Node 18. |
@dgozman |
|
Let's wait until 26 is out. There are still many users on 18, even though we don't officially support 18 anymore, we don't want to break them for no reason either. |
Fixes #38901