use php cs fixer

php-etl · Oct 20, 2023 · c7dab8d · c7dab8d
1 parent 92cb153
commit c7dab8d
Show file tree

Hide file tree

Showing 62 changed files with 121 additions and 121 deletions.
diff --git a/.github/workflows/quality.yaml b/.github/workflows/quality.yaml
@@ -14,7 +14,7 @@ jobs:
                 run: |
                     wget -q https://cs.symfony.com/download/php-cs-fixer-v3.phar -O php-cs-fixer
                     chmod a+x php-cs-fixer
-                    PHP_CS_FIXER_IGNORE_ENV=true ./php-cs-fixer fix src --dry-run
+                    ./php-cs-fixer fix src --dry-run
 
     phpstan:
         runs-on: ubuntu-latest

diff --git a/src/Action/Custom/Factory/Action.php b/src/Action/Custom/Factory/Action.php
@@ -44,7 +44,7 @@ public function normalize(array $config): array
     {
         try {
             return $this->processor->processConfiguration($this->configuration, $config);
-        } catch (Symfony\InvalidTypeException|Symfony\InvalidConfigurationException $exception) {
+        } catch (Symfony\InvalidConfigurationException|Symfony\InvalidTypeException $exception) {
             throw new Configurator\InvalidConfigurationException($exception->getMessage(), 0, $exception);
         }
     }
@@ -55,7 +55,7 @@ public function validate(array $config): bool
             $this->processor->processConfiguration($this->configuration, $config);
 
             return true;
-        } catch (Symfony\InvalidTypeException|Symfony\InvalidConfigurationException) {
+        } catch (Symfony\InvalidConfigurationException|Symfony\InvalidTypeException) {
             return false;
         }
     }

diff --git a/src/Action/Custom/Factory/Repository/RepositoryTrait.php b/src/Action/Custom/Factory/Repository/RepositoryTrait.php
@@ -14,7 +14,7 @@ trait RepositoryTrait
     /** @var string[] */
     private array $packages;
 
-    public function addFiles(Packaging\FileInterface|Packaging\DirectoryInterface ...$files): Configurator\RepositoryInterface
+    public function addFiles(Packaging\DirectoryInterface|Packaging\FileInterface ...$files): Configurator\RepositoryInterface
     {
         array_push($this->files, ...$files);
 

diff --git a/src/Action/Custom/Service.php b/src/Action/Custom/Service.php
@@ -43,7 +43,7 @@ public function normalize(array $config): array
     {
         try {
             return $this->processor->processConfiguration($this->configuration, $config);
-        } catch (Symfony\InvalidTypeException|Symfony\InvalidConfigurationException $exception) {
+        } catch (Symfony\InvalidConfigurationException|Symfony\InvalidTypeException $exception) {
             throw new Configurator\InvalidConfigurationException($exception->getMessage(), 0, $exception);
         }
     }
@@ -54,7 +54,7 @@ public function validate(array $config): bool
             $this->processor->processConfiguration($this->configuration, $config);
 
             return true;
-        } catch (Symfony\InvalidTypeException|Symfony\InvalidConfigurationException) {
+        } catch (Symfony\InvalidConfigurationException|Symfony\InvalidTypeException) {
             return false;
         }
     }

diff --git a/src/Action/SFTP/Factory/Action.php b/src/Action/SFTP/Factory/Action.php
@@ -39,7 +39,7 @@ public function normalize(array $config): array
     {
         try {
             return $this->processor->processConfiguration($this->configuration, $config);
-        } catch (Symfony\InvalidTypeException|Symfony\InvalidConfigurationException $exception) {
+        } catch (Symfony\InvalidConfigurationException|Symfony\InvalidTypeException $exception) {
             throw new Configurator\InvalidConfigurationException($exception->getMessage(), 0, $exception);
         }
     }
@@ -50,7 +50,7 @@ public function validate(array $config): bool
             $this->normalize($config);
 
             return true;
-        } catch (Symfony\InvalidTypeException|Symfony\InvalidConfigurationException) {
+        } catch (Symfony\InvalidConfigurationException|Symfony\InvalidTypeException) {
             return false;
         }
     }
@@ -68,7 +68,7 @@ public function compile(array $config): Repository
 
         try {
             return new Repository($builder);
-        } catch (Symfony\InvalidTypeException|Symfony\InvalidConfigurationException $exception) {
+        } catch (Symfony\InvalidConfigurationException|Symfony\InvalidTypeException $exception) {
             throw new Configurator\InvalidConfigurationException(message: $exception->getMessage(), previous: $exception);
         }
     }

diff --git a/src/Action/SFTP/Factory/Repository/Repository.php b/src/Action/SFTP/Factory/Repository/Repository.php
@@ -12,7 +12,7 @@
 {
     public function __construct(private Builder\Action $builder) {}
 
-    public function addFiles(Packaging\FileInterface|Packaging\DirectoryInterface ...$files): self
+    public function addFiles(Packaging\DirectoryInterface|Packaging\FileInterface ...$files): self
     {
         return $this;
     }

diff --git a/src/Action/SFTP/Service.php b/src/Action/SFTP/Service.php
@@ -45,7 +45,7 @@ public function normalize(array $config): array
     {
         try {
             return $this->processor->processConfiguration($this->configuration, $config);
-        } catch (Symfony\InvalidTypeException|Symfony\InvalidConfigurationException $exception) {
+        } catch (Symfony\InvalidConfigurationException|Symfony\InvalidTypeException $exception) {
             throw new Configurator\InvalidConfigurationException($exception->getMessage(), 0, $exception);
         }
     }
@@ -56,7 +56,7 @@ public function validate(array $config): bool
             $this->processor->processConfiguration($this->configuration, $config);
 
             return true;
-        } catch (Symfony\InvalidTypeException|Symfony\InvalidConfigurationException) {
+        } catch (Symfony\InvalidConfigurationException|Symfony\InvalidTypeException) {
             return false;
         }
     }

diff --git a/src/Adapter/Docker/Satellite.php b/src/Adapter/Docker/Satellite.php
@@ -22,7 +22,7 @@ final class Satellite implements Configurator\SatelliteInterface
     public function __construct(
         private readonly Dockerfile\Dockerfile $dockerfile,
         private readonly string $workdir,
-        Packaging\FileInterface|Packaging\DirectoryInterface ...$files
+        Packaging\DirectoryInterface|Packaging\FileInterface ...$files
     ) {
         $this->files = $files;
     }
@@ -34,7 +34,7 @@ public function addTags(string ...$imageTags): self
         return $this;
     }
 
-    public function withFile(Packaging\FileInterface|Packaging\DirectoryInterface ...$files): self
+    public function withFile(Packaging\DirectoryInterface|Packaging\FileInterface ...$files): self
     {
         array_push($this->files, ...$files);
 

diff --git a/src/Adapter/Docker/SatelliteBuilder.php b/src/Adapter/Docker/SatelliteBuilder.php
@@ -23,8 +23,8 @@ final class SatelliteBuilder implements Configurator\SatelliteBuilderInterface
     private iterable $command = [];
     /** @var iterable<string> */
     private iterable $tags = [];
-    private null|PackagingContract\FileInterface|PackagingContract\AssetInterface $composerJsonFile = null;
-    private null|PackagingContract\FileInterface|PackagingContract\AssetInterface $composerLockFile = null;
+    private null|PackagingContract\AssetInterface|PackagingContract\FileInterface $composerJsonFile = null;
+    private null|PackagingContract\AssetInterface|PackagingContract\FileInterface $composerLockFile = null;
     /** @var iterable<array<string, string>> */
     private iterable $paths = [];
     /** @var \AppendIterator<string,PackagingContract\FileInterface, \Iterator<string,PackagingContract\FileInterface>> */
@@ -66,8 +66,8 @@ public function withComposerRequire(string ...$package): self
     }
 
     public function withComposerFile(
-        PackagingContract\FileInterface|PackagingContract\AssetInterface $composerJsonFile,
-        PackagingContract\FileInterface|PackagingContract\AssetInterface $composerLockFile = null
+        PackagingContract\AssetInterface|PackagingContract\FileInterface $composerJsonFile,
+        PackagingContract\AssetInterface|PackagingContract\FileInterface $composerLockFile = null
     ): self {
         $this->composerJsonFile = $composerJsonFile;
         $this->composerLockFile = $composerLockFile;
@@ -76,7 +76,7 @@ public function withComposerFile(
     }
 
     public function withFile(
-        PackagingContract\FileInterface|PackagingContract\AssetInterface $source,
+        PackagingContract\AssetInterface|PackagingContract\FileInterface $source,
         string $destinationPath = null
     ): self {
         if (!$source instanceof PackagingContract\FileInterface) {

diff --git a/src/Adapter/Filesystem/Satellite.php b/src/Adapter/Filesystem/Satellite.php
@@ -18,12 +18,12 @@ final class Satellite implements Configurator\SatelliteInterface
     public function __construct(
         private readonly string $workdir,
         private readonly Composer $composer,
-        Packaging\FileInterface|Packaging\DirectoryInterface ...$files
+        Packaging\DirectoryInterface|Packaging\FileInterface ...$files
     ) {
         $this->files = $files;
     }
 
-    public function withFile(Packaging\FileInterface|Packaging\DirectoryInterface ...$files): self
+    public function withFile(Packaging\DirectoryInterface|Packaging\FileInterface ...$files): self
     {
         array_push($this->files, ...$files);
 

diff --git a/src/Adapter/Filesystem/SatelliteBuilder.php b/src/Adapter/Filesystem/SatelliteBuilder.php
@@ -20,8 +20,8 @@ final class SatelliteBuilder implements Configurator\SatelliteBuilderInterface
     ];
     private array $authenticationTokens = [];
     private array $repositories = [];
-    private null|PackagingContract\FileInterface|PackagingContract\AssetInterface $composerJsonFile = null;
-    private null|PackagingContract\FileInterface|PackagingContract\AssetInterface $composerLockFile = null;
+    private null|PackagingContract\AssetInterface|PackagingContract\FileInterface $composerJsonFile = null;
+    private null|PackagingContract\AssetInterface|PackagingContract\FileInterface $composerLockFile = null;
     /** @var iterable<array<string, string>> */
     private iterable $paths = [];
     /** @var \AppendIterator<string,PackagingContract\FileInterface, \Iterator<string,PackagingContract\FileInterface>> */
@@ -54,8 +54,8 @@ public function withComposerRequire(string ...$package): self
     }
 
     public function withComposerFile(
-        PackagingContract\FileInterface|PackagingContract\AssetInterface $composerJsonFile,
-        PackagingContract\FileInterface|PackagingContract\AssetInterface $composerLockFile = null
+        PackagingContract\AssetInterface|PackagingContract\FileInterface $composerJsonFile,
+        PackagingContract\AssetInterface|PackagingContract\FileInterface $composerLockFile = null
     ): self {
         $this->composerJsonFile = $composerJsonFile;
         $this->composerLockFile = $composerLockFile;
@@ -64,7 +64,7 @@ public function withComposerFile(
     }
 
     public function withFile(
-        PackagingContract\FileInterface|PackagingContract\AssetInterface $source,
+        PackagingContract\AssetInterface|PackagingContract\FileInterface $source,
         string $destinationPath = null
     ): self {
         if (!$source instanceof PackagingContract\FileInterface) {

diff --git a/src/Adapter/Tar/Satellite.php b/src/Adapter/Tar/Satellite.php
@@ -19,7 +19,7 @@ final class Satellite implements Configurator\SatelliteInterface
 
     public function __construct(
         private readonly string $outputPath,
-        Packaging\FileInterface|Packaging\DirectoryInterface ...$files
+        Packaging\DirectoryInterface|Packaging\FileInterface ...$files
     ) {
         $this->files = $files;
     }
@@ -31,7 +31,7 @@ public function addTags(string ...$imageTags): self
         return $this;
     }
 
-    public function withFile(Packaging\FileInterface|Packaging\DirectoryInterface ...$files): self
+    public function withFile(Packaging\DirectoryInterface|Packaging\FileInterface ...$files): self
     {
         array_push($this->files, ...$files);
 

diff --git a/src/Adapter/Tar/SatelliteBuilder.php b/src/Adapter/Tar/SatelliteBuilder.php
@@ -13,8 +13,8 @@ final class SatelliteBuilder implements Configurator\SatelliteBuilderInterface
 {
     /** @var iterable<string> */
     private iterable $composerRequire = [];
-    private null|PackagingContract\FileInterface|PackagingContract\AssetInterface $composerJsonFile = null;
-    private null|PackagingContract\FileInterface|PackagingContract\AssetInterface $composerLockFile = null;
+    private null|PackagingContract\AssetInterface|PackagingContract\FileInterface $composerJsonFile = null;
+    private null|PackagingContract\AssetInterface|PackagingContract\FileInterface $composerLockFile = null;
     /** @var \AppendIterator<string,PackagingContract\FileInterface, \Iterator<string,PackagingContract\FileInterface>> */
     private readonly iterable $files;
     /** @var array<string, array<string, string>> */
@@ -47,8 +47,8 @@ public function withComposerRequire(string ...$package): self
     }
 
     public function withComposerFile(
-        PackagingContract\FileInterface|PackagingContract\AssetInterface $composerJsonFile,
-        PackagingContract\FileInterface|PackagingContract\AssetInterface $composerLockFile = null
+        PackagingContract\AssetInterface|PackagingContract\FileInterface $composerJsonFile,
+        PackagingContract\AssetInterface|PackagingContract\FileInterface $composerLockFile = null
     ): self {
         $this->composerJsonFile = $composerJsonFile;
         $this->composerLockFile = $composerLockFile;
@@ -57,7 +57,7 @@ public function withComposerFile(
     }
 
     public function withFile(
-        PackagingContract\FileInterface|PackagingContract\AssetInterface $source,
+        PackagingContract\AssetInterface|PackagingContract\FileInterface $source,
         string $destinationPath = null
     ): self {
         if (!$source instanceof PackagingContract\FileInterface) {

diff --git a/src/Builder/Action.php b/src/Builder/Action.php
@@ -16,8 +16,8 @@ public function __construct(
     ) {}
 
     public function addAction(
-        Node\Expr|Builder $loader,
-        Node\Expr|Builder $state,
+        Builder|Node\Expr $loader,
+        Builder|Node\Expr $state,
     ): self {
         $this->action = new Node\Expr\MethodCall(
             var: $this->runtime,

diff --git a/src/Builder/Pipeline.php b/src/Builder/Pipeline.php
@@ -16,9 +16,9 @@ public function __construct(
     ) {}
 
     public function addExtractor(
-        Node\Expr|Builder $extractor,
-        Node\Expr|Builder $rejection,
-        Node\Expr|Builder $state,
+        Builder|Node\Expr $extractor,
+        Builder|Node\Expr $rejection,
+        Builder|Node\Expr $state,
     ): self {
         $this->steps[] = fn (Node\Expr $runtime) => new Node\Expr\MethodCall(
             var: $runtime,
@@ -34,9 +34,9 @@ public function addExtractor(
     }
 
     public function addTransformer(
-        Node\Expr|Builder $transformer,
-        Node\Expr|Builder $rejection,
-        Node\Expr|Builder $state,
+        Builder|Node\Expr $transformer,
+        Builder|Node\Expr $rejection,
+        Builder|Node\Expr $state,
     ): self {
         $this->steps[] = fn (Node\Expr $runtime) => new Node\Expr\MethodCall(
             var: $runtime,
@@ -52,9 +52,9 @@ public function addTransformer(
     }
 
     public function addLoader(
-        Node\Expr|Builder $loader,
-        Node\Expr|Builder $rejection,
-        Node\Expr|Builder $state,
+        Builder|Node\Expr $loader,
+        Builder|Node\Expr $rejection,
+        Builder|Node\Expr $state,
     ): self {
         $this->steps[] = fn (Node\Expr $runtime) => new Node\Expr\MethodCall(
             var: $runtime,

diff --git a/src/Builder/Repository/API.php b/src/Builder/Repository/API.php
@@ -17,7 +17,7 @@ final class API implements Configurator\RepositoryInterface
 
     public function __construct(private readonly Satellite\Builder\API $builder) {}
 
-    public function addFiles(Packaging\FileInterface|Packaging\DirectoryInterface ...$files): Configurator\RepositoryInterface
+    public function addFiles(Packaging\DirectoryInterface|Packaging\FileInterface ...$files): Configurator\RepositoryInterface
     {
         array_push($this->files, ...$files);
 

diff --git a/src/Builder/Repository/Action.php b/src/Builder/Repository/Action.php
@@ -17,7 +17,7 @@ final class Action implements Configurator\RepositoryInterface
 
     public function __construct(private readonly Satellite\Builder\Action $builder) {}
 
-    public function addFiles(Packaging\FileInterface|Packaging\DirectoryInterface ...$files): Configurator\RepositoryInterface
+    public function addFiles(Packaging\DirectoryInterface|Packaging\FileInterface ...$files): Configurator\RepositoryInterface
     {
         array_push($this->files, ...$files);
 

diff --git a/src/Builder/Repository/Hook.php b/src/Builder/Repository/Hook.php
@@ -17,7 +17,7 @@ final class Hook implements Configurator\RepositoryInterface
 
     public function __construct(private readonly Satellite\Builder\Hook $builder) {}
 
-    public function addFiles(Packaging\FileInterface|Packaging\DirectoryInterface ...$files): Configurator\RepositoryInterface
+    public function addFiles(Packaging\DirectoryInterface|Packaging\FileInterface ...$files): Configurator\RepositoryInterface
     {
         array_push($this->files, ...$files);
 

diff --git a/src/Builder/Repository/Pipeline.php b/src/Builder/Repository/Pipeline.php
@@ -17,7 +17,7 @@ final class Pipeline implements Configurator\RepositoryInterface
 
     public function __construct(private readonly Satellite\Builder\Pipeline $builder) {}
 
-    public function addFiles(Packaging\FileInterface|Packaging\DirectoryInterface ...$files): Configurator\RepositoryInterface
+    public function addFiles(Packaging\DirectoryInterface|Packaging\FileInterface ...$files): Configurator\RepositoryInterface
     {
         array_push($this->files, ...$files);
 

diff --git a/src/Builder/Repository/Workflow.php b/src/Builder/Repository/Workflow.php
@@ -17,7 +17,7 @@ final class Workflow implements Configurator\RepositoryInterface
 
     public function __construct(private readonly Satellite\Builder\Workflow $builder) {}
 
-    public function addFiles(Packaging\FileInterface|Packaging\DirectoryInterface ...$files): Configurator\RepositoryInterface
+    public function addFiles(Packaging\DirectoryInterface|Packaging\FileInterface ...$files): Configurator\RepositoryInterface
     {
         array_push($this->files, ...$files);
 

diff --git a/src/Cloud/Console/Command/CreateCommand.php b/src/Cloud/Console/Command/CreateCommand.php
@@ -82,7 +82,7 @@ protected function execute(Console\Input\InputInterface $input, Console\Output\O
 
         try {
             $configuration = $service->normalize($configuration);
-        } catch (Config\Definition\Exception\InvalidTypeException|Config\Definition\Exception\InvalidConfigurationException $exception) {
+        } catch (Config\Definition\Exception\InvalidConfigurationException|Config\Definition\Exception\InvalidTypeException $exception) {
             $style->error($exception->getMessage());
 
             return self::FAILURE;

diff --git a/src/Cloud/Console/Command/RemoveCommand.php b/src/Cloud/Console/Command/RemoveCommand.php
@@ -82,7 +82,7 @@ protected function execute(Console\Input\InputInterface $input, Console\Output\O
 
         try {
             $configuration = $service->normalize($configuration);
-        } catch (Config\Definition\Exception\InvalidTypeException|Config\Definition\Exception\InvalidConfigurationException $exception) {
+        } catch (Config\Definition\Exception\InvalidConfigurationException|Config\Definition\Exception\InvalidTypeException $exception) {
             $style->error($exception->getMessage());
 
             return self::FAILURE;

diff --git a/src/Cloud/Console/Command/UpdateCommand.php b/src/Cloud/Console/Command/UpdateCommand.php
@@ -82,7 +82,7 @@ protected function execute(Console\Input\InputInterface $input, Console\Output\O
 
         try {
             $configuration = $service->normalize($configuration);
-        } catch (Config\Definition\Exception\InvalidTypeException|Config\Definition\Exception\InvalidConfigurationException $exception) {
+        } catch (Config\Definition\Exception\InvalidConfigurationException|Config\Definition\Exception\InvalidTypeException $exception) {
             $style->error($exception->getMessage());
 
             return self::FAILURE;

diff --git a/src/Console/Command/BuildCommand.php b/src/Console/Command/BuildCommand.php
@@ -67,7 +67,7 @@ protected function execute(Console\Input\InputInterface $input, Console\Output\O
 
         try {
             $configuration = $service->normalize($configuration);
-        } catch (Config\Definition\Exception\InvalidTypeException|Config\Definition\Exception\InvalidConfigurationException $exception) {
+        } catch (Config\Definition\Exception\InvalidConfigurationException|Config\Definition\Exception\InvalidTypeException $exception) {
             $style->error($exception->getMessage());
 
             return 255;

diff --git a/src/Console/Command/ValidateCommand.php b/src/Console/Command/ValidateCommand.php
@@ -66,7 +66,7 @@ protected function execute(Console\Input\InputInterface $input, Console\Output\O
 
         try {
             $configuration = $service->normalize($configuration);
-        } catch (Config\Definition\Exception\InvalidTypeException|Config\Definition\Exception\InvalidConfigurationException $exception) {
+        } catch (Config\Definition\Exception\InvalidConfigurationException|Config\Definition\Exception\InvalidTypeException $exception) {
             $style->error($exception->getMessage());
 
             return 255;