Merge pull request #41 from Moros1138/fix-workflows

more attempts at fixing the workflow
Moros1138 · May 2, 2024 · 68da7d7 · 68da7d7
2 parents e26cd91 + 1c3265f
commit 68da7d7
Showing 1 changed file with 6 additions and 2 deletions.
diff --git a/.github/workflows/test-on-push-main.yml b/.github/workflows/test-on-push-main.yml
@@ -16,8 +16,12 @@ jobs:
       - name: Install Dependencies
         run: npm install && npm run build && composer install --prefer-dist --no-interaction --no-progress && php artisan dusk:chrome-driver --detect
 
+      - name: Get Short Hash
+        id: vars
+        run: echo "sha=$(git rev-parse HEAD)" >> $GITHUB_OUTPUT
+
       - name: Copy environment file and set version
-        run: cp .env.example .env && echo "VERSION=$(git rev-parse HEAD)" >> .env && echo "VITE_VERSION=$(git rev-parse HEAD)" >> .env && export PGETINKER_VERSION="$(git rev-parse HEAD)"
+        run: cp .env.example .env && echo "VERSION=${{ steps.vars.outputs.sha }}" >> .env && echo "VITE_VERSION=${{ steps.vars.outputs.sha }}" >> .env
 
       - name: Build Library Objects
         run: bash build-libs.sh
@@ -58,6 +62,6 @@ jobs:
         with:
           push: true
           tags: |
-            ${{ secrets.DOCKERHUB_USERNAME }}/pgetinker:${{ GITHUB_SHA }}
+            ${{ secrets.DOCKERHUB_USERNAME }}/pgetinker:${{ steps.vars.outputs.sha }}
             ${{ secrets.DOCKERHUB_USERNAME }}/pgetinker:latest