Benjamin E. Coe

Results 551 comments of Benjamin E. Coe

@stefan-guggisberg this was likely a bug in v8 itself, which appears to have been fixed in newer versions of Node.js _(I couldn't reproduce in v14/v16)_ Unfortunately these updates to v8...

@stefan-guggisberg perhaps keep this issue open, so other folks understand this category of issue.

Hello @Ramanujam1983, to make `c8` work with electron, you would need to configure Node.js to output coverage reports, by setting `NODE_V8_COVERAGE`. Similarly, you can get coverage reports from the inspector...

@mikecbrant, @connorjclark has been working on some functionality around being able to `start`/`stop` ignoring lines. For starters, I think you might want to try out this functionality, vs., the `ignore...

@blaizard I tend to run `c8` as a bin itself, assuming that it will use the version of Node configured in the shebang: ``` #!/usr/bin/env node ``` So I would...

@blaizard if we could figure out a way to test this, perhaps with a fake node bin and using `child_process`, I would be open to trying to make `c8` more...

@motss thank you for providing the test case 👍

@motss I dug into your example repo, when I run your test suite, `v8` reports a missing block in the `getWeekdays` method: ``` { "functionName": "getWeekdays", "ranges": [ { "startOffset":...

@Shigma as @j03m asks, could you provide a repository that demonstrates a minimal reproduction.