Fixed code-style

andrey-helldar · andrey-helldar · commit fbfaa8330c87 · 2024-05-21T19:37:19.000+03:00
diff --git a/database/migrations/2022_08_18_180137_change_migration_actions_table.php b/database/migrations/2022_08_18_180137_change_migration_actions_table.php
@@ -2,8 +2,8 @@
 
 declare(strict_types=1);
 
-use DragonCode\LaravelDeployOperations\Operation;
 use DragonCode\LaravelDeployOperations\Helpers\Config;
+use DragonCode\LaravelDeployOperations\Operation;
 use Illuminate\Database\Schema\Blueprint;
 use Illuminate\Support\Facades\Schema;
 
diff --git a/database/migrations/2023_01_21_172923_rename_migrations_actions_table_to_actions.php b/database/migrations/2023_01_21_172923_rename_migrations_actions_table_to_actions.php
@@ -2,8 +2,8 @@
 
 declare(strict_types=1);
 
-use DragonCode\LaravelDeployOperations\Operation;
 use DragonCode\LaravelDeployOperations\Helpers\Config;
+use DragonCode\LaravelDeployOperations\Operation;
 use Illuminate\Support\Facades\Schema;
 
 return new class extends Operation {
diff --git a/database/migrations/2024_05_21_112438_rename_actions_table_to_operations.php b/database/migrations/2024_05_21_112438_rename_actions_table_to_operations.php
@@ -2,8 +2,8 @@
 
 declare(strict_types=1);
 
-use DragonCode\LaravelDeployOperations\Operation;
 use DragonCode\LaravelDeployOperations\Helpers\Config;
+use DragonCode\LaravelDeployOperations\Operation;
 use Illuminate\Support\Facades\Schema;
 
 return new class extends Operation {
@@ -28,13 +28,7 @@ public function down(): void
     protected function validateTable(string $name): void
     {
         if (Schema::hasTable($name)) {
-            throw new RuntimeException(
-                sprintf(
-                    'A table named [%s] already exists. Change the table name settings in the [%s] configuration file.',
-                    $name,
-                    'config/deploy-operations.php'
-                )
-            );
+            throw new RuntimeException(sprintf('A table named [%s] already exists. Change the table name settings in the [%s] configuration file.', $name, 'config/deploy-operations.php'));
         }
     }
 
diff --git a/src/Concerns/Optionable.php b/src/Concerns/Optionable.php
@@ -49,57 +49,57 @@ protected function availableOptions(): array
                 Options::Before,
                 null,
                 InputOption::VALUE_NONE,
-                'Run operations marked as before'
+                'Run operations marked as before',
             ],
             [
                 Options::Connection,
                 null,
                 InputOption::VALUE_OPTIONAL,
-                'The database connection to use'
+                'The database connection to use',
             ],
             [
                 Options::Force,
                 null,
                 InputOption::VALUE_NONE,
-                'Force the operation to run when in production'
+                'Force the operation to run when in production',
             ],
             [
                 Options::Path,
                 null,
                 InputOption::VALUE_OPTIONAL,
-                'The path to the operations files to be executed'
+                'The path to the operations files to be executed',
             ],
             [
                 Options::Realpath,
                 null,
                 InputOption::VALUE_NONE,
-                'Indicate any provided operation file paths are pre-resolved absolute path'
+                'Indicate any provided operation file paths are pre-resolved absolute path',
             ],
             [
                 Options::Step,
                 null,
                 InputOption::VALUE_OPTIONAL,
-                'Force the operations to be run so they can be rolled back individually'
+                'Force the operations to be run so they can be rolled back individually',
             ],
             [
                 Options::Mute,
                 null,
                 InputOption::VALUE_NONE,
-                'Turns off the output of informational messages'
+                'Turns off the output of informational messages',
             ],
             [
                 Options::Isolated,
                 null,
                 InputOption::VALUE_OPTIONAL,
                 'Do not run the operations command if another instance of the operations command is already running',
-                false
+                false,
             ],
             [
                 Options::Sync,
                 null,
                 InputOption::VALUE_OPTIONAL,
                 'Makes all operations run synchronously',
-                false
+                false,
             ],
         ];
     }
diff --git a/src/Console/Install.php b/src/Console/Install.php
@@ -12,5 +12,5 @@ class Install extends Command
 
     protected $description = 'Create the deploy operations repository';
 
-    protected Processor | string $processor = InstallProcessor::class;
+    protected Processor|string $processor = InstallProcessor::class;
 }
diff --git a/src/Console/Make.php b/src/Console/Make.php
@@ -13,7 +13,7 @@ class Make extends Command
 
     protected $description = 'Create a new deploy operation file';
 
-    protected Processor | string $processor = MakeProcessor::class;
+    protected Processor|string $processor = MakeProcessor::class;
 
     protected array $arguments = [
         Options::Name,
diff --git a/src/Console/Operations.php b/src/Console/Operations.php
@@ -13,7 +13,7 @@ class Operations extends Command
 
     protected $description = 'Run the deploy operations';
 
-    protected Processor | string $processor = OperationsProcessor::class;
+    protected Processor|string $processor = OperationsProcessor::class;
 
     protected bool $secure = false;
 
diff --git a/src/Console/Refresh.php b/src/Console/Refresh.php
@@ -13,7 +13,7 @@ class Refresh extends Command
 
     protected $description = 'Reset and re-run all deploy operations';
 
-    protected Processor | string $processor = RefreshProcessor::class;
+    protected Processor|string $processor = RefreshProcessor::class;
 
     protected array $options = [
         Options::Connection,
diff --git a/src/Console/Reset.php b/src/Console/Reset.php
@@ -13,7 +13,7 @@ class Reset extends Command
 
     protected $description = 'Rollback all deploy operations';
 
-    protected Processor | string $processor = ResetProcessor::class;
+    protected Processor|string $processor = ResetProcessor::class;
 
     protected array $options = [
         Options::Connection,
diff --git a/src/Console/Rollback.php b/src/Console/Rollback.php
@@ -13,7 +13,7 @@ class Rollback extends Command
 
     protected $description = 'Rollback the last deploy operation';
 
-    protected Processor | string $processor = RollbackProcessor::class;
+    protected Processor|string $processor = RollbackProcessor::class;
 
     protected array $options = [
         Options::Connection,
diff --git a/src/Console/Status.php b/src/Console/Status.php
@@ -13,7 +13,7 @@ class Status extends Command
 
     protected $description = 'Show the status of each deploy operation';
 
-    protected Processor | string $processor = StatusProcessor::class;
+    protected Processor|string $processor = StatusProcessor::class;
 
     protected bool $secure = false;
 
diff --git a/src/Console/Upgrade.php b/src/Console/Upgrade.php
@@ -14,5 +14,5 @@ class Upgrade extends Command
 
     protected $description = 'Upgrading project files from version 5 to 6';
 
-    protected Processor | string $processor = UpgradeProcessor::class;
+    protected Processor|string $processor = UpgradeProcessor::class;
 }
diff --git a/src/Constants/Names.php b/src/Constants/Names.php
@@ -4,14 +4,14 @@
 
 class Names
 {
-    public const Operations = 'operations';
-    public const Fresh = 'operations:fresh';
-    public const Install = 'operations:install';
-    public const Make = 'make:operation';
-    public const Refresh = 'operations:refresh';
-    public const Reset = 'operations:reset';
-    public const Rollback = 'operations:rollback';
-    public const Status = 'operations:status';
+    public const Operations  = 'operations';
+    public const Fresh       = 'operations:fresh';
+    public const Install     = 'operations:install';
+    public const Make        = 'make:operation';
+    public const Refresh     = 'operations:refresh';
+    public const Reset       = 'operations:reset';
+    public const Rollback    = 'operations:rollback';
+    public const Status      = 'operations:status';
     public const StubPublish = 'operations:stub';
-    public const Upgrade = 'operations:upgrade';
+    public const Upgrade     = 'operations:upgrade';
 }
diff --git a/src/Constants/Options.php b/src/Constants/Options.php
@@ -6,14 +6,14 @@
 
 class Options
 {
-    public const Before = 'before';
+    public const Before     = 'before';
     public const Connection = 'connection';
-    public const Force = 'force';
-    public const Isolated = 'isolated';
-    public const Mute = 'mute';
-    public const Name = 'name';
-    public const Path = 'path';
-    public const Realpath = 'realpath';
-    public const Step = 'step';
-    public const Sync = 'sync';
+    public const Force      = 'force';
+    public const Isolated   = 'isolated';
+    public const Mute       = 'mute';
+    public const Name       = 'name';
+    public const Path       = 'path';
+    public const Realpath   = 'realpath';
+    public const Step       = 'step';
+    public const Sync       = 'sync';
 }
diff --git a/src/Constants/Order.php b/src/Constants/Order.php
@@ -6,6 +6,6 @@
 
 class Order
 {
-    public const Asc = 'asc';
+    public const Asc  = 'asc';
     public const Desc = 'desc';
 }
diff --git a/src/Events/BaseEvent.php b/src/Events/BaseEvent.php
@@ -9,6 +9,5 @@ abstract class BaseEvent
     public function __construct(
         public string $method,
         public bool $before
-    ) {
-    }
+    ) {}
 }
diff --git a/src/Events/DeployOperationFailed.php b/src/Events/DeployOperationFailed.php
@@ -4,6 +4,4 @@
 
 namespace DragonCode\LaravelDeployOperations\Events;
 
-class DeployOperationFailed extends BaseEvent
-{
-}
+class DeployOperationFailed extends BaseEvent {}
diff --git a/src/Helpers/Config.php b/src/Helpers/Config.php
@@ -16,8 +16,7 @@ class Config
 {
     public function __construct(
         protected Repository $config
-    ) {
-    }
+    ) {}
 
     public function environment(): ?string
     {
@@ -36,15 +35,15 @@ public function table(): string
 
     public function exclude(): array
     {
-        return Arr::of((array)$this->config->get('deploy-operations.exclude'))
+        return Arr::of((array) $this->config->get('deploy-operations.exclude'))
             ->map(fn (string $path) => str_replace(['\\', '/'], DIRECTORY_SEPARATOR, $path))
             ->filter()
             ->toArray();
     }
 
     public function path(?string $path = null): string
     {
-        return rtrim($this->directory(), '\\/') . DIRECTORY_SEPARATOR . ltrim((string)$path, '\\/');
+        return rtrim($this->directory(), '\\/') . DIRECTORY_SEPARATOR . ltrim((string) $path, '\\/');
     }
 
     public function gitPath(): string
diff --git a/src/Notifications/Notification.php b/src/Notifications/Notification.php
@@ -73,6 +73,6 @@ public function setOutput(OutputStyle $output, bool $silent = false): Notificati
 
     protected function canSpeak(): bool
     {
-        return !$this->silent;
+        return ! $this->silent;
     }
 }
diff --git a/src/Processors/Fresh.php b/src/Processors/Fresh.php
@@ -26,8 +26,8 @@ protected function operations(): void
     {
         $this->runCommand(Names::Operations, [
             '--' . Options::Connection => $this->options->connection,
-            '--' . Options::Path => $this->options->path,
-            '--' . Options::Realpath => true,
+            '--' . Options::Path       => $this->options->path,
+            '--' . Options::Realpath   => true,
         ]);
     }
 }
diff --git a/src/Processors/Make.php b/src/Processors/Make.php
@@ -51,7 +51,7 @@ protected function getPath(): string
     protected function getFilename(string $branch): string
     {
         $directory = Path::dirname($branch);
-        $filename = Path::filename($branch);
+        $filename  = Path::filename($branch);
 
         return Str::of($filename)->prepend($this->getTime())->finish('.php')->prepend($directory . '/')->toString();
     }
diff --git a/src/Processors/Operations.php b/src/Processors/Operations.php
@@ -31,8 +31,8 @@ protected function ensureRepository(): void
     {
         $this->runCommand(Names::Install, [
             '--' . Options::Connection => $this->options->connection,
-            '--' . Options::Force => true,
-            '--' . Options::Mute => true,
+            '--' . Options::Force      => true,
+            '--' . Options::Mute       => true,
         ]);
     }
 
@@ -50,7 +50,8 @@ protected function runOperations(array $completed): void
             }
 
             $this->fireEvent(NoPendingDeployOperations::class, 'up');
-        } catch (Throwable $e) {
+        }
+        catch (Throwable $e) {
             $this->fireEvent(DeployOperationFailed::class, 'up');
 
             throw $e;
@@ -73,7 +74,7 @@ protected function getNewFiles(array $completed): array
     {
         return $this->getFiles(
             path: $this->options->path,
-            filter: fn (string $file) => !Str::of($file)->replace('\\', '/')->contains($completed)
+            filter: fn (string $file) => ! Str::of($file)->replace('\\', '/')->contains($completed)
         );
     }
 
diff --git a/src/Processors/Processor.php b/src/Processors/Processor.php
@@ -26,6 +26,8 @@ abstract class Processor
 {
     use Artisan;
 
+    abstract public function handle(): void;
+
     public function __construct(
         protected Options $options,
         protected InputInterface $input,
@@ -44,8 +46,6 @@ public function __construct(
         $this->migrator->setConnection($this->options->connection)->setOutput($this->output);
     }
 
-    abstract public function handle(): void;
-
     protected function getFiles(string $path, ?Closure $filter = null): array
     {
         $file = Str::finish($path, '.php');
@@ -54,7 +54,7 @@ protected function getFiles(string $path, ?Closure $filter = null): array
 
         $files = Arr::filter(
             $files,
-            fn (string $path) => Str::endsWith($path, '.php') && !Str::contains($path, $this->config->exclude())
+            fn (string $path) => Str::endsWith($path, '.php') && ! Str::contains($path, $this->config->exclude())
         );
 
         return Arr::of($this->sorter->byValues($files))
@@ -69,7 +69,7 @@ protected function runCommand(string $command, array $options = []): void
 
     protected function tableNotFound(): bool
     {
-        if (!$this->repository->repositoryExists()) {
+        if (! $this->repository->repositoryExists()) {
             $this->notification->warning('Deploy operations table not found');
 
             return true;
diff --git a/src/Processors/Rollback.php b/src/Processors/Rollback.php
@@ -46,7 +46,7 @@ protected function run(array $rows): void
 
     protected function getOperations(?int $step): array
     {
-        return (int)$step > 0
+        return (int) $step > 0
             ? $this->repository->getByStep($step)
             : $this->repository->getLast();
     }
diff --git a/src/Processors/Status.php b/src/Processors/Status.php
@@ -63,7 +63,7 @@ protected function merge(array $items, array $completed): array
 
     protected function getData(): array
     {
-        $files = $this->getFiles($this->options->path);
+        $files     = $this->getFiles($this->options->path);
         $completed = $this->getCompleted();
 
         return [$files, $completed];
diff --git a/src/Processors/StubPublish.php b/src/Processors/StubPublish.php
diff --git a/src/Repositories/OperationsRepository.php b/src/Repositories/OperationsRepository.php
diff --git a/src/Services/Migrator.php b/src/Services/Migrator.php
diff --git a/tests/Commands/OperationsTest.php b/tests/Commands/OperationsTest.php
diff --git a/tests/Commands/RollbackTest.php b/tests/Commands/RollbackTest.php

Original file line number	Diff line number	Diff line change
`@@ -12,5 +12,5 @@ class Install extends Command`
`12`	`12`
`13`	`13`	`protected $description = 'Create the deploy operations repository';`
`14`	`14`
`15`		`- protected Processor \| string $processor = InstallProcessor::class;`
	`15`	`+ protected Processor\|string $processor = InstallProcessor::class;`
`16`	`16`	`}`