fix: isPullRequestFromFork returns false for any PR (#1437)

* fix: isPullRequestFromFork returns false for any PR * fix: add log statement * fix: force set token * fix: pass in empty string * fix: bump version * fix: revert to promise * fix: revert changes --------- Co-authored-by: Tom Hu <thomas.hu@sentry.io>
codecov · May 20, 2024 · c9dbf6a · c9dbf6a
1 parent 59fc46f
commit c9dbf6a
Show file tree

Hide file tree

Showing 4 changed files with 8 additions and 6 deletions.
diff --git a/.github/workflows/main.yml b/.github/workflows/main.yml
@@ -42,7 +42,7 @@ jobs:
           file: ./coverage/coverage-final.json
           flags: version,${{ matrix.os }}
           name: codecov-version
-          version: v0.2.0
+          version: v0.6.0
           verbose: true
           token: ${{ secrets.CODECOV_TOKEN }}
 
@@ -85,7 +85,7 @@ jobs:
           file: ./coverage/coverage-final.json
           flags: version,maxos-latest-xlarge
           name: codecov-version
-          version: v0.2.0
+          version: v0.6.0
           verbose: true
           token: ${{ secrets.CODECOV_TOKEN }}
 
@@ -125,6 +125,6 @@ jobs:
           file: ./coverage/coverage-final.json
           flags: version,${{ matrix.os }}
           name: codecov-version
-          version: v0.2.0
+          version: v0.6.0
           verbose: true
           token: ${{ secrets.CODECOV_TOKEN }}
diff --git a/dist/index.js b/dist/index.js
@@ -32340,7 +32340,8 @@ const getGitService = () => {
     return 'github';
 };
 const isPullRequestFromFork = () => {
-    if (`${context.eventName}` !== 'pull_request' ||
+    core.info(`evenName: ${context.eventName}`);
+    if (`${context.eventName}` !== 'pull_request' &&
         `${context.eventName}` !== 'pull_request_target') {
         return false;
     }

diff --git a/dist/index.js.map b/dist/index.js.map
diff --git a/src/buildExec.ts b/src/buildExec.ts
@@ -30,8 +30,9 @@ const getGitService = (): string => {
 };
 
 const isPullRequestFromFork = (): boolean => {
+  core.info(`evenName: ${context.eventName}`);
   if (
-    `${context.eventName}` !== 'pull_request' ||
+    `${context.eventName}` !== 'pull_request' &&
     `${context.eventName}` !== 'pull_request_target'
   ) {
     return false;