Correctly handle --keep_going (#279)

* Correctly handle --keep_going

I will work on making this logic more testable soon

Fixes fineGrainedHashExternalRepos parameter issue #267

Signed-off-by: Maxwell Elliott <[email protected]>

* rollback

Signed-off-by: Maxwell Elliott <[email protected]>

* more updates

Signed-off-by: Maxwell Elliott <[email protected]>

---------

Signed-off-by: Maxwell Elliott <[email protected]>

Author: tinder-maxwellelliott

MODULE.bazel

@@ -1,6 +1,6 @@
 module(
     name = "bazel-diff",
-    version = "9.1.0",
+    version = "10.0.0",
     compatibility_level = 0,
 )
 

MODULE.bazel.lock

@@ -130,6 +130,8 @@ class BazelQueryService(
 
     queryFile.writeText(query)
 
+    val allowedExitCodes = mutableListOf(0)
+
     val cmd: MutableList<String> =
         ArrayList<String>().apply {
           add(bazelPath.toString())
@@ -179,6 +181,7 @@ class BazelQueryService(
           }
           if (keepGoing) {
             add("--keep_going")
+            allowedExitCodes.add(3)
           }
           if (useCquery) {
             addAll(cqueryOptions)
@@ -204,8 +207,10 @@ class BazelQueryService(
           destroyForcibly = true,
       )
 
-    if (result.resultCode != 0)
-        throw RuntimeException("Bazel query failed, exit code ${result.resultCode}")
+    if (!allowedExitCodes.contains(result.resultCode)) {
+        logger.w { "Bazel query failed, output: ${result.output.joinToString("\n")}" }
+        throw RuntimeException("Bazel query failed, exit code ${result.resultCode}, allowed exit codes: ${allowedExitCodes.joinToString()}")
+    }
     return outputFile
   }