Merge pull request #40 from saschanaz/patch-1

Do not early return when no relevant code owner exists

Author: orta

index.js

@@ -52,14 +52,6 @@ async function commentOnMergablePRs() {
   core.info(`Code-owners: \n - ${codeowners.users.join("\n - ")}`)
   core.info(`Labels: \n - ${codeowners.labels.join("\n - ")}`)
 
-  if (!codeowners.users.length) {
-    console.log("This PR does not have any code-owners")
-    process.exit(0)
-  }
-
-
-
-
   // Determine who has access to merge every file in this PR
   const ownersWhoHaveAccessToAllFilesInPR = []
   codeowners.users.forEach(owner => {
@@ -86,6 +78,11 @@ async function commentOnMergablePRs() {
     process.exit(0)
   }
 
+  if (!codeowners.users.length) {
+    console.log("This PR does not have any code-owners")
+    process.exit(0)
+  }
+
   const ourSignature = "<!-- Message About Merging -->"
   const comments = await octokit.issues.listComments({ ...thisRepo, issue_number: pr.number })
   const existingComment = comments.data.find(c => c.body.includes(ourSignature))