Use non-zero exit code when printing buildozer commands

Author: DavidANeil

src/main/scala/higherkindness/rules_scala/workers/deps/BUILD

@@ -9,6 +9,7 @@ scala_binary(
     visibility = ["//visibility:public"],
     deps = [
         "//src/main/scala/higherkindness/rules_scala/common/args",
+        "//src/main/scala/higherkindness/rules_scala/common/error",
         "//src/main/scala/higherkindness/rules_scala/common/interrupt",
         "//src/main/scala/higherkindness/rules_scala/common/sandbox",
         "//src/main/scala/higherkindness/rules_scala/common/worker",

src/main/scala/higherkindness/rules_scala/workers/deps/DepsRunner.scala

@@ -4,9 +4,10 @@ package workers.deps
 import common.args.ArgsUtil
 import common.args.ArgsUtil.PathArgumentType
 import common.args.implicits.*
+import common.error.AnnexWorkerError
 import common.interrupt.InterruptUtil
-import common.worker.WorkerMain
 import common.sandbox.SandboxUtil
+import common.worker.WorkerMain
 import workers.common.AnnexMapper
 import workers.common.FileUtil
 import java.io.{File, PrintStream}
@@ -163,11 +164,6 @@ object DepsRunner extends WorkerMain[Unit] {
     } else {
       Nil
     }
-    labelsToRemove.foreach { depLabel =>
-      out.println(s"Target '$depLabel' not used, please remove it from the deps.")
-      out.println(s"You can use the following buildozer command:")
-      out.println(s"buildozer 'remove deps $depLabel' ${workRequest.label}")
-    }
 
     InterruptUtil.throwIfInterrupted()
     val labelsToAdd = if (workRequest.checkDirect) {
@@ -182,15 +178,19 @@ object DepsRunner extends WorkerMain[Unit] {
     } else {
       Nil
     }
-    labelsToAdd.foreach { depLabel =>
-      out.println(s"Target '$depLabel' is used but isn't explicitly declared, please add it to the deps.")
-      out.println(s"You can use the following buildozer command:")
-      out.println(s"buildozer 'add deps $depLabel' ${workRequest.label}")
-    }
 
     if (labelsToAdd.isEmpty && labelsToRemove.isEmpty) {
       try Files.createFile(workRequest.successFile)
       catch { case _: FileAlreadyExistsException => }
+    } else {
+      throw new AnnexWorkerError(
+        1,
+        (labelsToRemove.map { depLabel =>
+          s"Target '$depLabel' not used, please remove it from the deps.\nYou can use the following buildozer command:\nbuildozer 'remove deps $depLabel' ${workRequest.label}\n"
+        } ++ labelsToAdd.map { depLabel =>
+          s"Target '$depLabel' is used but isn't explicitly declared, please add it to the deps.\nYou can use the following buildozer command:\nbuildozer 'add deps $depLabel' ${workRequest.label}\n"
+        }).mkString("\n"),
+      )
     }
     InterruptUtil.throwIfInterrupted()
   }