put param in obj

src/helpers/approvals-satisfied.ts

@@ -29,12 +29,17 @@ export class ApprovalsSatisfied extends HelperInputs {
   number_of_reviewers?: string;
   required_review_overrides?: string;
   pull_number?: string;
+  approvals_not_met_message?: string;
 }
 
-export const approvalsSatisfied = async (
-  { teams, users, number_of_reviewers = '1', required_review_overrides, pull_number }: ApprovalsSatisfied = {},
-  approvalsNotMetMessage: string | undefined = undefined
-) => {
+export const approvalsSatisfied = async ({
+  teams,
+  users,
+  number_of_reviewers = '1',
+  required_review_overrides,
+  pull_number,
+  approvals_not_met_message
+}: ApprovalsSatisfied = {}) => {
   const prNumber = pull_number ? Number(pull_number) : context.issue.number;
 
   const teamOverrides = required_review_overrides?.split(',').map(overrideString => {
@@ -94,8 +99,8 @@ export const approvalsSatisfied = async (
   if (!approvalsSatisfied) {
     logs.unshift('Required approvals not satisfied:\n');
 
-    if (approvalsNotMetMessage) {
-      logs.unshift(approvalsNotMetMessage + '\n');
+    if (approvals_not_met_message) {
+      logs.unshift(approvals_not_met_message + '\n');
     }
 
     await createPrComment({

src/helpers/manage-merge-queue.ts

@@ -45,7 +45,9 @@ export const manageMergeQueue = async ({ max_queue_size, login, slack_webhook_ur
     core.info('This PR is not in the merge queue.');
     return removePrFromQueue(pullRequest);
   }
-  const prMeetsRequiredApprovals = await approvalsSatisfied({}, 'PRs must meet all required approvals before entering the merge queue.');
+  const prMeetsRequiredApprovals = await approvalsSatisfied({
+    approvals_not_met_message: 'PRs must meet all required approvals before entering the merge queue.'
+  });
   if (!prMeetsRequiredApprovals) {
     return removePrFromQueue(pullRequest);
   }

test/helpers/approvals-satisfied.test.ts

@@ -613,13 +613,11 @@ Number of required reviews: 1`
           }
         ]
       });
-      await approvalsSatisfied(
-        {
-          users: '@user1,@user2',
-          pull_number: '12345'
-        },
-        'PRs must meet all required approvals before entering the merge queue.'
-      );
+      await approvalsSatisfied({
+        users: '@user1,@user2',
+        pull_number: '12345',
+        approvals_not_met_message: 'PRs must meet all required approvals before entering the merge queue.'
+      });
       expect(octokit.issues.createComment).toHaveBeenCalledWith(
         expect.objectContaining({
           body: `PRs must meet all required approvals before entering the merge queue.