Skip to content

Commit b303973

Browse files
committed
Remove length text and use passive to avoid other issues
1 parent 3b845e2 commit b303973

File tree

1 file changed

+4
-8
lines changed

1 file changed

+4
-8
lines changed

doc/user_guide/features/metrics/sonar.rst

Lines changed: 4 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -32,10 +32,8 @@ A GitHub Admin will need to:
3232
#. **Post-merge**: update the branch protections to include SonarQube analysis.
3333

3434
* This should only be done when tests exist for the project, & that the project is
35-
at a state in which enforced code coverage would not be a burden. For new projects,
36-
we recommend creating an issue to add the SonarQube analysis to the branch protections
37-
at a later point. In such scenarios, SonarQube analysis will still report its analysis
38-
results to the PR, but it will not prevent the PR from being merged.
35+
at a state in which enforced code coverage would not be a burden. If you do
36+
not enact branch protections, it is recommended to create an issue to do so later.
3937

4038
In Sonar
4139
""""""""
@@ -76,10 +74,8 @@ A GitHub Admin will need to:
7674
#. **Post-merge**: update the branch protections to include SonarQube analysis.
7775

7876
* This should only be done when tests exist for the project, & that the project is
79-
at a state in which enforced code coverage would not be a burden. For new projects,
80-
we recommend creating an issue to add the SonarQube analysis to the branch protections
81-
at a later point. In such scenarios, SonarQube analysis will still report its analysis
82-
results to the PR, but it will not prevent the PR from being merged.
77+
at a state in which enforced code coverage would not be a burden. If you do
78+
not enact branch protections, it is recommended to create an issue to do so later.
8379

8480
In Sonar
8581
""""""""

0 commit comments

Comments
 (0)