Reverted back unneeded changes to workflow

Signed-off-by: Theodor Mihalache <[email protected]>
tmihalac · Sep 23, 2024 · b8e66b0 · b8e66b0
1 parent f57d64f
commit b8e66b0
Show file tree

Hide file tree

Showing 3 changed files with 12 additions and 9 deletions.
diff --git a/.github/fork_workflows/fork_pr_integration_tests_aws.yml b/.github/fork_workflows/fork_pr_integration_tests_aws.yml
@@ -27,9 +27,10 @@ jobs:
     steps:
       - uses: actions/checkout@v4
         with:
-          repository: ${{ github.event.repository.full_name }}  # Uses the full repository name
-          ref: ${{ github.ref }}                                  # Uses the ref from the event
-          token: ${{ secrets.GITHUB_TOKEN }}                     # Automatically provided token
+          # pull_request_target runs the workflow in the context of the base repo
+          # as such actions/checkout needs to be explicit configured to retrieve
+          # code from the PR.
+          ref: refs/pull/${{ github.event.pull_request.number }}/merge
           submodules: recursive
       - name: Setup Python
         uses: actions/setup-python@v5

diff --git a/.github/fork_workflows/fork_pr_integration_tests_gcp.yml b/.github/fork_workflows/fork_pr_integration_tests_gcp.yml
@@ -27,9 +27,10 @@ jobs:
     steps:
       - uses: actions/checkout@v4
         with:
-          repository: ${{ github.event.repository.full_name }}  # Uses the full repository name
-          ref: ${{ github.ref }}                                  # Uses the ref from the event
-          token: ${{ secrets.GITHUB_TOKEN }}                     # Automatically provided token
+          # pull_request_target runs the workflow in the context of the base repo
+          # as such actions/checkout needs to be explicit configured to retrieve
+          # code from the PR.
+          ref: refs/pull/${{ github.event.pull_request.number }}/merge
           submodules: recursive
       - name: Setup Python
         uses: actions/setup-python@v5

diff --git a/.github/fork_workflows/fork_pr_integration_tests_snowflake.yml b/.github/fork_workflows/fork_pr_integration_tests_snowflake.yml
@@ -27,9 +27,10 @@ jobs:
     steps:
       - uses: actions/checkout@v4
         with:
-          repository: ${{ github.event.repository.full_name }}  # Uses the full repository name
-          ref: ${{ github.ref }}                                  # Uses the ref from the event
-          token: ${{ secrets.GITHUB_TOKEN }}                     # Automatically provided token
+          # pull_request_target runs the workflow in the context of the base repo
+          # as such actions/checkout needs to be explicit configured to retrieve
+          # code from the PR.
+          ref: refs/pull/${{ github.event.pull_request.number }}/merge
           submodules: recursive
       - name: Setup Python
         uses: actions/setup-python@v5