davelosert/vitest-coverage-report-action

Error when add Summary

Closed this issue · 5 comments

sfer23 commented

image
Report cannot be processed with big amount of files.

Possible solution:
add option to disable File Coverage section in comment.

Hi @sfer23, thank you for reporting this!

Hm, indeed, that's a problem I didn't think of yet. Having that option would be great - do you mind submitting a PR for this?

Mid-Term, this could also be solved (or at least worked around to some degree) by having the coverage-split feature suggested in #160 to only show the file coverage for the files changed in the PR.

I just went ahead and put this in myself. I will have to do a final test tomorrow, but will release it then. 🙂

sfer23 commented

Nice solution, I'll also try to provide some variant PR, and also will check #160, nice idea also.

sfer23 commented

Also will test my version and describe the results

@sfer23: I just released the new version 2.0.0 with our changes and the new file-coverage-mode-setting. So this should be fixed now!

Thank you again for the contribution! 🥳