Remove publish tag again (#63)

* remove publish tag again * try only running on a merge when the pull request is closed
DitchOoM · May 12, 2024 · 5c44d5b · 5c44d5b
1 parent 0c13e24
commit 5c44d5b
Showing 1 changed file with 6 additions and 2 deletions.
diff --git a/.github/workflows/merged.yaml b/.github/workflows/merged.yaml
@@ -1,14 +1,17 @@
 name: "Build Test and Deploy"
 on:
-  push:
+  pull_request_target:
     paths-ignore:
       - '*.md'
     branches: [ main ]
+    types:
+      - closed
 env:
   username: ${{ secrets.SONATYPE_NEXUS_USERNAME }}
   password: ${{ secrets.SONATYPE_NEXUS_PASSWORD }}
 jobs:
   create-staging-repository:
+    if: github.event.pull_request.merged == true
     name: Create staging repository
     runs-on: ubuntu-latest
     outputs:
@@ -30,6 +33,7 @@ jobs:
           fi
   macos:
     name: MacOS Build & Upload artifacts to sonatype
+    if: github.event.pull_request.merged == true
     needs: create-staging-repository
     runs-on: macOS-latest
     env:
@@ -93,7 +97,7 @@ jobs:
   finalize:
     runs-on: ubuntu-latest
     needs: [create-staging-repository, macos]
-    if: ${{ always() && needs.create-staging-repository.result == 'success' }}
+    if: ${{ always() && needs.create-staging-repository.result == 'success' && github.event.pull_request.merged == true }}
     steps:
       - name: Discard
         if: ${{ needs.macos.result != 'success' }}