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: test with node.js v18, remove v17 #4478
Conversation
Codecov Report
@@ Coverage Diff @@
## master #4478 +/- ##
=======================================
Coverage 98.75% 98.75%
=======================================
Files 205 205
Lines 7327 7327
Branches 2080 2080
=======================================
Hits 7236 7236
Misses 33 33
Partials 58 58 Continue to review full report at Codecov.
|
dang, related issue: nodejs/node#42638 @lukastaegert seems like there's multiple ways to solve this issue with node.js v18.
I applied #1 this as a temporary ci fix. would it also be an option to use built-in source maps? https://nodejs.medium.com/source-maps-in-node-js-482872b56116 |
c91aa90
to
cd83cba
Compare
The |
This PR contains:
Are tests included?
Breaking Changes?
List any relevant issue numbers:
Description