Closed
Description
Problem
I noticed on a couple PRs, that the coverage comment left by codecov.io was hidden.
I'm guessing that part of why folks wanted to hide this notification, is that it takes up a lot of screen space on the PR 👇
Proposed Solution
I have a PR open here that:
- enables coverage for a couple more platforms.
- uses codecov's layout setting to reduce the screen real-estate used by a coverage report.
- only posts the report if coverage has changed.
What if folks still aren't happy with the report?
If folks are still finding they find coverage a little bit noisy (and are inclined to hide the comment), we can turn off the comments in configuration.
@nodejs/testing