Merge pull request #40 from Moros1138/fix-workflows

god damnit
Moros1138 · May 2, 2024 · e26cd91 · e26cd91
2 parents 97debc1 + 53efa5c
commit e26cd91
Showing 1 changed file with 5 additions and 2 deletions.
diff --git a/.github/workflows/test-on-push-main.yml b/.github/workflows/test-on-push-main.yml
@@ -17,7 +17,7 @@ jobs:
         run: npm install && npm run build && composer install --prefer-dist --no-interaction --no-progress && php artisan dusk:chrome-driver --detect
 
       - 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=""
+        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)"
 
       - name: Build Library Objects
         run: bash build-libs.sh
@@ -57,4 +57,7 @@ jobs:
         uses: docker/build-push-action@v5
         with:
           push: true
-          tags: moros1138/pgetinker:latest,moros1138/pgetinker:$PGETINKER_VERSION
+          tags: |
+            ${{ secrets.DOCKERHUB_USERNAME }}/pgetinker:${{ GITHUB_SHA }}
+            ${{ secrets.DOCKERHUB_USERNAME }}/pgetinker:latest
+