Merge pull request #732 from ocaml/process-exit

Force all actions to exit with `process.exit`
ocaml · Dec 6, 2023 · a382c86 · a382c86
2 parents 33d766a + 42c664e
commit a382c86
Show file tree

Hide file tree

Showing 9 changed files with 48 additions and 2 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -8,6 +8,10 @@ and this project adheres to
 
 ## [unreleased]
 
+### Fixed
+
+- Force all actions to exit with `process.exit`.
+
 ## [2.1.7]
 
 ### Changed

diff --git a/dist/index.js b/dist/index.js
diff --git a/lint-doc/dist/index.js b/lint-doc/dist/index.js
diff --git a/lint-fmt/dist/index.js b/lint-fmt/dist/index.js
diff --git a/lint-opam/dist/index.js b/lint-opam/dist/index.js
diff --git a/packages/lint-doc/src/index.ts b/packages/lint-doc/src/index.ts
@@ -1,3 +1,6 @@
+/* eslint-disable unicorn/no-process-exit */
+import * as process from "node:process";
+
 import * as core from "@actions/core";
 
 import { lintOdoc } from "./odoc.js";
@@ -8,10 +11,12 @@ async function run() {
     await installOpamPackages();
     await installOdoc();
     await lintOdoc();
+    process.exit(0);
   } catch (error) {
     if (error instanceof Error) {
       core.setFailed(error.message);
     }
+    process.exit(1);
   }
 }
 

diff --git a/packages/lint-fmt/src/index.ts b/packages/lint-fmt/src/index.ts
@@ -1,3 +1,6 @@
+/* eslint-disable unicorn/no-process-exit */
+import * as process from "node:process";
+
 import * as core from "@actions/core";
 
 import { checkFmt } from "./lint.js";
@@ -9,10 +12,12 @@ async function run() {
     const version = await getOcamlformatVersion();
     await installOcamlformat(version);
     await checkFmt();
+    process.exit(0);
   } catch (error) {
     if (error instanceof Error) {
       core.setFailed(error.message);
     }
+    process.exit(1);
   }
 }
 

diff --git a/packages/lint-opam/src/index.ts b/packages/lint-opam/src/index.ts
@@ -1,3 +1,6 @@
+/* eslint-disable unicorn/no-process-exit */
+import * as process from "node:process";
+
 import * as core from "@actions/core";
 
 import { opamDuneLint, opamLint } from "./lint.js";
@@ -9,10 +12,12 @@ async function run() {
     await installOpamDuneLint();
     await opamLint();
     await opamDuneLint();
+    process.exit(0);
   } catch (error) {
     if (error instanceof Error) {
       core.setFailed(error.message);
     }
+    process.exit(1);
   }
 }
 

diff --git a/packages/setup-ocaml/src/index.ts b/packages/setup-ocaml/src/index.ts
@@ -1,14 +1,19 @@
+/* eslint-disable unicorn/no-process-exit */
+import * as process from "node:process";
+
 import * as core from "@actions/core";
 
 import { installer } from "./installer.js";
 
 async function run() {
   try {
     await installer();
+    process.exit(0);
   } catch (error) {
     if (error instanceof Error) {
       core.setFailed(error.message);
     }
+    process.exit(1);
   }
 }