feat: Default to number of processors for exec concurrency (#654)

* better default value for exec concurrency * reformat file * change default value inside function
invertase · Mar 10, 2024 · 43c6ddb · 43c6ddb
1 parent 95f2371
commit 43c6ddb
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 2 deletions.
diff --git a/packages/melos/lib/src/command_runner/exec.dart b/packages/melos/lib/src/command_runner/exec.dart
@@ -23,7 +23,11 @@ import 'base.dart';
 class ExecCommand extends MelosCommand {
   ExecCommand(super.config) {
     setupPackageFilterParser();
-    argParser.addOption('concurrency', defaultsTo: '5', abbr: 'c');
+    argParser.addOption(
+      'concurrency',
+      defaultsTo: Platform.numberOfProcessors.toString(),
+      abbr: 'c',
+    );
     argParser.addFlag(
       'fail-fast',
       abbr: 'f',

diff --git a/packages/melos/lib/src/commands/exec.dart b/packages/melos/lib/src/commands/exec.dart
@@ -5,10 +5,11 @@ mixin _ExecMixin on _Melos {
     List<String> execArgs, {
     GlobalOptions? global,
     PackageFilters? packageFilters,
-    int concurrency = 5,
+    int? concurrency,
     bool failFast = false,
     bool orderDependents = false,
   }) async {
+    concurrency ??= Platform.numberOfProcessors;
     final workspace =
         await createWorkspace(global: global, packageFilters: packageFilters);
     final packages = workspace.filteredPackages.values;