[1.x] Adds --bail flag (#261)

* Adds `--bail` flag * Update DefaultCommand.php --------- Co-authored-by: Taylor Otwell <taylor@laravel.com>
laravel · Mar 20, 2024 · 2f46ab4 · 2f46ab4
1 parent fb4e240
commit 2f46ab4
Show file tree

Hide file tree

Showing 4 changed files with 5 additions and 4 deletions.
diff --git a/app/Actions/ElaborateSummary.php b/app/Actions/ElaborateSummary.php
@@ -41,7 +41,7 @@ public function execute($totalFiles, $changes)
             0,
             0,
             $this->output->getVerbosity() >= OutputInterface::VERBOSITY_VERBOSE,
-            $this->input->getOption('test'),
+            $this->input->getOption('test') || $this->input->getOption('bail'),
             $this->output->isDecorated()
         );
 

diff --git a/app/Commands/DefaultCommand.php b/app/Commands/DefaultCommand.php
@@ -39,6 +39,7 @@ protected function configure()
                     new InputOption('no-config', '', InputOption::VALUE_NONE, 'Disable loading any configuration file'),
                     new InputOption('preset', '', InputOption::VALUE_REQUIRED, 'The preset that should be used'),
                     new InputOption('test', '', InputOption::VALUE_NONE, 'Test for code style errors without fixing them'),
+                    new InputOption('bail', '', InputOption::VALUE_NONE, 'Test for code style errors without fixing them and stop on first error'),
                     new InputOption('dirty', '', InputOption::VALUE_NONE, 'Only fix files that have uncommitted changes'),
                     new InputOption('format', '', InputOption::VALUE_REQUIRED, 'The output format that should be used'),
                     new InputOption('cache-file', '', InputArgument::OPTIONAL, 'The path to the cache file'),

diff --git a/app/Factories/ConfigurationResolverFactory.php b/app/Factories/ConfigurationResolverFactory.php
@@ -54,7 +54,7 @@ public static function fromIO($input, $output)
                     sprintf('%s.php', $preset),
                 ]),
                 'diff' => $output->isVerbose(),
-                'dry-run' => $input->getOption('test'),
+                'dry-run' => $input->getOption('test') || $input->getOption('bail'),
                 'path' => $path,
                 'path-mode' => ConfigurationResolver::PATH_MODE_OVERRIDE,
                 'cache-file' => $input->getOption('cache-file') ?? $localConfiguration->cacheFile() ?? implode(DIRECTORY_SEPARATOR, [
@@ -65,7 +65,7 @@ public static function fromIO($input, $output)
                         : (string) microtime()
                     ),
                 ]),
-                'stop-on-violation' => false,
+                'stop-on-violation' => $input->getOption('bail'),
                 'verbosity' => $output->getVerbosity(),
                 'show-progress' => 'true',
             ],

diff --git a/app/Output/Concerns/InteractsWithSymbols.php b/app/Output/Concerns/InteractsWithSymbols.php
@@ -39,7 +39,7 @@ public function getSymbol($status)
         $statusSymbol = $this->statuses[$status];
 
         if (! isset($statusSymbol['symbol'])) {
-            $statusSymbol = $this->input->getOption('test')
+            $statusSymbol = ($this->input->getOption('test') || $this->input->getOption('bail'))
                 ? $statusSymbol[0]
                 : $statusSymbol[1];
         }