Merge pull request #83 from muonsoft/docker-tags-fix

fix: docker semver tags

.github/workflows/release-docker.yml

@@ -20,6 +20,9 @@ jobs:
  type=schedule
  type=ref,event=branch
  type=ref,event=pr
+ type=semver,pattern={{version}}
+ type=semver,pattern={{major}}.{{minor}}
+ type=semver,pattern={{major}}
  type=sha
  - name: Set APP_VERSION env
  run: echo "APP_VERSION=$(echo ${GITHUB_REF} | rev | cut -d'/' -f 1 | rev )" >> $GITHUB_ENV
@@ -32,7 +35,8 @@ jobs:
  - name: Set up Docker Buildx
  uses: docker/setup-buildx-action@v2
  - name: Login to DockerHub
- uses: docker/login-action@v2 
+ if: github.event_name != 'pull_request'
+ uses: docker/login-action@v2
  with:
  username: ${{ secrets.DOCKER_USERNAME }}
  password: ${{ secrets.DOCKER_PASSWORD }}
@@ -41,7 +45,7 @@ jobs:
  with:
  context: .
  platforms: linux/amd64,linux/arm64
- push: true
+ push: ${{ github.event_name != 'pull_request' }}
  pull: true
  build-args: |
  APP_VERSION=${{ env.APP_VERSION }}

Dockerfile

@@ -3,7 +3,8 @@ FROM --platform=$BUILDPLATFORM golang:alpine AS build-env
 
 ARG APP_VERSION=""
 ARG BUILD_TIME=""
-ARG TARGETOS TARGETARCH
+ARG TARGETOS
+ARG TARGETARCH
 
 ADD . /project