Skip to content

Commit e352d63

Browse files
authored
Update quality-checks.md (#121)
1 parent a9d1a61 commit e352d63

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

quality-checks.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -49,7 +49,7 @@ We rate our applications against each of these checks as follows:
4949
| Chaos tests | Resilience | Contextual | | | | | |
5050
| Code coverage | Maintainability | Universal | The proportion of the application code which is executed (in this context: during testing) | The higher the code coverage, the more thorough the testing, and therefore the higher the likelihood of detecting functional issues early | For new code: 70% | | |
5151
| Duplicate code scan | Maintainability | Universal | | | | | |
52-
| Code smells scan | Maintainability | Universal | Adhere to the principles, patterns and practices of writing clean code | The code is written once but read and executed many times. Without paying attention to the fundamental code quality checks maintainability, complexity and therefore the cost of the service increase exponentially with time, adding up to the cumulative technical debt | | [SonarQube](https://www.sonarqube.org/), [SonarCloud](https://sonarcloud.io/) | [Clean Code: Smells and Heuristics](https://moderatemisbehaviour.github.io/clean-code-smells-and-heuristics/) |
52+
| Code smells scan | Maintainability | Universal | Check whether the software code adheres to the principles, patterns and practices of writing clean code | The code is written once but read and executed many times. If the code is not clean, the cost and risk of making software changes both increase exponentially over time | | [SonarQube](https://www.sonarqube.org/), [SonarCloud](https://sonarcloud.io/) | [Clean Code: Smells and Heuristics](https://moderatemisbehaviour.github.io/clean-code-smells-and-heuristics/) |
5353
| Dead code scan | Maintainability | Universal | | | | | |
5454
| Code review | Other | Universal | A second person manually checking a code change | Quality check by a human, as opposed to via a tool | Enforced & audited step within workflow | TBC | [Code review guidance](./patterns/everything-as-code.md#code-review) |
5555
| Accessibility tests | Other | Universal | | | | | |

0 commit comments

Comments
 (0)