Refactor CI for forked repositories - Enhance (yoanm#125)

Author: yoanm

.github/actions/reports-group/.nvmrc

@@ -0,0 +1 @@
+20

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

@@ -1,6 +1,6 @@
 const core = require("@actions/core"); // @TODO move to 'imports from' when moved to TS !
 
-if (!!core.getState('check-run-id')) {
+if (!!core.getState('has-been-triggered')) {
  require('./src/cleanup');
 } else {
  require('./src/main');

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

@@ -31,6 +31,7 @@ async function run() {
  core.debug('API params=' + JSON.stringify(requestParams));
 
  const apiResponse = await core.group('Conclude check-run ', async () => {
+ /** @type {OctokitInterface} */
  const octokit = github.getOctokit(githubToken);
 
  // @TODO Move back to `octokit.rest.checks.update()`
@@ -39,4 +40,5 @@ async function run() {
  core.debug('API call to ' +apiResponse.url + ' => HTTP ' + apiResponse.status);
 }
 
+// No need to wrap cleanup with try/catch, error are ignored anyway
 run();

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

@@ -6,14 +6,15 @@ const ghaHelpers = require('../node-gha-helpers');
 const formatMarkdownUrl = (title, link) => '<a href="' + link + '" target="blank">' + title + '</a>';
 
 async function run() {
+ core.saveState('has-been-triggered', 1);
  /** INPUTS **/
  const githubToken = core.getInput('github-token', {required: true});
  const jobStatus = core.getInput('job-status', {required: true});
  const checkName = core.getInput('name');
  const failsOnTriggeringWorkflowFailure = core.getBooleanInput('fails-on-triggering-workflow-failure', {required: true});
 
  const isSuccessfulJobAsOfNow = 'success' === jobStatus;
- const octokit = getOctokit(githubToken);
+ const octokit = /** @type {OctokitInterface} */getOctokit(githubToken);
 
  const requestParams = await core.group(
  'Build API params',
@@ -70,4 +71,7 @@ async function run() {
  }
 }
 
-run();
+run().catch(e => {
+ core.warning('Error caught and ignored ' + e.message);
+ core.debug('Error=' + JSON.stringify(e));
+});

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

@@ -116,7 +116,6 @@ async function run() {
  return res;
  }
  );
- core.debug('outputs=' + JSON.stringify(outputs));
  SDK.outputs.bindFrom(outputs);
 }
 

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

@@ -4,7 +4,7 @@ description: TODO
 inputs:
  from-triggering-workflow:
  description: |
- Weither to look for triggering workflow context or just current workflow context (default `true` !).
+ Whether to look for triggering workflow context or just current workflow context (default `true` !).
  - True: In case current workflow event is `workflow_run`, the context of the workflow which **triggered** the current one will be returned. Else returns the current workflow context
  - False: Return the context for the current workflow.
  required: false