Skip to content

Coverage report left in comments is "noisy" #35696

Closed
@bcoe

Description

@bcoe

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 👇

Screen Shot 2020-10-17 at 9 12 24 PM

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

Metadata

Metadata

Assignees

No one assigned

    Labels

    testIssues and PRs related to the tests.

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions