Ran php-cs-fixer

php-etl · Jan 10, 2024 · 088a627 · 088a627
1 parent 2c758e4
commit 088a627
Show file tree

Hide file tree

Showing 190 changed files with 555 additions and 350 deletions.
diff --git a/src/Action/Action.php b/src/Action/Action.php
@@ -15,7 +15,9 @@
 
 final readonly class Action
 {
-    public function __construct(private ?string $plugin, private ExpressionLanguage $interpreter = new Satellite\ExpressionLanguage()) {}
+    public function __construct(private ?string $plugin, private ExpressionLanguage $interpreter = new Satellite\ExpressionLanguage())
+    {
+    }
 
     public function __invoke(array $config, ActionBuilder $action, RepositoryInterface $repository): void
     {

diff --git a/src/Action/ConfigurationApplier.php b/src/Action/ConfigurationApplier.php
@@ -17,7 +17,8 @@ public function __construct(
         private readonly string $plugin,
         private readonly FactoryInterface $service,
         private readonly ExpressionLanguage $interpreter,
-    ) {}
+    ) {
+    }
 
     public function withAction(): self
     {

diff --git a/src/Action/Custom/Builder/Action.php b/src/Action/Custom/Builder/Action.php
@@ -12,7 +12,9 @@ final class Action implements ActionBuilderInterface
     private ?Node\Expr $logger = null;
     private ?Node\Expr $state = null;
 
-    public function __construct(private readonly Node\Expr $service, private readonly string $containerNamespace) {}
+    public function __construct(private readonly Node\Expr $service, private readonly string $containerNamespace)
+    {
+    }
 
     public function withLogger(Node\Expr $logger): self
     {

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/Builder/Action.php b/src/Action/SFTP/Builder/Action.php
@@ -18,7 +18,8 @@ public function __construct(
         private readonly Node\Expr $password,
         private readonly Node\Expr $localFilePath,
         private readonly Node\Expr $serverFilePath,
-    ) {}
+    ) {
+    }
 
     public function withLogger(Node\Expr $logger): self
     {

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
@@ -10,9 +10,11 @@
 
 final readonly class Repository implements Configurator\RepositoryInterface
 {
-    public function __construct(private Builder\Action $builder) {}
+    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/AdapterChoice.php b/src/Adapter/AdapterChoice.php
@@ -10,7 +10,8 @@
 {
     public function __construct(
         private array $adapters,
-    ) {}
+    ) {
+    }
 
     public function __invoke(array $configuration): Configurator\SatelliteBuilderInterface
     {

diff --git a/src/Adapter/AdapterNotFoundException.php b/src/Adapter/AdapterNotFoundException.php
@@ -4,4 +4,6 @@
 
 namespace Kiboko\Component\Satellite\Adapter;
 
-final class AdapterNotFoundException extends \RuntimeException {}
+final class AdapterNotFoundException extends \RuntimeException
+{
+}
diff --git a/src/Adapter/Composer.php b/src/Adapter/Composer.php
@@ -8,6 +8,7 @@
 use Psr\Log\LoggerInterface;
 use React\ChildProcess\Process;
 use React\Promise\Deferred;
+
 use function React\Async\await;
 use function React\Promise\Timer\timeout;
 
@@ -28,16 +29,16 @@ private function execute(Process $process, float $timeout = 300): void
     {
         $process->start();
 
-        $process->stdout->on('data', function ($chunk) {
+        $process->stdout->on('data', function ($chunk): void {
             $this->logger->debug($chunk);
         });
-        $process->stderr->on('data', function ($chunk) {
+        $process->stderr->on('data', function ($chunk): void {
             $this->logger->info($chunk);
         });
 
         $deferred = new Deferred();
 
-        $process->on('exit', function () use ($deferred) {
+        $process->on('exit', function () use ($deferred): void {
             $deferred->resolve();
         });
 
@@ -53,7 +54,7 @@ private function execute(Process $process, float $timeout = 300): void
     private function command(string ...$command): void
     {
         $process = new Process(
-            implode (' ', array_map(fn ($part) => escapeshellarg($part), $command)),
+            implode(' ', array_map(fn ($part) => escapeshellarg($part), $command)),
             $this->workdir,
         );
 

diff --git a/src/Adapter/Docker/FileOrDirectoryNotFoundException.php b/src/Adapter/Docker/FileOrDirectoryNotFoundException.php
@@ -4,4 +4,6 @@
 
 namespace Kiboko\Component\Satellite\Adapter\Docker;
 
-class FileOrDirectoryNotFoundException extends \RuntimeException {}
+class FileOrDirectoryNotFoundException extends \RuntimeException
+{
+}
diff --git a/src/Adapter/Docker/NetworkInterface.php b/src/Adapter/Docker/NetworkInterface.php
@@ -4,4 +4,6 @@
 
 namespace Kiboko\Component\Satellite\Adapter\Docker;
 
-interface NetworkInterface {}
+interface NetworkInterface
+{
+}
diff --git a/src/Adapter/Docker/Satellite.php b/src/Adapter/Docker/Satellite.php
@@ -13,6 +13,7 @@
 use React\ChildProcess\Process;
 use React\Promise\Deferred;
 use React\Stream\ReadableResourceStream;
+
 use function React\Async\await;
 use function React\Promise\Timer\timeout;
 
@@ -27,7 +28,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;
     }
@@ -39,7 +40,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);
 
@@ -73,7 +74,7 @@ public function build(
         $command = ['docker', 'build', '--rm', '-', ...iterator_to_array($iterator($this->imageTags))];
 
         $process = new Process(
-            implode (' ', array_map(fn ($part) => escapeshellarg((string) $part), $command)),
+            implode(' ', array_map(fn ($part) => escapeshellarg((string) $part), $command)),
             $this->workdir,
         );
 
@@ -95,16 +96,16 @@ private function execute(
         Process $process,
         float $timeout = 300
     ): void {
-        $process->stdout->on('data', function ($chunk) use ($logger) {
+        $process->stdout->on('data', function ($chunk) use ($logger): void {
             $logger->debug($chunk);
         });
-        $process->stderr->on('data', function ($chunk) use ($logger) {
+        $process->stderr->on('data', function ($chunk) use ($logger): void {
             $logger->info($chunk);
         });
 
         $deferred = new Deferred();
 
-        $process->on('exit', function () use ($deferred) {
+        $process->on('exit', function () use ($deferred): void {
             $deferred->resolve();
         });
 

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/FactoryInterface.php b/src/Adapter/FactoryInterface.php
@@ -2,9 +2,9 @@
 
 declare(strict_types=1);
 
-trigger_deprecation('php-etl/satellite', '0.4', 'The "%s" interface is deprecated, use "%s" instead.', 'Kiboko\\Component\\Satellite\\Adapter\\FactoryInterface', \Kiboko\Contract\Configurator\Adapter\FactoryInterface::class);
+trigger_deprecation('php-etl/satellite', '0.4', 'The "%s" interface is deprecated, use "%s" instead.', 'Kiboko\\Component\\Satellite\\Adapter\\FactoryInterface', Kiboko\Contract\Configurator\Adapter\FactoryInterface::class);
 
 /*
  * @deprecated since Satellite 0.4, use Kiboko\Contract\Configurator\Adapter\FactoryInterface instead.
  */
-class_alias(\Kiboko\Contract\Configurator\Adapter\FactoryInterface::class, 'Kiboko\\Component\\Satellite\\Adapter\\FactoryInterface');
+class_alias(Kiboko\Contract\Configurator\Adapter\FactoryInterface::class, 'Kiboko\\Component\\Satellite\\Adapter\\FactoryInterface');
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);
-Original file line number
+Diff line change
@@ Expand Up / @@ -10,7 +10,8 @@ @@
     {
         public function __construct(
             private array $adapters,
-        ) {}
+        ) {
+        }
         public function __invoke(array $configuration): Configurator\SatelliteBuilderInterface
         {
@@ Expand Down @@