Skip to content
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

Make coverage pluggable #902

Closed
garthk opened this issue Dec 31, 2018 · 3 comments
Closed

Make coverage pluggable #902

garthk opened this issue Dec 31, 2018 · 3 comments
Assignees
Labels
feature New functionality or improvement

Comments

@garthk
Copy link

garthk commented Dec 31, 2018

If we could replace coverage.js with an API-compatible module at runtime (cf: custom reporters and transform modules), we could experiment with native v8 coverage without an extra transpile step. In turn, that could plausibly fix #856 as a side-effect.

@geek geek added the request label Jan 2, 2019
@geek
Copy link
Member

geek commented Jan 2, 2019

@garthk I do want to add support to run coverage through the new v8 coverage, this should also make it easier to track coverage in child processes

@Marsup Marsup added feature New functionality or improvement and removed request labels Sep 21, 2019
@hueniverse hueniverse self-assigned this Sep 22, 2019
@hueniverse
Copy link
Contributor

I would love to use v8 native coverage. This will happen after we stop supporting node 8 in a few months. I do not want to make coverage pluggable as you can already do that by simply not using lab for coverage...

@lock
Copy link

lock bot commented Jan 9, 2020

This thread has been automatically locked due to inactivity. Please open a new issue for related bugs or questions following the new issue template instructions.

@lock lock bot locked as resolved and limited conversation to collaborators Jan 9, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
feature New functionality or improvement
Projects
None yet
Development

No branches or pull requests

4 participants