Show notice when --verbose and --concurrency > 1

Author: kraenhansen

packages/ferric/src/build.ts

@@ -134,6 +134,10 @@ const verboseOption = new Option(
   "Print more output from underlying compiler & tools",
 ).default(process.env.CI ? true : false, `false in general and true on CI`);
 
+function logNotice(message: string, ...params: string[]) {
+  console.log(`${chalk.yellow("ℹ︎")} ${message}`, ...params);
+}
+
 export const buildCommand = new Command("build")
   .description("Build Rust Node-API module")
   .addOption(targetOption)
@@ -168,13 +172,16 @@ export const buildCommand = new Command("build")
             {
               text: "Cleaning target directory",
               successText: "Cleaned target directory",
-              failText: (error) =>
-                `Failed to cleaning target directory: ${error}`,
+              failText: (error) => `Failed to clean target directory: ${error}`,
             },
           );
         }
-        // Force a limit of 1 concurrent task to avoid interleaving output
-        const limit = pLimit(verbose ? 1 : concurrency);
+        if (verbose && concurrency > 1) {
+          logNotice(
+            `Consider passing ${chalk.blue("--concurrency")} 1 when running in verbose mode`,
+          );
+        }
+        const limit = pLimit(concurrency);
         const targets = new Set([...targetArg]);
         if (apple) {
           for (const target of APPLE_TARGETS) {
@@ -200,15 +207,12 @@ export const buildCommand = new Command("build")
               targets.add("aarch64-apple-ios-sim");
             }
           }
-          console.error(
-            chalk.yellowBright("ℹ"),
-            chalk.dim(
-              `Using default targets, pass ${chalk.italic(
-                "--android",
-              )}, ${chalk.italic("--apple")} or individual ${chalk.italic(
-                "--target",
-              )} options, to avoid this.`,
-            ),
+          logNotice(
+            `Using default targets, pass ${chalk.blue(
+              "--android",
+            )}, ${chalk.blue("--apple")} or individual ${chalk.blue(
+              "--target",
+            )} options, choose exactly what to target`,
           );
         }
         ensureCargo();