Merge pull request #263 from percy/fix-github-runner-config

Fixed sca issues
percy · Mar 5, 2024 · abaf3de · abaf3de
2 parents cd0291c + 7d522cc
commit abaf3de
Showing 1 changed file with 6 additions and 2 deletions.
diff --git a/.github/workflows/test.yml b/.github/workflows/test.yml
@@ -4,6 +4,7 @@ on:
   workflow_dispatch:
     inputs:
       branch:
+        description: Used to set which branch of CLI to use for current run
         required: false
         type: string
         default: master
@@ -40,16 +41,19 @@ jobs:
       - run: yarn
       - name: Set up @percy/cli from git
         if: ${{ github.event_name == 'workflow_dispatch' }}
+        env:
+          BRNACH: ${{ github.event.inputs.branch }}
+          WORKSPACE: ${{ github.workspace }} 
         run: |
           cd /tmp
-          git clone --branch ${{ github.event.inputs.branch }} --depth 1 https://github.com/percy/cli
+          git clone --branch "$BRANCH" --depth 1 https://github.com/percy/cli
           cd cli
           PERCY_PACKAGES=`find packages -mindepth 1 -maxdepth 1 -type d | sed -e 's/packages/@percy/g' | tr '\n' ' '`
           git log -1
           yarn
           yarn build
           yarn global:link
-          cd ${{ github.workspace }} 
+          cd "$WORKSPACE"
           yarn remove @percy/cli && yarn link `echo $PERCY_PACKAGES`
           npx percy --version
       - run: yarn test:coverage