Run tsc before linting

nicolo-ribaudo · Apr 22, 2024 · 2ecb92a · 2ecb92a
1 parent bd75691
commit 2ecb92a
Show file tree

Hide file tree

Showing 3 changed files with 11 additions and 8 deletions.
diff --git a/Makefile.js b/Makefile.js
diff --git a/Makefile.source.mjs b/Makefile.source.mjs
@@ -349,15 +349,16 @@ target["clone-license"] = function () {
  */
 
 target["lint"] = function () {
-  // Pre-run tscheck, otherwise typescript-eslint
-  // will run it by itsels
-  target["tscheck"]();
   env(
     () => {
+      // Pre-run tscheck, otherwise typescript-eslint
+      // will run it by itsels
+      target["tscheck"]();
       yarn(EslintArgs);
     },
     {
       BABEL_ENV: "test",
+      TSCHECK_SILENT: "true",
     }
   );
 };

diff --git a/scripts/parallel-tsc/tsc.js b/scripts/parallel-tsc/tsc.js
@@ -195,9 +195,11 @@ const pool = new Pool({
     );
 
     done += projects.length;
-    console.log(
-      `[${done}/${total}] ${projects.map(project => "./" + relativePath(process.cwd(), project)).join(" ")} took ${Math.round(duration) / 1e3}s`
-    );
+    if (!process.env.TSCHECK_SILENT) {
+      console.log(
+        `[${done}/${total}] ${projects.map(project => "./" + relativePath(process.cwd(), project)).join(" ")} took ${Math.round(duration) / 1e3}s`
+      );
+    }
     for (const project of projects) {
       for (const dependent of projectToDependents.get(project) ?? []) {
         const dependencies = projectToDependencies.get(dependent);