-
-
Notifications
You must be signed in to change notification settings - Fork 941
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
Enable coverage again #1688
Comments
c8 is a better and faster alternative to nyc and it works with esm |
@sayem314 it's not better because it's not reliable. It gives false line numbers. |
@szmarczak I see. For me, it works fine hence recommended. |
@sayem314 Thanks for the suggestion :) |
@sindresorhus it looks like got/package.json tells ava to use |
Relevant: nodejs/modules#513 “Using multiple loaders”. |
I think this could be done with a single loader, as long as it’s Babel instead of ts-node: use {
"presets": ["@babel/typescript"],
"env": {
"test": {
"plugins": ["babel-plugin-istanbul"]
}
}
} |
Had to be disabled for ESM: 1eee7c2
Blocked by istanbuljs/nyc#1353 (I tried using the mentioned
@istanbuljs/esm-loader-hook
, but it didn't work)The text was updated successfully, but these errors were encountered: