Skip to content

Commit 1d93ace

Browse files
committed
rewrite some things
1 parent 7e40286 commit 1d93ace

File tree

1 file changed

+10
-7
lines changed

1 file changed

+10
-7
lines changed

docs/organizations/codacy-usage.md

Lines changed: 10 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -49,18 +49,21 @@ This chart tracks the total volume of pull requests merged across your Codacy-ma
4949

5050
If you see high numbers here, it usually boils down to two scenarios:
5151

52-
1. A developer merged the PR immediately after opening it, before the Codacy analysis could finish.
53-
2. In self-service organizations, if a PR is authored by an email/user not yet added to your Codacy seat count, the analysis may be skipped.
52+
1. A developer merged the PR immediately after opening it, before the Codacy analysis could finish. Ensure your team adjusts their workflow to wait for the "Codacy/Analysis" check to turn green before merging.
5453

55-
If you notice a high volume of PRs in Without gates, or Never analyzed categories, consider the following actions:
54+
2. In self-service organizations, if a PR is authored by an email/user not yet added to your Codacy seat count, the analysis may be skipped. You should check your [seat management](../organizations/managing-people.md) to confirm that all active contributors are correctly mapped to a Codacy seat.
5655

57-
#### Without gates:
56+
#### Why are some PRs merged with "Failing gates"?
5857

59-
To ensure your code meets organizational standards, you should define a Quality Gate rule for the repository, such as setting thresholds for "New issues > 0" or "Coverage < 80%".
58+
Pull requests are merged with failing Quality Gates usually due two reasons:
6059

61-
#### Never analyzed:
60+
1. Quality Gates are advisory, not blocking. [Protect your pull requests](../getting-started/integrating-codacy-with-your-git-workflow.md#blocking-pull-requests).
6261

63-
To ensure your team adjusts their workflow to wait for the "Codacy/Analysis" check to turn green before merging. Additionally, you should check your [seat management](../organizations/managing-people.md) to confirm that all active contributors are correctly mapped to a Codacy seat.
62+
2. Intentional exceptions or risk acceptance:
63+
64+
- Developers/ maintainers may override failing checks to address urgent fixes, unblock critical work, or accept a known risk temporarily.
65+
66+
- Sometimes pull requests are small and diff coverage / coverage variation may fail. Make sure your team follows the practices that were agreed upon within the organization.
6467

6568
Interacting with the chart allows you to filter the data, with the resulting repositories displayed in the "Top repositories" list.
6669

0 commit comments

Comments
 (0)