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

More arbitrary c8 ignores #4305

Merged
merged 2 commits into from
Mar 6, 2024
Merged

More arbitrary c8 ignores #4305

merged 2 commits into from
Mar 6, 2024

Commits on Mar 6, 2024

  1. More arbitrary c8 ignores 

    As we add code, more existing code suddenly gets marked as
uncovered by unit tests intermittently. This adds c8 ignore
comments for everything that turned up when running

   npm test -- --no-cache --runInBand

But I really hope this gets fixed upstream soon, because this isn't
sustainable.

For more context, see:
#4234

And the (hopefully correct) upstream issue:
istanbuljs/v8-to-istanbul#236
    @Vinnl
    Vinnl committed Mar 6, 2024
    Configuration menu
    Copy the full SHA
    e57bfd6 View commit details
    Browse the repository at this point in the history

  2. Merge branch 'main' into c8-ignores

    @flozia
    flozia committed Mar 6, 2024
    Configuration menu
    Copy the full SHA
    869ffdf View commit details
    Browse the repository at this point in the history