-
Notifications
You must be signed in to change notification settings - Fork 395
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
ci: Add Node.js 20.x to CI and remove 14.x #1603
Conversation
a798d24
to
025a985
Compare
Codecov Report
@@ Coverage Diff @@
## main #1603 +/- ##
==========================================
- Coverage 96.87% 96.85% -0.02%
==========================================
Files 200 200
Lines 39191 39205 +14
Branches 25 23 -2
==========================================
+ Hits 37966 37972 +6
- Misses 1225 1233 +8
Flags with carried forward coverage won't be shown. Click here to find out more.
... and 2 files with indirect coverage changes 📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
There are 2 smoke test files failing on 20 because of the self-signed cert. I'm triaging that separately. |
…s as they are not necessary
Proposed Release Notes
Links
Closes #1726
Details
This was mostly a clean upgrade. I have skipped the ESM tests as we know the ESM loader isn't working. I will unleash those tests when I PR that fix. The highlights here are that server.close changed behavior so I just made tests bind to dynamic ports to workaround this issue. There's also 2 features that no longer ship with node around dtrace and etw.