consolidation / robo

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/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

Doc Comments +1 added lines, -1 removed lines

@@ -27,7 +27,7 @@
      *
      * @param \Robo\Result $result
      *
-     * @return bool
+     * @return null|boolean
      */
     public function printResult(Result $result)
     {

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);

Doc Comments +1 added lines, -1 removed lines

@@ -380,7 +380,7 @@
     }
 
     /**
-     * @param $command
+     * @param string $command
      *
      * @return mixed
      */

src/Collection/CollectionBuilder.php

Indentation +9 added lines, -9 removed lines

@@ -140,15 +140,15 @@
         return $this;
     }
 
-  /**
-   * Add arbitrary code to execute as a task.
-   *
-   * @see \Robo\Collection\CollectionInterface::addCode
-   *
-   * @param callable $code
-   * @param int|string $name
-   * @return $this
-   */
+    /**
+     * Add arbitrary code to execute as a task.
+     *
+     * @see \Robo\Collection\CollectionInterface::addCode
+     *
+     * @param callable $code
+     * @param int|string $name
+     * @return $this
+     */
     public function addCode(callable $code, $name = \Robo\Collection\CollectionInterface::UNNAMEDTASK)
     {
         $this->getCollection()->addCode($code, $name);

Doc Comments +7 added lines

@@ -76,6 +76,10 @@ 
         $this->resetState();
     }
 
+    /**
+     * @param \League\Container\ContainerInterface $container
+     * @param BuilderAwareInterface $commandFile
+     */
     public static function create($container, $commandFile)
     {
         $builder = new self($commandFile);
@@ -284,6 +288,9 @@ 
         return $this->callCollectionStateFuntion(__FUNCTION__, func_get_args());
     }
 
+    /**
+     * @param string $functionName
+     */
     protected function callCollectionStateFuntion($functionName, $args)
     {
         $currentTask = ($this->currentTask instanceof WrappedTaskInterface) ? $this->currentTask->original() : $this->currentTask;

src/Common/ProcessExecutor.php

Doc Comments +4 added lines

@@ -24,6 +24,10 @@
         $this->process = $process;
     }
 
+    /**
+     * @param \League\Container\ContainerInterface $container
+     * @param Process $process
+     */
     public static function create($container, $process)
     {
         $processExecutor = new self($process);

tests/src/RoboFileFixture.php

Unused Use Statements -1 removed lines

@@ -4,7 +4,6 @@
 
 use Psr\Log\LoggerAwareTrait;
 use Psr\Log\LoggerAwareInterface;
-
 use Consolidation\AnnotatedCommand\Events\CustomEventAwareInterface;
 use Consolidation\AnnotatedCommand\Events\CustomEventAwareTrait;
 use Consolidation\OutputFormatters\StructuredData\PropertyList;