Add comment in code

Flank · Aug 3, 2020 · c9926e9 · c9926e9
1 parent 818e591
commit c9926e9
Showing 1 changed file with 5 additions and 3 deletions.
diff --git a/test_runner/src/main/kotlin/ftl/json/SavedMatrix.kt b/test_runner/src/main/kotlin/ftl/json/SavedMatrix.kt
@@ -148,9 +148,11 @@ class SavedMatrix(matrix: TestMatrix) {
         flakyOutcome: Boolean
     ) {
         outcome = when {
-            // the matrix outcome is failure if any step fails
-            // if the matrix outcome is already set to failure then we can ignore the other step outcomes.
-            // inconclusive is treated as a failure
+            // The matrix outcome is failure if any step fails
+            // If the matrix outcome is already set to failure then we can ignore the other step outcomes.
+            // Inconclusive is treated as a failure
+            // This particular conditions order should not be changed, for more details check:
+            // https://github.com/Flank/flank/issues/914
             outcome == failure || outcome == inconclusive -> return
             flakyOutcome -> flaky
             outcome == flaky -> this?.summary?.takeIf { it == failure || it == inconclusive }