-
Notifications
You must be signed in to change notification settings - Fork 61
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
[build] Use Node.js v18.17.1 (LTS) #3141
[build] Use Node.js v18.17.1 (LTS) #3141
Conversation
Fixes: redhat-developer#3121 Signed-off-by: Victor Rubezhny <vrubezhny@redhat.com>
Codecov ReportPatch and project coverage have no change.
Additional details and impacted files@@ Coverage Diff @@
## main #3141 +/- ##
=======================================
Coverage 31.99% 31.99%
=======================================
Files 74 74
Lines 5122 5122
Branches 974 974
=======================================
Hits 1639 1639
Misses 3483 3483 ☔ View full report in Codecov by Sentry. |
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.
lgtm
@@ -1 +1 @@ | |||
v16.17.1 | |||
v18.17.1 |
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.
Just as a heads up, we can't use any nodejs 18 specific features until VS Code itself runs our extension using NodeJS 18, which will probably land in the September update (which will probably be out mid September)
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.
Changing the Node.js version here doesn't automatically leads to build (or whatever else) to immediately start using the new Node.js version. The version that is currently configured will still be used independently of what you have written in .nvmrc
until you apply it by nvm use
, won't it"
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.
Yes
Works well for me. I would note that even with Node 16, if I remove the
|
Fixes: #3121