-
Notifications
You must be signed in to change notification settings - Fork 103
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 #1056
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #1056 +/- ##
=======================================
Coverage 66.78% 66.78%
=======================================
Files 51 51
Lines 849 849
Branches 175 173 -2
=======================================
Hits 567 567
Misses 272 272
Partials 10 10 ☔ View full report in Codecov by Sentry. |
@BilalQamar95 Looks like this one might have a conflict too 😬. Sorry! |
@justinhynes I have resolved conflicts & updated the PR |
PR will be updated and ready for review once the prerequisite PR for step 1 of the node v20 upgrade process is reviewed/merged. |
#1069 has been merged 👍 |
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.