fix(manage-merge-queue): improve logging further (#658)

ExpediaGroup · Aug 22, 2024 · 1cb1521 · 1cb1521
1 parent 28f44ef
commit 1cb1521
Show file tree

Hide file tree

Showing 6 changed files with 12 additions and 10 deletions.
diff --git a/dist/374.index.js b/dist/374.index.js
diff --git a/dist/374.index.js.map b/dist/374.index.js.map
diff --git a/dist/676.index.js b/dist/676.index.js
diff --git a/dist/676.index.js.map b/dist/676.index.js.map
diff --git a/src/helpers/is-user-in-team.ts b/src/helpers/is-user-in-team.ts
@@ -26,7 +26,7 @@ export const isUserInTeam = async ({ login = context.actor, team }: IsUserInTeam
     org: context.repo.owner,
     team_slug: team
   });
-  core.debug(`Checking if ${login} is in team ${team}`);
-  core.debug(`Team members: ${response.data.map(({ login }) => login).join(', ')}`);
+  core.info(`Checking if ${login} is in team ${team}`);
+  core.info(`Team members: ${response.data.map(({ login }) => login).join(', ')}`);
   return response.data.some(({ login: memberLogin }) => memberLogin === login);
 };
diff --git a/src/helpers/manage-merge-queue.ts b/src/helpers/manage-merge-queue.ts
@@ -73,7 +73,8 @@ export const manageMergeQueue = async ({
   }
   if (pullRequest.labels.find(label => label.name === JUMP_THE_QUEUE_PR_LABEL)) {
     if (allow_only_for_maintainers === 'true') {
-      const isMaintainer = await isUserInTeam({ login, team: team });
+      core.info(`Checking if user ${login} is a maintainer...`);
+      const isMaintainer = await isUserInTeam({ team: team });
       if (!isMaintainer) {
         await removeLabelIfExists(JUMP_THE_QUEUE_PR_LABEL, pullRequest.number);
         return await createPrComment({