fix: don't post PR comments on forks

getsentry · Jan 9, 2023 · 7a4e90f · 7a4e90f
1 parent bc6455e
commit 7a4e90f
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 1 deletion.
diff --git a/.github/workflows/build.yml b/.github/workflows/build.yml
@@ -791,11 +791,14 @@ jobs:
         id: process
         run: yarn ci:process
         working-directory: packages/replay/metrics
+        # Don't run on forks - the PR comment cannot be added.
+        if: github.event_name != 'pull_request' || github.event.pull_request.head.repo.full_name == github.repository
         env:
           GITHUB_TOKEN: ${{ github.token }}
 
       - name: Upload results
         uses: actions/upload-artifact@v3
+        if: github.event_name != 'pull_request' || github.event.pull_request.head.repo.full_name == github.repository
         with:
           name: ${{ steps.process.outputs.artifactName }}
           path: ${{ steps.process.outputs.artifactPath }}
diff --git a/packages/replay/metrics/src/util/github.ts b/packages/replay/metrics/src/util/github.ts
@@ -98,7 +98,7 @@ async function tryAddOrUpdateComment(commentBuilder: PrCommentBuilder): Promise<
       body: commentBuilder.body,
     });
   } else {
-    console.log(`Adding new PR comment to PR ${prNumber}`)
+    console.log(`Adding a new comment to PR ${prNumber}`)
     await octokit.rest.issues.createComment({
       ...defaultArgs,
       issue_number: prNumber,