major: use parallel config by default for future mode

# Conflicts: # .php-cs-fixer.dist.php # src/Config.php
Wirone · May 5, 2024 · 4b8812e · 4b8812e
1 parent 4686a14
commit 4b8812e
Show file tree

Hide file tree

Showing 3 changed files with 11 additions and 4 deletions.
diff --git a/.php-cs-fixer.dist.php b/.php-cs-fixer.dist.php
@@ -17,7 +17,7 @@
 use PhpCsFixer\Runner\Parallel\ParallelConfig;
 
 return (new Config())
-    ->setParallelConfig(ParallelConfig::detect())
+    ->setParallelConfig(ParallelConfigFactory::detect()) // @TODO 4.0 no need to call this manually
     ->setRiskyAllowed(true)
     ->setRules([
         '@PHP74Migration' => true,

diff --git a/src/Config.php b/src/Config.php
@@ -48,7 +48,7 @@ class Config implements ConfigInterface, ParallelRunnerConfigInterface
 
     private string $name;
 
-    private ?ParallelConfig $parallelRunnerConfig;
+    private ParallelConfig $parallelConfig;
 
     /**
      * @var null|string
@@ -74,6 +74,13 @@ public function __construct(string $name = 'default')
             $this->name = $name;
             $this->rules = ['@PSR12' => true];
         }
+
+        // @TODO 4.0 cleanup
+        if (Utils::isFutureModeEnabled() || filter_var(getenv('PHP_CS_FIXER_PARALLEL'), FILTER_VALIDATE_BOOL)) {
+            $this->parallelConfig = ParallelConfigFactory::detect();
+        } else {
+            $this->parallelConfig = ParallelConfigFactory::sequential();
+        }
     }
 
     public function getCacheFile(): string
@@ -125,7 +132,7 @@ public function getName(): string
 
     public function getParallelConfig(): ParallelConfig
     {
-        return $this->parallelRunnerConfig ?? ParallelConfig::sequential();
+        return $this->parallelConfig;
     }
 
     public function getPhpExecutable(): ?string

diff --git a/src/Console/Command/FixCommand.php b/src/Console/Command/FixCommand.php
@@ -272,7 +272,7 @@ protected function execute(InputInterface $input, OutputInterface $output): int
                 ) : ' sequentially'
             ));
 
-            // @TODO remove when parallel runner is mature enough and works as expected
+            // @TODO v4 remove the warning
             if ($isParallel) {
                 $stdErr->writeln(
                     sprintf(