Skip to content

Refactor coverage tests to use c8 instead of nyc #77

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
Merged
wooorm merged 1 commit into from
Nov 3, 2019
Merged

Refactor coverage tests to use c8 instead of nyc #77

wooorm merged 1 commit into from
Nov 3, 2019

Conversation

@ChristianMurphy
Copy link
Member

@ChristianMurphy ChristianMurphy commented Nov 3, 2019

resolves #76 (comment)

@wooorm wooorm added 🏗 area/tools This affects tooling 🏡 area/internal This affects the hidden internals 📦 area/deps This affects dependencies 🕸️ area/tests This affects tests 🙆 yes/confirmed This is confirmed and ready to be worked on 🦋 type/enhancement This is great to have labels Nov 3, 2019
@codecov-io

This comment has been minimized.

Sign in to view
@wooorm wooorm merged commit d8b5cf5 into unifiedjs:master Nov 3, 2019
@wooorm wooorm changed the title test: migrate to c8 for coverage reporting Refactor coverage tests to use c8 instead of nyc Nov 3, 2019
@ChristianMurphy ChristianMurphy deleted the test/c8 branch November 3, 2019 17:22
wooorm pushed a commit that referenced this pull request Nov 9, 2019
@ChristianMurphy @wooorm
Refactor coverage tests to use c8 instead of nyc
e702307 
Closes GH-77.

Reviewed-by: Titus Wormer <[email protected]>
@wooorm wooorm added ⛵️ status/released and removed 🙆 yes/confirmed This is confirmed and ready to be worked on labels Dec 1, 2019
@wooorm wooorm added the 💪 phase/solved Post is done label May 20, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@wooorm wooorm wooorm approved these changes

Assignees

No one assigned

Labels

📦 area/deps This affects dependencies 🏡 area/internal This affects the hidden internals 🕸️ area/tests This affects tests 🏗 area/tools This affects tooling 💪 phase/solved Post is done 🦋 type/enhancement This is great to have

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@ChristianMurphy @codecov-io @wooorm