Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Binary file modified docs/repositories/images/pull-requests-tab-coverage.png
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
25 changes: 25 additions & 0 deletions docs/repositories/pull-requests.md
Original file line number Diff line number Diff line change
Expand Up @@ -39,6 +39,31 @@
end="<!--tabs-end-->"
%}

## Coverage tab

The Coverage tab provides a detailed breakdown of how your changes impact the project's test coverage. It allows you to ensure that new code is properly tested before it is merged.

Check failure on line 44 in docs/repositories/pull-requests.md

View workflow job for this annotation

GitHub Actions / vale

[vale] reported by reviewdog 🐶 [Microsoft.Contractions] Use 'it's' instead of 'it is'. Raw Output: {"message": "[Microsoft.Contractions] Use 'it's' instead of 'it is'.", "location": {"path": "docs/repositories/pull-requests.md", "range": {"start": {"line": 44, "column": 168}}}, "severity": "ERROR"}

Check warning on line 44 in docs/repositories/pull-requests.md

View workflow job for this annotation

GitHub Actions / vale

[vale] reported by reviewdog 🐶 [Microsoft.Adverbs] Consider removing 'properly'. Raw Output: {"message": "[Microsoft.Adverbs] Consider removing 'properly'.", "location": {"path": "docs/repositories/pull-requests.md", "range": {"start": {"line": 44, "column": 145}}}, "severity": "WARNING"}

**Coverage Overview**

At the top of the tab, two primary indicators give you an immediate health check of the Pull Request:

* **Diff Coverage**: The percentage of new or modified executable lines that are covered by tests.
* **Coverage Variation**: The net impact (increase or decrease) the PR has on the overall project coverage percentage.

**File Breakdown**

On the left sidebar, you can find a list of all files modified in the PR. For each file, Codacy displays its specific diff coverage percentage, helping you quickly identify which parts of your contribution might be missing tests.

Check warning on line 55 in docs/repositories/pull-requests.md

View workflow job for this annotation

GitHub Actions / vale

[vale] reported by reviewdog 🐶 [Microsoft.Adverbs] Consider removing 'quickly'. Raw Output: {"message": "[Microsoft.Adverbs] Consider removing 'quickly'.", "location": {"path": "docs/repositories/pull-requests.md", "range": {"start": {"line": 55, "column": 157}}}, "severity": "WARNING"}

**Diff Visualization**
The main view displays the code changes with a specialized coverage overlay to pinpoint exactly what is being tested:

Check failure on line 58 in docs/repositories/pull-requests.md

View workflow job for this annotation

GitHub Actions / vale

[vale] reported by reviewdog 🐶 [Microsoft.Contractions] Use 'what's' instead of 'what is'. Raw Output: {"message": "[Microsoft.Contractions] Use 'what's' instead of 'what is'.", "location": {"path": "docs/repositories/pull-requests.md", "range": {"start": {"line": 58, "column": 97}}}, "severity": "ERROR"}

* **Covered Lines (Green)**: Lines executed by your test suite. These are marked with a green highlight and a label indicating the number of hits (e.g., 1 hit), representing how many times that line was executed during the test run.

Check failure on line 60 in docs/repositories/pull-requests.md

View workflow job for this annotation

GitHub Actions / vale

[vale] reported by reviewdog 🐶 [Microsoft.Foreign] Use 'for example' instead of 'e.g.,'. Raw Output: {"message": "[Microsoft.Foreign] Use 'for example' instead of 'e.g.,'.", "location": {"path": "docs/repositories/pull-requests.md", "range": {"start": {"line": 60, "column": 148}}}, "severity": "ERROR"}
* **Uncovered Lines (Red)**: Executable lines that were not reached by any tests. These are marked with a red highlight and a Not covered label.

Check failure on line 61 in docs/repositories/pull-requests.md

View workflow job for this annotation

GitHub Actions / vale

[vale] reported by reviewdog 🐶 [Microsoft.Contractions] Use 'weren't' instead of 'were not'. Raw Output: {"message": "[Microsoft.Contractions] Use 'weren't' instead of 'were not'.", "location": {"path": "docs/repositories/pull-requests.md", "range": {"start": {"line": 61, "column": 52}}}, "severity": "ERROR"}

*Note: Lines that are not executable (like comments or empty lines) will not show a coverage status.*

Check failure on line 63 in docs/repositories/pull-requests.md

View workflow job for this annotation

GitHub Actions / vale

[vale] reported by reviewdog 🐶 [Microsoft.Contractions] Use 'won't' instead of 'will not'. Raw Output: {"message": "[Microsoft.Contractions] Use 'won't' instead of 'will not'.", "location": {"path": "docs/repositories/pull-requests.md", "range": {"start": {"line": 63, "column": 69}}}, "severity": "ERROR"}

Check failure on line 63 in docs/repositories/pull-requests.md

View workflow job for this annotation

GitHub Actions / vale

[vale] reported by reviewdog 🐶 [Microsoft.Contractions] Use 'aren't' instead of 'are not'. Raw Output: {"message": "[Microsoft.Contractions] Use 'aren't' instead of 'are not'.", "location": {"path": "docs/repositories/pull-requests.md", "range": {"start": {"line": 63, "column": 19}}}, "severity": "ERROR"}

![Coverage tab](images/pull-requests-tab-coverage.png)

## Commits tab

The **Commits** tab displays an overview of each commit included in the pull request, such as the analysis status and the number of issues introduced.
Expand Down
Loading