[GHA] Disable the e2e comment step

oneapi-src · Jun 10, 2024 · 6ea9078 · 6ea9078
1 parent 01f1f51
commit 6ea9078
Showing 1 changed file with 20 additions and 17 deletions.
diff --git a/.github/workflows/e2e_core.yml b/.github/workflows/e2e_core.yml
@@ -178,20 +178,23 @@ jobs:
       id: tests
       run: ninja -C build-e2e check-sycl-e2e
 
-    - name: Add comment to PR
-      uses: actions/github-script@60a0d83039c74a4aee543508d2ffcb1c3799cdea # v7.0.1
-      if: ${{ always() }}
-      with:
-        script: |
-          const adapter = '${{ matrix.adapter.name }}';
-          const url = '${{ github.server_url }}/${{ github.repository }}/actions/runs/${{ github.run_id }}';
-          const test_status = '${{ steps.tests.outcome }}';
-          const job_status = '${{ job.status }}';
-          const body = `E2E ${adapter} build:\n${url}\nJob status: ${job_status}. Test status: ${test_status}`;
-
-          github.rest.issues.createComment({
-            issue_number: context.issue.number,
-            owner: context.repo.owner,
-            repo: context.repo.repo,
-            body: body
-          })
+    # FIXME: Requires pull-request: write permissions but this is only granted
+    # on pull requests from forks if using pull_request_target workflow
+    # trigger but not the pull_request trigger..
+    # - name: Add comment to PR
+    #   uses: actions/github-script@60a0d83039c74a4aee543508d2ffcb1c3799cdea # v7.0.1
+    #   if: ${{ always() }}
+    #   with:
+    #     script: |
+    #       const adapter = '${{ matrix.adapter.name }}';
+    #       const url = '${{ github.server_url }}/${{ github.repository }}/actions/runs/${{ github.run_id }}';
+    #       const test_status = '${{ steps.tests.outcome }}';
+    #       const job_status = '${{ job.status }}';
+    #       const body = `E2E ${adapter} build:\n${url}\nJob status: ${job_status}. Test status: ${test_status}`;
+
+    #       github.rest.issues.createComment({
+    #         issue_number: context.issue.number,
+    #         owner: context.repo.owner,
+    #         repo: context.repo.repo,
+    #         body: body
+    #       })