build(docker): fix docker release tags (#1723)

Signed-off-by: Stefan Dej <[email protected]>

.github/workflows/publish_docker.yml

@@ -2,7 +2,8 @@ name: Create and publish a Docker image
 
 on:
   push:
-    branches: ['master', 'develop']
+    branches:
+      - 'develop'
     tags:
       - 'v*'
   pull_request:
@@ -25,6 +26,7 @@ jobs:
         uses: actions/checkout@v4
 
       - name: Log in to the Container registry
+        if: github.event_name != 'pull_request'
         uses: docker/login-action@v3
         with:
           registry: ${{ env.REGISTRY }}
@@ -37,11 +39,12 @@ jobs:
         with:
           images: ${{ env.REGISTRY }}/${{ env.IMAGE_NAME }}
           tags: |
-            type=raw,value=latest,enable=${{ github.ref == 'refs/heads/master' }}
             type=edge,branch=develop
             type=semver,pattern=v{{major}}
             type=semver,pattern=v{{major}}.{{minor}}
             type=semver,pattern=v{{major}}.{{minor}}.{{patch}}
+            type=sha
+
       - name: Set up QEMU
         uses: docker/setup-qemu-action@v3
 

.github/workflows/release.yml

@@ -12,7 +12,7 @@ jobs:
   push-version-number:
     runs-on: ubuntu-latest
     steps:
-      - name: Clone develop repository
+      - name: Checkout develop branch
         uses: actions/checkout@v4
         with:
           token: ${{ secrets.PAT }}
@@ -50,6 +50,7 @@ jobs:
 
   release:
     needs: ['update-master-branch']
+    name: Build and release
     runs-on: ubuntu-latest
     steps:
       - name: Fetch repo
@@ -92,7 +93,7 @@ jobs:
         with:
           name: v${{ github.event.inputs.version }}
           tag_name: v${{ github.event.inputs.version }}
-          token: ${{ secrets.GITHUB_TOKEN }}
+          token: ${{ secrets.PAT }}
           files: ./mainsail.zip
           body: ${{ steps.generate-changelog.outputs.content }}
 

Dockerfile

@@ -8,7 +8,7 @@ RUN apk add zip
 WORKDIR /app
 COPY package*.json /app/
 
-RUN npm install
+RUN npm ci
 
 COPY ./ /app/