Merge pull request #201 from ByteInternet/Only-build-AfterTasks-that-are-Commands

Only execute AfterTasks that are Commands in the Commands-flow

Author: poespas

src/Deployer/Task/After/AfterTaskGlobal.php

@@ -4,7 +4,9 @@
 
 use Hypernode\Deploy\Deployer\Task\TaskBase;
 use Hypernode\Deploy\Deployer\TaskBuilder;
+use Hypernode\DeployConfiguration\Command\Command;
 use Hypernode\DeployConfiguration\Configuration;
+use Hypernode\DeployConfiguration\TaskConfigurationInterface;
 
 use function count;
 use function Deployer\task;
@@ -24,7 +26,12 @@ public function __construct(TaskBuilder $taskBuilder)
 
     public function configure(Configuration $config): void
     {
-        $tasks = $this->taskBuilder->buildAll($config->getAfterDeployTasks(), 'deploy:after');
+        $commands = array_values(array_filter(
+            $config->getAfterDeployTasks(),
+            fn (TaskConfigurationInterface $task): bool => $task instanceof Command
+        ));
+
+        $tasks = $this->taskBuilder->buildAll($commands, 'deploy:after');
         if (count($tasks) === 0) {
             $tasks = function (): void {
                 writeln('No after deploy tasks defined');

src/Deployer/TaskBuilder.php

@@ -19,7 +19,7 @@
 class TaskBuilder
 {
     /**
-     * @param TaskConfigurationInterface[] $commands
+     * @param Command[] $commands
      *
      * @param string $namePrefix
      * @return string[]
@@ -32,7 +32,6 @@ public function buildAll(array $commands, string $namePrefix): array
         foreach ($commands as $command) {
             $name = $namePrefix . ':' . \count($tasks);
 
-            /** @var Command $command */
             $this->build($command, $name);
 
             $tasks[] = $name;