search

denoland / deno

A modern runtime for JavaScript and TypeScript.
https://deno.com
MIT License
97.2k stars 5.37k forks source link

`deno-coverage-ignore` comments #16626

Open eliassjogreen opened 1 year ago

eliassjogreen commented 1 year ago

It would be nice to have the deno-coverage-ignore and deno-coverage-ignore-file comments which would essentially act the same as all other deno-*-ignore comments but for ignoring a specified line, branch or file when collecting coverage.

bartlomieju commented 1 year ago

I think it makes sense. PRs are welcome!

gofenix commented 1 year ago

Hello, Maybe I can try to implement it.

bartlomieju commented 1 year ago

Hello, Maybe I can try to implement it.

Please do

loynoir commented 1 year ago

@zhenfeng-zhu

Any news? 🥺

Maybe section pair is easier to implement?

https://github.com/bcoe/c8

/* c8 ignore start */
function dontMindMe() {
  // ...
}
/* c8 ignore stop */
JAS-lzn commented 1 year ago

work on this seems abandoned, I'd like to try and help, but I'm unsure of where to get started.

@bartlomieju, any guidance on how this should be approached?