merge script into "comment_tf_output" due to overlap of concerns

Signed-off-by: Rishav Dhar <19497993+rdhar@users.noreply.github.com>

action.yml

@@ -354,17 +354,6 @@ runs:
         retries: 3
         script: await require(`${process.env.GITHUB_ACTION_PATH}/scripts/remove_reaction.js`)({ github, context });
 
-    # On pull_request, update check status with the job summary before exiting.
-    - name: Update check status
-      if: ${{ (success() || failure()) && steps.pr_comment.outcome == 'success' }}
-      uses: actions/github-script@60a0d83039c74a4aee543508d2ffcb1c3799cdea # v7.0.1
-      env:
-        check_id: ${{ steps.pr_comment.outputs.check_id }}
-        tf_summary: ${{ steps.pr_comment.outputs.summary }}
-      with:
-        retries: 3
-        script: await require(`${process.env.GITHUB_ACTION_PATH}/scripts/update_check_status.js`)({ github, context });
-
     # On issue_comment, update commit status with the job status before exiting.
     - name: Update commit status
       if: ${{ (success() || failure()) && github.event.issue.pull_request }}

scripts/comment_tf_output.js

@@ -26,7 +26,15 @@ ${process.env.tf_fmt}
   const matrix = JSON.parse(process.env.matrix);
   const job_name = `${context.job}${matrix ? ` (${Object.values(matrix).join(", ")})` : ""}`;
   const check_url = workflow_run.jobs.find((job) => job.name === job_name).html_url;
-  core.setOutput("check_id", workflow_run.jobs.find((job) => job.name === job_name).id);
+  const check_id = workflow_run.jobs.find((job) => job.name === job_name).id;
+
+  // Update the check status with TF output summary.
+  const update_check_status = await github.rest.checks.update({
+    check_run_id: check_id,
+    output: { title: comment_summary },
+    owner: context.repo.owner,
+    repo: context.repo.repo,
+  });
 
   // Display the: TF command, TF output, and workflow authorip.
   const comment_output = `