Complain on errors

oasis-tcs · Jul 22, 2023 · 8635966 · 8635966
1 parent 6517a56
commit 8635966
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 21 deletions.
diff --git a/lib/build-pdf.js b/lib/build-pdf.js
@@ -1,8 +1,6 @@
 const pdf = require("./pdf.js");
 const fs = require("fs");
 
-let errors = false;
-
 fs.readdirSync(__dirname + "/..", { withFileTypes: true }).forEach(function (
   doc
 ) {
@@ -14,17 +12,15 @@ fs.readdirSync(__dirname + "/..", { withFileTypes: true }).forEach(function (
     const pdfStat = fs.statSync(pdfFile, { throwIfNoEntry: false });
 
     if (pdfStat === undefined || htmlStat?.mtime > pdfStat?.mtime) {
-      console.log(doc.name);
-
-      pdf(doc.name).catch((error) => {
-        console.error(error);
-        errors = true;
-      });
+      pdf(doc.name)
+        .then(() => {
+          console.log("✓ " + doc.name);
+        })
+        .catch((error) => {
+          console.log("❌ " + doc.name);
+          console.error(error);
+          process.exitCode = 1;
+        });
     }
   }
 });
-
-//TODO: needs to await all threads
-// if (errors) {
-//   process.exit(1);
-// }
diff --git a/lib/build.js b/lib/build.js
@@ -3,8 +3,6 @@ const pandoc = require("./pandoc");
 const fs = require("fs");
 const { PassThrough } = require("stream");
 
-let errors = false;
-
 fs.readdirSync(__dirname + "/..", { withFileTypes: true }).forEach(function (
   doc
 ) {
@@ -37,14 +35,10 @@ fs.readdirSync(__dirname + "/..", { withFileTypes: true }).forEach(function (
         console.log("✓ " + doc.name);
       })
       .catch((err) => {
+        console.log("❌ " + doc.name);
         console.error(err.join("\n"));
         console.error();
-        errors = true;
+        process.exitCode = 1;
       });
   }
 });
-
-//TODO: needs to await all threads
-// if (errors) {
-//   process.exit(1);
-// }