fix: Fix run-options

factorial-io · Feb 3, 2025 · ca6336c · ca6336c
1 parent ebd0996
commit ca6336c
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/src/Artifact/Actions/Base/CopyAction.php b/src/Artifact/Actions/Base/CopyAction.php
@@ -5,6 +5,7 @@
 use Phabalicious\Artifact\Actions\ActionBase;
 use Phabalicious\Configuration\HostConfig;
 use Phabalicious\Method\TaskContextInterface;
+use Phabalicious\ShellProvider\RunOptions;
 use Phabalicious\ShellProvider\ShellProviderInterface;
 use Phabalicious\Validation\ValidationService;
 
@@ -18,7 +19,7 @@ protected function validateArgumentsConfig(array $action_arguments, ValidationSe
 
     private function getDirectoryContents(ShellProviderInterface $shell, $install_dir): array
     {
-        $contents = $shell->run('ls -1a '.$install_dir, true);
+        $contents = $shell->run('ls -1a '.$install_dir, RunOptions::CAPTURE_AND_HIDE_OUTPUT);
 
         return array_filter($contents->getOutput(), function ($elem) {
             return !in_array($elem, ['.', '..']);

diff --git a/src/Artifact/Actions/Base/DockerCopyAction.php b/src/Artifact/Actions/Base/DockerCopyAction.php
@@ -42,7 +42,7 @@ protected function getDirectoryContents(ShellProviderInterface $shell, string $p
             'docker run --entrypoint /bin/ls --rm %s -1a %s',
             $this->dockerImageName,
             $path
-        ), true);
+        ), RunOptions::CAPTURE_AND_HIDE_OUTPUT);
 
         if ($result->failed()) {
             $result->throwException('Could not get directory contents from docker image!');