Skip to content
This repository has been archived by the owner on May 2, 2024. It is now read-only.

Use c8 instead of nyc for code coverage #87

Closed
sholladay opened this issue Nov 15, 2021 · 3 comments
Closed

Use c8 instead of nyc for code coverage #87

sholladay opened this issue Nov 15, 2021 · 3 comments

Comments

@sholladay
Copy link

sholladay commented Nov 15, 2021

Inspired by: avajs/ava#2843

I'm actually having problems with nyc while on the AVA 4 release candidate, where the coverage table is empty, presumably having something to do with AVA now using worker threads instead of child processes. I tried c8 and it just works, including on AVA 3.

One caveat: c8 seems to be quite a bit slower, at least in a medium sized app where it adds almost 30 seconds to npm test compared to without coverage, while nyc adds less than 10 seconds (including on AVA 3 where it works properly).

@sindresorhus
Copy link
Owner

Yeah, I want to switch to c8.

@sindresorhus
Copy link
Owner

One caveat: c8 seems to be quite a bit slower, at least in a medium sized app where it adds almost 30 seconds to npm test compared to without coverage, while nyc adds less than 10 seconds (including on AVA 3 where it works properly).

I'm ok with that. It will probably be improved in the future.

@sindresorhus
Copy link
Owner

Closing as this repo is archived.

@sindresorhus sindresorhus closed this as not planned Won't fix, can't repro, duplicate, stale May 2, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

2 participants