-
Notifications
You must be signed in to change notification settings - Fork 155
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: Upgrade to Node 20 #1295
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #1295 +/- ##
=======================================
Coverage 87.69% 87.69%
=======================================
Files 124 124
Lines 2299 2299
Branches 646 642 -4
=======================================
Hits 2016 2016
Misses 274 274
Partials 9 9 ☔ View full report in Codecov by Sentry. |
PR will be updated and ready for review once the prerequisite PR for step 1 of the node v20 upgrade process is reviewed/merged. |
…ilalqamar95/node-v20-upgrade
Hey @BilalQamar95! Thank you so much for making these upgrade PRs! It looks like this one is out of date with master. If you could rebase it I'll push to land it today! |
Description
package-lock.json
with nodev20
and NPM10
..nvmrc
tov20
lockfile
version check workflowSecond step in the Node 20 upgrade process, See the tracking issue for futher information.