Merge pull request #8649 from OpenMined/post-release-workflow

force parse to string vars evaluated from github inputs
OpenMined · Apr 2, 2024 · da6f7a6 · da6f7a6
2 parents 8c2c819 + be0c135
commit da6f7a6
Showing 1 changed file with 8 additions and 8 deletions.
diff --git a/.github/workflows/cd-post-release-tests.yml b/.github/workflows/cd-post-release-tests.yml
@@ -170,11 +170,11 @@ jobs:
 
       - name: Install Syft
         run: |
-          if (${{ inputs.release_platform }} == "REAL_PYPI"); then
-            pip install syft==${{ inputs.syft_version }}
+          if [[ ${{inputs.release_platform}} == "REAL_PYPI" ]]; then
+            pip install syft==${{inputs.syft_version}}
           fi
-          if (${{ inputs.release_platform }}" == "TEST_PYPI"); then
-            pip install -i https://test.pypi.org/simple/ syft==${{ inputs.syft_version }}
+          if [[ ${{ inputs.release_platform }} == "TEST_PYPI" ]]; then
+            pip install -i https://test.pypi.org/simple/ syft==${{inputs.syft_version}}
           fi
 
       - name: Check Syft version
@@ -282,11 +282,11 @@ jobs:
         env:
           SYFT_VERSION: ${{ inputs.syft_version }}
         run: |
-          if [[ "${{ inputs.release_platform }}" == "TEST_PYPI" ]]; then
-            pip install -i https://test.pypi.org/simple/ syft==${{ inputs.syft_version }}[data_science];
+          if [[ ${{ inputs.release_platform }} == "TEST_PYPI" ]]; then
+            pip install -i https://test.pypi.org/simple/ syft[data_science,dev]==${{ inputs.syft_version }}
           fi
-          if [[ "${{ inputs.release_platform }}" == "REAL_PYPI" ]]; then
-            pip install syft==${{ inputs.syft_version }}[data_science];
+          if [[ ${{ inputs.release_platform }} == "REAL_PYPI" ]]; then
+            pip install syft[data_science,dev]==${{ inputs.syft_version }}
           fi
 
       - name: Install Hagrid, tox and uv