deepsourcelabs/test-coverage-action

Coverage complaining about lines outside the PR delta

Closed this issue · 5 comments

First of all, I apologize if this is the wrong place to post this bug report. I do not have permissions to create a new topic in discuss.

My issue is that when we have a PR that touches a file with existing coverage issues, all of these issues are surfaced in a given PR rather than only the issues introduced or resolved by the pull request line delta. The analysis scope is set to "granular", yet existing coverage issues service anyway on lines not affected by a given pull request.

Screenshot from 2020-10-09 16-34-26

For a specific example, this pull request prysmaticlabs/prysm#7488 was rather small in scope and only reduced a few redundant error checks. However, the coverage reporting says this pull request introduces two new coverage issues which are completely unrelated.

stale commented

Is this issue still relevant? There has been no recent activity found and is considered stale. If relevant please comment or update to unstale.

Yes, it is still relevant

stale commented

Is this issue still relevant? There has been no recent activity found and is considered stale. If relevant please comment or update to unstale.

stale commented

This issue has been automatically closed because of inactivity. You can re-open if needed.

Please re-open. This issue was never addressed.