Update action.yml

jagapiou · Sep 12, 2024 · ef9f110 · ef9f110
1 parent 6e1cf5f
commit ef9f110
Showing 1 changed file with 11 additions and 16 deletions.
diff --git a/.github/actions/install-concordia/action.yml b/.github/actions/install-concordia/action.yml
@@ -15,31 +15,26 @@ outputs:
 runs:
   using: composite
   steps:
-    - name: Get current runner
-      id: os-info
-      shell: bash
-      run: |
-        if [ "${RUNNER_OS}" = 'macOS' ]; then
-          echo "name=$(sw_vers -productName)" >> $GITHUB_OUTPUT
-          echo "version=$(sw_vers -productVersion)" >> $GITHUB_OUTPUT
-        elif [ "${RUNNER_OS}" = 'Linux' ]; then
-          echo "name=$(lsb_release -i -s)" >> $GITHUB_OUTPUT
-          echo "version=$(lsb_release -r -s)" >> $GITHUB_OUTPUT
-        else
-          exit 1
-        fi
-
     - name: Set up Python ${{ inputs.python-version }}
       id: setup-python
       uses: actions/setup-python@f677139bbe7f9c59b41e40162b753c062f5d49a3
       with:
         python-version: ${{ inputs.python-version }}
-
+        
     - name: Get cache key
       id: cache-key
       shell: bash
       run: |
-         echo "cache-key=${{ steps.os-info.outputs.name }}-${{ steps.os-info.outputs.version }}-py${{ steps.setup-python.outputs.python-version }}-${{ hashFiles('setup.py') }}" >> $GITHUB_OUTPUT
+        if [ "${RUNNER_OS}" = 'macOS' ]; then
+          readonly name="$(sw_vers -productName)"
+          readonly version="$(sw_vers -productVersion)"
+        elif [ "${RUNNER_OS}" = 'Linux' ]; then
+          readonly name="$(lsb_release -i -s)"
+          readonly version="$(lsb_release -r -s)"
+        else
+          exit 1
+        fi
+        echo "cache-key=${name}-${version}-py${{ steps.setup-python.outputs.python-version }}-${{ hashFiles('setup.py') }}" >> $GITHUB_OUTPUT
 
     - name: Restore Concordia installation
       id: restore