Remove unneeded outputs from shell_exec

Author: kylekatarnls

src/Phug/Split/Command/CommandBase.php

@@ -109,6 +109,10 @@ protected function getCurrentLinkedCommitHash(): ?string
      */
     protected function remove(string $fileOrDirectory): bool
     {
+        if (!is_writable($fileOrDirectory)) {
+            return false;
+        }
+
         @shell_exec('rm -rf '.escapeshellarg($fileOrDirectory).' 2>&1');
         file_exists($fileOrDirectory) && @shell_exec('rmdir /S /Q '.escapeshellarg($fileOrDirectory).' 2>&1');
 

src/Phug/Split/Command/Dist.php

@@ -131,9 +131,7 @@ protected function distributePackage(Split $cli, array $package, string $branch)
         $this->info($cli, ' => '.($branchRevision === '' ? 'no revision' : $branchRevision));
         $option = preg_match('/^[0-9a-f]+$/i', $branchRevision) ? '' : ' -b';
         $cli->writeLine("git checkout$option $branch", 'light_green');
-        $cli->gray();
-        $this->git("checkout$option $branch");
-        $cli->discolor();
+        $this->git("checkout$option $branch", [], '2>&1');
 
         return true;
     }

tests/Phug/Command/DistTest.php

@@ -101,7 +101,7 @@ public function testRun()
             'done.',
             "#[0m#[1;34mcd {$path}dist/vendor/sub-package",
             '#[0m#[1;32mgit checkout master',
-            "#[0m#[1;30m#[0m#[1;35mBuild distributed in {$path}dist",
+            "#[0m#[1;35mBuild distributed in {$path}dist",
             '#[0m',
         ]);
         $this->assertSame($expected, $output);