Merge pull request #193 from AlexVCaron/fix/big-runner-easyreg

matrix strategy to not run easyreg when absent
scilus · Aug 26, 2024 · 0b4e145 · 0b4e145
2 parents e1d4476 + 3f0d0ef
commit 0b4e145
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 4 deletions.
diff --git a/.github/workflows/checks.yml b/.github/workflows/checks.yml
@@ -121,13 +121,15 @@ jobs:
       fail-fast: false
       matrix:
         path: [ "${{ fromJson(needs.nf-test-changes.outputs.paths) }}" ]
+        profile: [ "docker" ]
+        exclude:
+          - path: subworkflows/nf-scil/load_test_data
         include:
-          - profile: docker
           - runner: scilus-nf-scil-runners
+          - path: modules/nf-scil/registration/easyreg
+            profile: do-not-run
           - runner: scilus-nf-scil-bigmem-runners
             path: modules/nf-scil/registration/easyreg
-        exclude:
-          - path: subworkflows/nf-scil/load_test_data
     uses: ./.github/workflows/nf-test_module.yml
     with:
       profile: ${{ matrix.profile }}

diff --git a/.github/workflows/nf-test_module.yml b/.github/workflows/nf-test_module.yml
@@ -45,7 +45,7 @@ jobs:
   nf-test:
     runs-on: ${{ inputs.runner }}
     name: nf-test-${{ inputs.paths }}
-    if: inputs.paths != ''
+    if: inputs.paths != '' && inputs.profile != 'do-not-run'
     env:
       NXF_ANSI_LOG: false
       NFTEST_VER: "0.9.0-rc1"