Skip to content

Commit 79c33ba

Browse files
authored
Clarification/context
Some added context and clarification around the definitions of green/amber
1 parent bcbacc8 commit 79c33ba

File tree

1 file changed

+3
-1
lines changed

1 file changed

+3
-1
lines changed

quality-checks.md

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -26,10 +26,12 @@ All of these checks are important, even where their purpose overlaps with other
2626
We rate our applications against each of these checks as follows:
2727

2828
* Green = the quality check is applied frequently and consistently (in practice this typically means [automated](./patterns/automate-everything.md) via [continuous integration / continuous deployment](./practices/continuous-integration.md)), the output of the check is a quality gate (as opposed to just a warning / for information), and the tolerances for that quality gate (e.g. code coverage %) are agreed and understood
29-
* Amber = the quality check is applied, but not all conditions for green are met - for example the check generates warnings that may or may not be acted on
29+
* Amber = the quality check is applied, but not all conditions for green are met - for example the check generates warnings that may or may not be acted on, or is executed on an ad-hoc basis and cannot offer a consistent quality gate guarantee
3030
* Red = the quality check is not applied
3131
* N/A = the quality check is not applicable
3232

33+
There may be further nuances to these definitions, and the sections detailing each check can expand considerably on the criteria as required. Everyone is encouraged to contribute to those, and discuss how they can be best be tailored.
34+
3335
## Tracking progress
3436

3537
We recommend tracking progress on an Engineering Quality dashboard, for example:

0 commit comments

Comments
 (0)