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.

Sign up for GitHub

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

Jump to bottom

Use V8 builtin coverage reporting #69

Closed
mayurkale22 opened this issue Jun 28, 2019 · 0 comments · Fixed by #93
Closed

Use V8 builtin coverage reporting #69

mayurkale22 opened this issue Jun 28, 2019 · 0 comments · Fixed by #93
Labels
feature-request

Comments

@mayurkale22
Copy link
Member

🚀 Feature Request

V8 provides a native way of getting code coverage.

Here's a blog post about it.

Motivation

Make coverage 2-3x faster than nyc.
@mayurkale22 mayurkale22 added feature-request up-for-grabs Good for taking. Extra help will be provided by maintainers labels Jun 28, 2019
@markwolff markwolff mentioned this issue Jul 10, 2019
Merged
2 tasks
@mayurkale22 mayurkale22 removed the up-for-grabs Good for taking. Extra help will be provided by maintainers label Jul 24, 2019
dyladan pushed a commit to dyladan/opentelemetry-js that referenced this issue Sep 9, 2022
@vmarchaud
feat(context): add utils method to remove keys from context open-tele…
d107cee 
…metry#66 (open-telemetry#69)
dyladan pushed a commit to dyladan/opentelemetry-js that referenced this issue Sep 9, 2022
@vmarchaud
feat(context): add utils method to remove keys from context open-tele…
698e484 
…metry#66 (open-telemetry#69)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
feature-request
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Use V8 builtin coverage over nyc markwolff/opentelemetry-js
1 participant
@mayurkale22