major: use parallel config by default for future mode

# Conflicts: # .php-cs-fixer.dist.php # src/Config.php
Wirone · May 7, 2024 · 4e8d5c1 · 4e8d5c1
1 parent 38559fc
commit 4e8d5c1
Show file tree

Hide file tree

Showing 3 changed files with 10 additions and 5 deletions.
diff --git a/.php-cs-fixer.dist.php b/.php-cs-fixer.dist.php
@@ -17,7 +17,7 @@
 use PhpCsFixer\Runner\Parallel\ParallelConfigFactory;
 
 return (new Config())
-    ->setParallelConfig(ParallelConfigFactory::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
@@ -49,7 +49,7 @@ class Config implements ConfigInterface, ParallelAwareConfigInterface
 
     private string $name;
 
-    private ?ParallelConfig $parallelConfig;
+    private ParallelConfig $parallelConfig;
 
     /**
      * @var null|string
@@ -75,6 +75,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
@@ -124,8 +131,6 @@ public function getName(): string
 
     public function getParallelConfig(): ParallelConfig
     {
-        $this->parallelConfig ??= ParallelConfigFactory::sequential();
-
         return $this->parallelConfig;
     }
 

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(