fix: Replace 'return await' with 'return', since these functions return Promises and aren’t doing any error-handling that’d require 'return await'

Author: smockle

.github/actions/file/src/closeIssueForFinding.ts

@@ -8,7 +8,7 @@ export async function closeIssueForFinding(octokit: Octokit, repoWithOwner: stri
   if (!issueNumber) {
     throw new Error(`Invalid issue URL: ${finding.issueUrl}`);
   }
-  return await octokit.request(`PATCH /repos/${owner}/${repo}/issues/${issueNumber}`, {
+  return octokit.request(`PATCH /repos/${owner}/${repo}/issues/${issueNumber}`, {
     owner,
     repo,
     issue_number: issueNumber,

.github/actions/file/src/openIssueForFinding.ts

@@ -9,7 +9,7 @@ export async function openIssueForFinding(octokit: Octokit, repoWithOwner: strin
   const issueNumber = finding.issueUrl?.split('/').pop();
   if (issueNumber) {
     // If an issue already exists, ensure it is open
-    return await octokit.request(`PATCH /repos/${owner}/${repo}/issues/${issueNumber}`, {
+    return octokit.request(`PATCH /repos/${owner}/${repo}/issues/${issueNumber}`, {
       owner,
       repo,
       issue_number: issueNumber,
@@ -35,7 +35,7 @@ To fix this, ${finding.solutionShort}.
 ${solutionLong ? `\nSpecifically:\n\n${solutionLong}` : ''}
 `;
 
-    return await octokit.request(`POST /repos/${owner}/${repo}/issues`, {
+    return octokit.request(`POST /repos/${owner}/${repo}/issues`, {
       owner,
       repo,
       title,

.github/actions/fix/src/fixIssue.ts

@@ -48,7 +48,7 @@ export async function fixIssue(octokit: Octokit, repoWithOwner: string, issueUrl
     return;
   }
   // Assign issue to Copilot
-  return await octokit.graphql(
+  return octokit.graphql(
     `mutation($issueId: ID!, $assigneeId: ID!) {
       replaceActorsForAssignable(input: {assignableId: $issueId, actorIds: [$assigneeId]}) {
         assignable {