Support multiple owners per glob (#23)

mdzhang · Apr 10, 2024 · 7145304 · 7145304
1 parent 79a6caa
commit 7145304
Show file tree

Hide file tree

Showing 3 changed files with 20 additions and 12 deletions.
diff --git a/badges/coverage.svg b/badges/coverage.svg
diff --git a/dist/index.js b/dist/index.js
diff --git a/src/api.ts b/src/api.ts
@@ -103,14 +103,18 @@ export async function getCodeowners(
     .filter(l => l.trim().length > 0)
     .filter(l => !l.startsWith('#'))
     .map(l => l.split(' '))
-    .filter(([glob, team]) => {
-      if (team === undefined) {
+    .filter(([glob, ...teams]) => {
+      if (
+        teams.length === 0 ||
+        (teams.length === 1 && teams[0] === undefined)
+      ) {
         core.warning(`CODEOWNERS had glob ${glob} w/o matching team`)
+        return false
       }
 
-      return team !== undefined
+      return true
     })
-    .map(([glob, team]) => {
+    .map(([glob, ...teams]) => {
       // do some munging to support CODEOWNER format globs
       let finalGlob = glob
 
@@ -125,8 +129,9 @@ export async function getCodeowners(
         }
       }
 
-      return [finalGlob, team]
+      return teams.map(team => [finalGlob, team])
     })
+    .flat(1)
 
   if (!codeowners.length) {
     core.warning(`Pull request #${prNumber} has no codeowners`)