properly fix docker push action

Author: bristermitten

.github/workflows/docker-check.yml

@@ -28,7 +28,6 @@ jobs:
   build:
     name: Build Docker image
     runs-on: ubuntu-latest
-    permissions: write-all
     needs: dockerfile-lint
     permissions:
       contents: read
@@ -53,6 +52,7 @@ jobs:
           provenance: false
 
       - name: Log in to the Container registry
+        if: github.event_name != 'pull_request'
         uses: docker/login-action@v3
         with:
           registry: ${{ env.REGISTRY }}
@@ -70,11 +70,15 @@ jobs:
         uses: docker/build-push-action@v6
         with:
           context: .
-          push: true
+          push: ${{ github.event_name != 'pull_request' }}
           tags: ${{ steps.meta.outputs.tags }}
           labels: ${{ steps.meta.outputs.labels }}
+          cache-from: type=gha
+          cache-to: type=gha,mode=max
+          
       - name: Generate artifact attestation
         uses: actions/attest-build-provenance@v3
+        if: github.event_name != 'pull_request'
         with:
           subject-name: ${{ env.REGISTRY }}/${{ env.IMAGE_NAME}}
           subject-digest: ${{ steps.push.outputs.digest }}