Results 207 comments of Corey Farrell

I've opened #506 for adding dark mode. The plan is that this would be accomplished through CSS `@media (prefers-color-scheme: dark)` rather than a report option, so your browser would decide...

I'm not actually that familiar with jest so I don't know if it has the ability to configure per report options. That said we don't have an option to configure...

We don't have docs directly for this specific use case but an example is https://github.com/istanbuljs/istanbuljs/blob/master/monorepo-merge-reports.js. This script is run by the `posttest` script at the monorepo root. The monorepo-merge-reports.js script...

Something is injecting code with many branches. Have you tried adding `/* istanbul ignore next */` right before the `import authenticatedStyle from 'authenticated/style';`? That's the only thought I have.

@jdalton could you point me to an example of where this is a problem? istanbul should already detect that files are already instrumented.. the only exception is if you have...

Any thoughts about dropping collection of statements data entirely? This could simplify the instrumentation code and decrease performance penalty of running instrumented code. I don't think this change should be...

I believe this should be done as a semver-major change across the board (major bump to effected istanbuljs libraries, nyc and babel plugin). So not part of istanbuljs/nyc#972. I think...

I've added an item to the checklist for reporting `0x` for instrumented but not covered lines, this will be important when the report doesn't show that the statement was not...

> One of the main design disagreements I have is that c8 is that it does not parse the source text. This choice has a huge pro: it gets rid...

> I double-checked, c8 supports /* c8 ignore next */ by parsing the line. That's great news, I hadn't realized. > Regarding if (condition1 || condition2) {}, V8 cannot distinguish...