Merge pull request #5 from sensein/dev

workflow_dispatch #2
sensein · Sep 18, 2024 · b0f61cf · b0f61cf
2 parents 0f5ab9c + 6e2c710
commit b0f61cf
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 5 deletions.
diff --git a/.github/workflows/macos_tests.yaml b/.github/workflows/macos_tests.yaml
@@ -32,3 +32,5 @@ jobs:
         shell: bash
       - name: Run unit tests
         run: poetry run pytest
+
+
diff --git a/.github/workflows/ubuntu_tests.yaml b/.github/workflows/ubuntu_tests.yaml
@@ -6,11 +6,15 @@ on:
     types:
       - completed
 
+# Define the matrix at the top level
 jobs:
   start-runner:
-    # if: ${{ github.event.workflow_run.conclusion == 'success' }} # Trigger only if macOS tests succeed
+    if: ${{ github.event.workflow_run.conclusion == 'success' }} # Trigger only if macOS tests succeed
     name: start-runner
     runs-on: ubuntu-latest
+    strategy:
+      matrix:
+        python-version: ['3.10', '3.11']  # Define matrix at the top level
     outputs:
       label: ${{ steps.start-ec2-runner.outputs.label }}
       ec2-instance-id: ${{ steps.start-ec2-runner.outputs.ec2-instance-id }}
@@ -36,9 +40,6 @@ jobs:
     name: ubuntu-tests
     needs: start-runner
     runs-on: ${{ needs.start-runner.outputs.label }}
-    strategy:
-      matrix:
-        python-version: ['3.10', '3.11']
     env:
       POETRY_CACHE_DIR: ${{ vars.POETRY_CACHE_DIR }}
     steps:
@@ -78,7 +79,7 @@ jobs:
       - name: Run unit tests
         run: poetry run pytest
         shell: bash
-      - name: Delete poetry env with python {{ matrix.python-version }}
+      - name: Delete poetry env with python ${{ matrix.python-version }}
         run: |
           poetry env remove ${{ matrix.python-version }}
         shell: bash