Leverage post-hook

Author: yoanm

.github/actions/reports-group/attach-check-run-to-triggering-workflow-action/action.yml

@@ -12,12 +12,9 @@ inputs:
   github-token:
     description: Github Authentication token used to create the check through GitHub API
     required: true
-  status:
-    description: Status check status
-    required: false
-  conclusion:
-    description: Status check conclusion
-    required: false
+  job-status:
+    description: Job check status
+    required: true
   external-id:
     description: Status check external ID
     required: false

.github/actions/reports-group/attach-check-run-to-triggering-workflow-action/dist/index.js

@@ -1,49 +1,46 @@
 const github = require('@actions/github'); // @TODO move to 'imports from' when moved to TS !
 const core = require('@actions/core');
 
+const {GITHUB_REPOSITORY} = process.env;
+
 async function run() {
+    if (core.getState('check-run-is-concluded').length) {
+        core.info('Check run already concluded, skipping check run update');
+    }
+
+    const [repoOwner, repoName] = GITHUB_REPOSITORY.split('/');
+    /** INPUTS **/
+    const jobStatus = core.getInput('job-status', {required: true});
     const checkRunId = core.getState('check-run-id');
     if (checkRunId.length === 0) {
         throw new Error('Unable to retrieve check run id !');
     }
+    const githubToken = core.getInput('github-token', {required: true});
 
     const requestParams = await core.group(
         'Build API params',
         async () => {
-            const res = {
-                name: checkName,
-                head_sha: commitSha,
-                details_url: undefinedIfEmpty(detailsUrl),
-                external_id: undefinedIfEmpty(externalId),
-                status: undefinedIfEmpty(checkStatus),
-                conclusion: undefinedIfEmpty(checkConclusion),
-                started_at: undefinedIfEmpty(startedAt),
-                completed_at: undefinedIfEmpty(completedAt),
+            return {
+                conclusion: jobStatus,
                 owner: repoOwner,
-                repo: repoName
+                repo: repoName,
+                check_run_id: checkRunId
             };
-            if (!isEmpty(outputTitle) || !isEmpty(outputSummary)) {
-                res.output = {title: undefinedIfEmpty(outputTitle), summary: undefinedIfEmpty(outputSummary)};
-            }
-
-            return res;
         }
     );
     core.debug('API params=' + JSON.stringify(requestParams));
 
-    const apiResponse = await core.group('Call API', async () => {
+    const apiResponse = await core.group('Retrieve ', async () => {
         const octokit = github.getOctokit(githubToken);
 
-        // @TODO Move back to `octokit.rest.checks.create()`
-        const res = await octokit.request('POST /repos/{owner}/{repo}/check-runs', requestParams);
+        // @TODO Move back to `octokit.rest.checks.update()`
+        const res = await octokit.request('PATCH /repos/{owner}/{repo}/check-runs/{check_run_id}', requestParams);
 
         core.info('TMP DEBUG0 ' + JSON.stringify(res));
 
         return res;
     });
     core.info('TMP DEBUG' + JSON.stringify(apiResponse));
-
-    core.setOutput('check-run-id', apiResponse.data.id);
 }
 
 run();

.github/actions/reports-group/attach-check-run-to-triggering-workflow-action/dist/index.js.map

@@ -1,24 +1,26 @@
 const github = require('@actions/github'); // @TODO move to 'imports from' when moved to TS !
 const core = require('@actions/core');
 
+const {GITHUB_REPOSITORY} = process.env;
+
 async function run() {
-    const {GITHUB_REPOSITORY: repository} = process.env;
-    const [repoOwner, repoName] = repository.split('/');
+    const [repoOwner, repoName] = GITHUB_REPOSITORY.split('/');
     /** INPUTS **/
     const commitSha = core.getInput('commit-sha', {required: true});
     const checkName = core.getInput('name', {required: true});
     const githubToken = core.getInput('github-token', {required: true});
+    const jobStatus = core.getInput('job-status', {required: true});
 
     // Following inputs are not required and may not have any value attached !
-    const checkStatus = core.getInput('status');
-    const checkConclusion = core.getInput('conclusion');
     const externalId = core.getInput('external-id');
     const startedAt = core.getInput('started-at');
     const completedAt = core.getInput('completed-at');
     const detailsUrl = core.getInput('details-url');
     const outputTitle = core.getInput('output');
     const outputSummary = core.getInput('output-summary');
 
+    const isSuccessfulJobAsOfNow = 'success' === jobStatus;
+
     const requestParams = await core.group(
         'Build API params',
         async () => {
@@ -27,8 +29,8 @@ async function run() {
                 head_sha: commitSha,
                 details_url: undefinedIfEmpty(detailsUrl),
                 external_id: undefinedIfEmpty(externalId),
-                status: undefinedIfEmpty(checkStatus),
-                conclusion: undefinedIfEmpty(checkConclusion),
+                status: isSuccessfulJobAsOfNow ? 'in_progress' : 'completed',
+                conclusion: isSuccessfulJobAsOfNow ? undefined : jobStatus,
                 started_at: undefinedIfEmpty(startedAt),
                 completed_at: undefinedIfEmpty(completedAt),
                 owner: repoOwner,
@@ -57,6 +59,9 @@ async function run() {
 
     core.setOutput('check-run-id', apiResponse.data.id);
     core.saveState('check-run-id', apiResponse.data.id); // In order to use it during POST hook
+    if (true === isSuccessfulJobAsOfNow) {
+        core.saveState('check-run-already-concluded', 'yes'); // In order to use it during POST hook
+    }
 }
 
 /**

.github/actions/reports-group/attach-check-run-to-triggering-workflow-action/src/cleanup.js

@@ -50,7 +50,7 @@ jobs:
           name: "${{ github.workflow }} (${{ github.event.workflow.name}}) / Codacy / Upload all reports"
           commit-sha: "${{ ('workflow_run' == github.event_name && ('pull_request' == github.event.workflow_run.event || 'push' == github.event.workflow_run.event) && github.event.workflow_run.head_sha) || ('pull_request' == github.event_name && github.event.pull_request.head.sha) || ('push' == github.event_name && github.sha) || null }}"
           github-token: ${{ github.token }}
-          status: queued
+          job-status: ${{ job.status }}
 
       - name: Download artifacts
         uses: actions/download-artifact@v4

.github/actions/reports-group/attach-check-run-to-triggering-workflow-action/src/main.js

@@ -42,7 +42,7 @@ jobs:
           name: "${{ github.workflow }} (${{ github.event.workflow.name}}) / Codecov / Prepare"
           commit-sha: "${{ ('workflow_run' == github.event_name && ('pull_request' == github.event.workflow_run.event || 'push' == github.event.workflow_run.event) && github.event.workflow_run.head_sha) || ('pull_request' == github.event_name && github.event.pull_request.head.sha) || ('push' == github.event_name && github.sha) || null }}"
           github-token: ${{ github.token }}
-          status: queued
+          job-status: ${{ job.status }}
 
       - name: Download artifacts
         uses: actions/download-artifact@v4
@@ -129,7 +129,7 @@ jobs:
           name: "${{ github.workflow }} (${{ github.event.workflow.name}}) / Codecov / ${{ matrix.artifact}} - ${{ matrix.path }}"
           commit-sha: "${{ ('workflow_run' == github.event_name && ('pull_request' == github.event.workflow_run.event || 'push' == github.event.workflow_run.event) && github.event.workflow_run.head_sha) || ('pull_request' == github.event_name && github.event.pull_request.head.sha) || ('push' == github.event_name && github.sha) || null }}"
           github-token: ${{ github.token }}
-          status: queued
+          job-status: ${{ job.status }}
 
       - name: Download '${{ matrix.artifact }}' artifact
         uses: actions/download-artifact@v4