[llvm-branch-commits] [CI] Make premerge advisor exit with code 0 if failures are explained (PR #172394)

[David Spickett via llvm-branch-commits]

================
@@ -263,7 +283,11 @@ def plural(num_tests):
         # No tests failed but the build was in a failed state. Bring this to the user's
         # attention.
         ninja_failures = find_failure_in_ninja_logs(ninja_logs)
+        all_failures_explained &= are_all_failures_explained(
+            ninja_failures, failure_explanations
+        )
----------------
DavidSpickett wrote:

This could go in the `else` block of `if not ninja_failures`. Then each arm of the if has 1 line setting of all_failures_explained.

https://github.com/llvm/llvm-project/pull/172394