consolidation / robo

src/Task/Testing/Codecept.php

Unused Use Statements -1 removed lines

@@ -5,7 +5,6 @@
 use Robo\Exception\TaskException;
 use Robo\Task\BaseTask;
 use Robo\Contract\CommandInterface;
-use Symfony\Component\Process\Process;
 
 /**
  * Executes Codeception tests

src/Task/Assets/ImageMinify.php

Spacing +10 added lines, -10 removed lines

@@ -182,12 +182,12 @@ 
         // guess the best path for the executables based on __DIR__
         if (($pos = strpos(__DIR__, 'consolidation/robo')) !== false) {
             // the executables should be stored in vendor/bin
-            $this->executableTargetDir = substr(__DIR__, 0, $pos).'bin';
+            $this->executableTargetDir = substr(__DIR__, 0, $pos) . 'bin';
         }
 
         // check if the executables are already available
         foreach ($this->imageminRepos as $exec => $url) {
-            $path = $this->executableTargetDir.'/'.$exec;
+            $path = $this->executableTargetDir . '/' . $exec;
             // if this is Windows add a .exe extension
             if (substr($this->getOS(), 0, 3) == 'win') {
                 $path .= '.exe';
@@ -289,7 +289,7 @@ 
             } catch (\InvalidArgumentException $e) {
                 // if finder cannot handle it, try with in()->name()
                 if (strpos($dir, '/') === false) {
-                    $dir = './'.$dir;
+                    $dir = './' . $dir;
                 }
                 $parts = explode('/', $dir);
                 $new_dir = implode('/', array_slice($parts, 0, -1));
@@ -319,7 +319,7 @@ 
      */
     protected function getTarget($file, $to)
     {
-        $target = $to.'/'.basename($file);
+        $target = $to . '/' . basename($file);
 
         return $target;
     }
@@ -432,7 +432,7 @@ 
     protected function getOS()
     {
         $os = php_uname('s');
-        $os .= '/'.php_uname('m');
+        $os .= '/' . php_uname('m');
         // replace x86_64 to x64, because the imagemin repo uses that
         $os = str_replace('x86_64', 'x64', $os);
         // replace i386, i686, etc to x86, because of imagemin
@@ -492,10 +492,10 @@ 
         $this->printTaskInfo('Downloading the {executable} executable from the imagemin repository', ['executable' => $executable]);
 
         $os = $this->getOS();
-        $url = $this->imageminRepos[$executable].'/blob/master/vendor/'.$os.'/'.$executable.'?raw=true';
+        $url = $this->imageminRepos[$executable] . '/blob/master/vendor/' . $os . '/' . $executable . '?raw=true';
         if (substr($os, 0, 3) == 'win') {
             // if it is win, add a .exe extension
-            $url = $this->imageminRepos[$executable].'/blob/master/vendor/'.$os.'/'.$executable.'.exe?raw=true';
+            $url = $this->imageminRepos[$executable] . '/blob/master/vendor/' . $os . '/' . $executable . '.exe?raw=true';
         }
         $data = @file_get_contents($url, false, null);
         if ($data === false) {
@@ -525,10 +525,10 @@ 
             mkdir($this->executableTargetDir);
         }
         // save the executable into the target dir
-        $path = $this->executableTargetDir.'/'.$executable;
+        $path = $this->executableTargetDir . '/' . $executable;
         if (substr($os, 0, 3) == 'win') {
             // if it is win, add a .exe extension
-            $path = $this->executableTargetDir.'/'.$executable.'.exe';
+            $path = $this->executableTargetDir . '/' . $executable . '.exe';
         }
         $result = file_put_contents($path, $data);
         if ($result === false) {
@@ -540,7 +540,7 @@ 
         chmod($path, 0755);
 
         // if everything successful, store the executable path
-        $this->executablePaths[$executable] = $this->executableTargetDir.'/'.$executable;
+        $this->executablePaths[$executable] = $this->executableTargetDir . '/' . $executable;
         // if it is win, add a .exe extension
         if (substr($os, 0, 3) == 'win') {
             $this->executablePaths[$executable] .= '.exe';

src/Task/Testing/Atoum.php

Indentation -1 removed lines

@@ -83,7 +83,6 @@
 
     /**
      * Path to the bootstrap file.
-
      * @param string $file
      *
      * @return $this

Doc Comments +3 added lines, -1 removed lines

@@ -48,7 +48,7 @@ 
     /**
      * Tag or Tags to filter.
      *
-     * @param string|string[] $tags
+     * @param string $tags
      *
      * @return $this
      */
@@ -125,6 +125,7 @@ 
      * Test file or test files to run.
      *
      * @param string|string[]
+     * @param string[] $files
      *
      * @return $this
      */
@@ -138,6 +139,7 @@ 
      *
      * @param string|string[]
      *   A single directory or a list of directories.
+     * @param string $directories
      *
      * @return $this
      */

src/Task/Development/PackPhar.php

Spacing +1 added lines, -1 removed lines

@@ -129,7 +129,7 @@
     {
         // run() will call advanceProgressIndicator() once for each
         // file, one after calling stopBuffering, and again after compression.
-        return count($this->files)+2;
+        return count($this->files) + 2;
     }
 
     /**

src/Task/Development/OpenBrowser.php

Spacing +1 added lines, -1 removed lines

@@ -35,7 +35,7 @@
      */
     public function __construct($url)
     {
-        $this->urls = (array) $url;
+        $this->urls = (array)$url;
     }
 
     /**

src/Result.php

Spacing +1 added lines, -1 removed lines

@@ -189,7 +189,7 @@
      */
     public function cloneTask()
     {
-        $reflect  = new \ReflectionClass(get_class($this->task));
+        $reflect = new \ReflectionClass(get_class($this->task));
         return $reflect->newInstanceArgs(func_get_args());
     }
 

src/Log/ResultPrinter.php

Unused Use Statements -1 removed lines

@@ -6,7 +6,6 @@
 use Robo\Contract\ProgressIndicatorAwareInterface;
 use Robo\Contract\VerbosityThresholdInterface;
 use Robo\Common\ProgressIndicatorAwareTrait;
-
 use Psr\Log\LogLevel;
 use Psr\Log\LoggerAwareInterface;
 use Psr\Log\LoggerAwareTrait;

src/Common/ExecTrait.php

Spacing +1 added lines, -1 removed lines

@@ -247,7 +247,7 @@
         $this->process = $process;
 
         if (!$output_callback) {
-            $output_callback = function ($type, $buffer) {
+            $output_callback = function($type, $buffer) {
                 $progressWasVisible = $this->hideTaskProgress();
                 $this->writeMessage($buffer);
                 $this->showTaskProgress($progressWasVisible);

src/Task/ApiGen/ApiGen.php

Spacing +1 added lines, -1 removed lines

@@ -71,7 +71,7 @@
         if (!is_array($args)) {
             $args = func_get_args();
         }
-        $args = array_map(function ($arg) {
+        $args = array_map(function($arg) {
             if (preg_match('/^\w+$/', trim($arg)) === 1) {
                 $this->operation = $arg;
                 return null;

Doc Comments +5 added lines, -5 removed lines

@@ -165,7 +165,7 @@ 
     }
 
     /**
-     * @param array|string|\Traversable $src
+     * @param string $src
      *   One or more source values.
      *
      * @return $this
@@ -190,7 +190,7 @@ 
     }
 
     /**
-     * @param array|string $exts
+     * @param string $exts
      *   One or more extensions.
      *
      * @return $this
@@ -202,7 +202,7 @@ 
     }
 
     /**
-     * @param array|string $exclude
+     * @param string[] $exclude
      *   One or more exclusions.
      *
      * @return $this
@@ -216,7 +216,7 @@ 
     }
 
     /**
-     * @param array|string|\Traversable $path
+     * @param \SplDoublyLinkedList $path
      *   One or more skip-doc-path values.
      *
      * @return $this
@@ -244,7 +244,7 @@ 
     }
 
     /**
-     * @param array|string $charset
+     * @param string[] $charset
      *   One or more charsets.
      *
      * @return $this