Skip to content

Commit 9698fd5

Browse files
committed
rewrite some things
1 parent 7e40286 commit 9698fd5

File tree

1 file changed

+11
-6
lines changed

1 file changed

+11
-6
lines changed

docs/organizations/codacy-usage.md

Lines changed: 11 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -49,18 +49,23 @@ 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.
53+
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.
5455

5556
If you notice a high volume of PRs in Without gates, or Never analyzed categories, consider the following actions:
5657

57-
#### Without gates:
58+
#### Why are some PRs merged with "Failing gates"?
59+
60+
Pull requests are merged with failing Quality Gates usually due two reasons:
61+
62+
1. Quality Gates are advisory, not blocking. [Protect your pull requests](../getting-started/integrating-codacy-with-your-git-workflow.md#blocking-pull-requests).
5863

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%".
64+
2. Intentional exceptions or risk acceptance:
6065

61-
#### Never analyzed:
66+
- Developers/ maintainers may override failing checks to address urgent fixes, unblock critical work, or accept a known risk temporarily.
6267

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.
68+
- 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.
6469

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

0 commit comments

Comments
 (0)