Improving check to run workflows

diemol · diemol · commit 668d131529cc · 2022-03-03T11:43:10.000+01:00
Aside from checking bazel targets,
we also need to check if there are
any test targets we need to run.

For example, a change in Grid needs
to trigger remote tests in all bindings.
diff --git a/scripts/github-actions/should-workflow-run.sh b/scripts/github-actions/should-workflow-run.sh
@@ -31,8 +31,19 @@ fi
 
 if [[ " ${bazel_targets[*]} " == *"$BAZEL_TARGET_PREFIX"* ]]; then
   echo "::set-output name=run-workflow::true"
-  echo "Bazel targets found: ${bazel_targets[*]}."
+  echo "Bazel targets found: ${bazel_targets[*]}"
+  exit 0
 fi
 
+# Now check if we need to run some tests based on this change
+# E.g. A change in Grid needs to trigger remote tests in other bindings
+tests=$(bazel query \
+    --keep_going \
+    --noshow_progress \
+    "kind(test, rdeps(//..., set(${bazel_targets[*]})))")
 
-
+if [[ " ${tests[*]} " == *"$BAZEL_TARGET_PREFIX"* ]]; then
+  echo "::set-output name=run-workflow::true"
+  echo "Test targets found: ${tests[*]}"
+  exit 0
+fi
