Fix workflow_dispatch bypass: force evaluation of skipped dependency conditions (#324)

* Initial plan

* Fix workflow_dispatch detection bypass in all check workflows

Add always() to job conditions that check for skipped detect-changes.
This forces GitHub Actions to evaluate the condition even when the
detect-changes job is skipped (e.g., during workflow_dispatch).

Without always(), GitHub Actions skips jobs before evaluating their
if conditions when a needed job is skipped.

Fixes workflow_dispatch invocation for:
- coverage workflow
- python-check workflow
- cmake-build workflow
- clang-format-check workflow
- clang-tidy-check workflow
- cmake-format-check workflow
- actionlint-check workflow
- jsonnet-format-check workflow
- markdown-check workflow

Co-authored-by: greenc-FNAL <2372949+greenc-FNAL@users.noreply.github.com>

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: greenc-FNAL <2372949+greenc-FNAL@users.noreply.github.com>

Author: Copilot

.github/workflows/actionlint-check.yaml

@@ -107,10 +107,13 @@ jobs:
   actionlint-check:
     needs: [pre-check, detect-changes]
     if: >
-      needs.detect-changes.result == 'skipped' ||
+      always() &&
       (
-        needs.detect-changes.result == 'success' &&
-        needs.detect-changes.outputs.has_changes == 'true'
+        needs.detect-changes.result == 'skipped' ||
+        (
+          needs.detect-changes.result == 'success' &&
+          needs.detect-changes.outputs.has_changes == 'true'
+        )
       )
     runs-on: ubuntu-latest
     permissions:

.github/workflows/clang-format-check.yaml

@@ -69,10 +69,13 @@ jobs:
   clang-format-check:
     needs: [pre-check, detect-changes]
     if: >
-      needs.detect-changes.result == 'skipped' ||
+      always() &&
       (
-        needs.detect-changes.result == 'success' &&
-        needs.detect-changes.outputs.has_changes == 'true'
+        needs.detect-changes.result == 'skipped' ||
+        (
+          needs.detect-changes.result == 'success' &&
+          needs.detect-changes.outputs.has_changes == 'true'
+        )
       )
     runs-on: ubuntu-latest
     permissions:

.github/workflows/clang-tidy-check.yaml

@@ -74,10 +74,13 @@ jobs:
   clang-tidy-check:
     needs: [pre-check, detect-changes]
     if: >
-      needs.detect-changes.result == 'skipped' ||
+      always() &&
       (
-        needs.detect-changes.result == 'success' &&
-        needs.detect-changes.outputs.has_changes == 'true'
+        needs.detect-changes.result == 'skipped' ||
+        (
+          needs.detect-changes.result == 'success' &&
+          needs.detect-changes.outputs.has_changes == 'true'
+        )
       )
     runs-on: ubuntu-24.04
 

.github/workflows/cmake-build.yaml

@@ -174,6 +174,7 @@ jobs:
   build:
     needs: [pre-check, detect-changes, generate-matrix]
     if: >
+      always() &&
       needs.pre-check.result == 'success' &&
       (
         needs.detect-changes.result == 'skipped' ||

.github/workflows/cmake-format-check.yaml

@@ -103,10 +103,13 @@ jobs:
   cmake-format-check:
     needs: [pre-check, detect-changes]
     if: >
-      needs.detect-changes.result == 'skipped' ||
+      always() &&
       (
-        needs.detect-changes.result == 'success' &&
-        needs.detect-changes.outputs.has_changes == 'true'
+        needs.detect-changes.result == 'skipped' ||
+        (
+          needs.detect-changes.result == 'success' &&
+          needs.detect-changes.outputs.has_changes == 'true'
+        )
       )
     runs-on: ubuntu-latest
 

.github/workflows/coverage.yaml

@@ -84,10 +84,13 @@ jobs:
   coverage:
     needs: [pre-check, detect-changes]
     if: >
-      needs.detect-changes.result == 'skipped' ||
+      always() &&
       (
-        needs.detect-changes.result == 'success' &&
-        needs.detect-changes.outputs.has_changes == 'true'
+        needs.detect-changes.result == 'skipped' ||
+        (
+          needs.detect-changes.result == 'success' &&
+          needs.detect-changes.outputs.has_changes == 'true'
+        )
       )
     runs-on: ubuntu-24.04
 

.github/workflows/jsonnet-format-check.yaml

@@ -104,10 +104,13 @@ jobs:
   jsonnet-format-check:
     needs: [pre-check, detect-changes]
     if: >
-      needs.detect-changes.result == 'skipped' ||
+      always() &&
       (
-        needs.detect-changes.result == 'success' &&
-        needs.detect-changes.outputs.has_changes == 'true'
+        needs.detect-changes.result == 'skipped' ||
+        (
+          needs.detect-changes.result == 'success' &&
+          needs.detect-changes.outputs.has_changes == 'true'
+        )
       )
     runs-on: ubuntu-latest
     container:

.github/workflows/markdown-check.yaml

@@ -104,10 +104,13 @@ jobs:
   markdown-check:
     needs: [pre-check, detect-changes]
     if: >
-      needs.detect-changes.result == 'skipped' ||
+      always() &&
       (
-        needs.detect-changes.result == 'success' &&
-        needs.detect-changes.outputs.has_changes == 'true'
+        needs.detect-changes.result == 'skipped' ||
+        (
+          needs.detect-changes.result == 'success' &&
+          needs.detect-changes.outputs.has_changes == 'true'
+        )
       )
     runs-on: ubuntu-latest
     permissions:

.github/workflows/python-check.yaml

@@ -104,10 +104,13 @@ jobs:
   python-check:
     needs: [pre-check, detect-changes]
     if: >
-      needs.detect-changes.result == 'skipped' ||
+      always() &&
       (
-        needs.detect-changes.result == 'success' &&
-        needs.detect-changes.outputs.has_changes == 'true'
+        needs.detect-changes.result == 'skipped' ||
+        (
+          needs.detect-changes.result == 'success' &&
+          needs.detect-changes.outputs.has_changes == 'true'
+        )
       )
     runs-on: ubuntu-latest
     permissions: