Merge pull request #1348 from danger/revert-1337-fix-github-action

Revert "Moving the GITHUB_WORKFLOW check before actually getting the userInfo"
danger · Jan 18, 2023 · 6fc25ff · 6fc25ff
2 parents 47dfcfa + fed6d23
commit 6fc25ff
Showing 1 changed file with 5 additions and 5 deletions.
diff --git a/source/platforms/github/GitHubAPI.ts b/source/platforms/github/GitHubAPI.ts
@@ -130,6 +130,11 @@ export class GitHubAPI {
       return parseInt(perilID)
     }
 
+    const info = await this.getUserInfo()
+    if (info.id) {
+      return info.id
+    }
+
     const useGitHubActionsID = process.env["GITHUB_WORKFLOW"]
     if (useGitHubActionsID) {
       // This is the user.id of the github-actions app (https://github.com/apps/github-actions)
@@ -138,11 +143,6 @@ export class GitHubAPI {
       return 41898282
     }
 
-    const info = await this.getUserInfo()
-    if (info.id) {
-      return info.id
-    }
-
     return undefined
   }