Skip to content

Commit 36a29b0

Browse files
Update insights/review.md
Co-authored-by: Alex Young <[email protected]>
1 parent c5f2130 commit 36a29b0

File tree

1 file changed

+1
-0
lines changed

1 file changed

+1
-0
lines changed

insights/review.md

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -168,6 +168,7 @@ You may wish to score each individual component or system separately for these a
168168

169169
- We have great test coverage.
170170
- Testing is everyone's responsibility and test is a first-class concern.
171+
- A failing test suite in CI gets immediate attention.
171172
- We support all team members to practice good testing, including by holding no-blame sessions to discuss any automation tests we should have added, and what we can learn from having missed them initially.
172173
- We build code for testability.
173174
- Tests (including both test code and test coverage & whether there are gaps) are part of our standard peer-review process.

0 commit comments

Comments
 (0)