fix publish_commit to be triggered in the context of GJ repo and only on merged PRs

andimarek · andimarek · commit 1cb8d075aa4e · 2025-05-01T20:54:13.000+10:00
diff --git a/.github/workflows/publish_commit.yml b/.github/workflows/publish_commit.yml
@@ -1,6 +1,6 @@
 name: Publish Commit SHA for performance testing
 on:
-  pull_request:
+  pull_request_target:
     types:
       - closed
     branches:
@@ -12,10 +12,11 @@ permissions:
   contents: read  # This is required for actions/checkout
 jobs:
   publishCommit:
+    if: github.event.pull_request.merged == true
     runs-on: ubuntu-latest
     steps:
       - uses: aws-actions/configure-aws-credentials@v4
         with:
           role-to-assume: arn:aws:iam::637423498965:role/GitHubActionGrahQLJava
           aws-region: "ap-southeast-2"
-      - run: aws sns publish  --topic-arn "arn:aws:sns:ap-southeast-2:637423498965:graphql-java-commits.fifo" --message $GITHUB_SHA --message-group-id "graphql-java-commits"
+      - run: aws sns publish  --topic-arn "arn:aws:sns:ap-southeast-2:637423498965:graphql-java-commits.fifo" --message $GITHUB_SHA --message-group-id "graphql-java-commits"
