consolidation / robo

tests/cli/CollectionCest.php

Spacing +12 added lines, -12 removed lines

@@ -10,7 +10,7 @@ 
 {
     public function _before(CliGuy $I)
     {
-        $I->amInPath(codecept_data_dir().'sandbox');
+        $I->amInPath(codecept_data_dir() . 'sandbox');
     }
 
     public function toRunMultipleTasksViaACollectionBuilder(CliGuy $I)
@@ -122,7 +122,7 @@ 
             ->rollback(
                 $I->taskDeleteDir('j')
             )
-            ->rollbackCode(function () {
+            ->rollbackCode(function() {
                 throw new AbortTasksException('Aborting rollback.');
             })
             ->taskFilesystemStack()
@@ -181,7 +181,7 @@ 
                 ->mkdir('stuff')
             ->taskForEach($processList)
                 ->withBuilder(
-                    function ($builder, $key, $value) {
+                    function($builder, $key, $value) {
                         return $builder
                             ->taskFilesystemStack()
                                 ->touch("stuff/{$value}.txt");
@@ -238,32 +238,32 @@ 
 
     public function toRollbackInCorrectOrder(CliGuy $I)
     {
-        $expected_order = [6,5,4,3,2,1];
+        $expected_order = [6, 5, 4, 3, 2, 1];
         $actual_order = [];
         $collection = $I->collectionBuilder();
-        $collection->rollbackCode(function () use (&$actual_order) {
+        $collection->rollbackCode(function() use (&$actual_order) {
             $actual_order[] = 1;
         });
-        $collection->rollbackCode(function () use (&$actual_order) {
+        $collection->rollbackCode(function() use (&$actual_order) {
             $actual_order[] = 2;
         });
-        $collection->rollbackCode(function () use (&$actual_order) {
+        $collection->rollbackCode(function() use (&$actual_order) {
             $actual_order[] = 3;
         });
         // Add a nested collection with rollbacks.
         $nested_collection = $I->collectionBuilder();
-        $nested_collection->rollbackCode(function () use (&$actual_order) {
+        $nested_collection->rollbackCode(function() use (&$actual_order) {
             $actual_order[] = 4;
         });
-        $nested_collection->rollbackCode(function () use (&$actual_order) {
+        $nested_collection->rollbackCode(function() use (&$actual_order) {
             $actual_order[] = 5;
         });
         $collection->addTask($nested_collection);
 
-        $collection->rollbackCode(function () use (&$actual_order) {
+        $collection->rollbackCode(function() use (&$actual_order) {
             $actual_order[] = 6;
         });
-        $collection->addCode(function () {
+        $collection->addCode(function() {
             return Result::EXITCODE_ERROR;
         });
         $result = $collection->run();
@@ -463,7 +463,7 @@ 
         $collection = $I->getContainer()->get('collection');
 
         $collection->addCode(
-            function () {
+            function() {
                 throw new \RuntimeException('Error');
             }
         );

src/GlobalOptionsEventListener.php

Doc Comments +1 added lines, -1 removed lines

@@ -130,7 +130,7 @@
      *
      * @param string $value
      *
-     * @return array
+     * @return boolean[]
      */
     protected function splitConfigKeyValue($value)
     {

src/Runner.php

Doc Comments +3 added lines, -3 removed lines

@@ -136,7 +136,7 @@ 
     }
 
     /**
-     * @param array $argv
+     * @param string[] $argv
      * @param null|string $appName
      * @param null|string $appVersion
      * @param null|\Symfony\Component\Console\Output\OutputInterface $output
@@ -175,7 +175,7 @@ 
      * @param null|\Symfony\Component\Console\Input\InputInterface $input
      * @param null|\Symfony\Component\Console\Output\OutputInterface $output
      * @param null|\Robo\Application $app
-     * @param array[] $commandFiles
+     * @param null|string $commandFiles
      * @param null|ClassLoader $classLoader
      *
      * @return int
@@ -535,7 +535,7 @@ 
     }
 
     /**
-     * @param $selfUpdateRepository
+     * @param null|string $selfUpdateRepository
      *
      * @return $this
      */

Spacing +5 added lines, -5 removed lines

@@ -80,7 +80,7 @@ 
     public function __construct($roboClass = null, $roboFile = null)
     {
         // set the const as class properties to allow overwriting in child classes
-        $this->roboClass = $roboClass ? $roboClass : self::ROBOCLASS ;
+        $this->roboClass = $roboClass ? $roboClass : self::ROBOCLASS;
         $this->roboFile  = $roboFile ? $roboFile : self::ROBOFILE;
         $this->dir = getcwd();
     }
@@ -279,7 +279,7 @@ 
     {
         /** @var \Robo\ClassDiscovery\RelativeNamespaceDiscovery $discovery */
         $discovery = Robo::service('relativeNamespaceDiscovery');
-        $discovery->setRelativeNamespace($relativeNamespace.'\Commands')
+        $discovery->setRelativeNamespace($relativeNamespace . '\Commands')
             ->setSearchPattern('/.*Commands?\.php$/');
         return $discovery->getClasses();
     }
@@ -460,9 +460,9 @@ 
 
         if (substr($argv[$pos], 0, 12) == '--load-from=') {
             $this->dir = substr($argv[$pos], 12);
-        } elseif (isset($argv[$pos +1])) {
-            $this->dir = $argv[$pos +1];
-            unset($argv[$pos +1]);
+        } elseif (isset($argv[$pos + 1])) {
+            $this->dir = $argv[$pos + 1];
+            unset($argv[$pos + 1]);
         }
         unset($argv[$pos]);
         // Make adjustments if '--load-from' points at a file.

src/Task/Base/loadTasks.php

Doc Comments +1 added lines, -1 removed lines

@@ -4,7 +4,7 @@
 trait loadTasks
 {
     /**
-     * @param string|\Robo\Contract\CommandInterface $command
+     * @param string $command
      *
      * @return \Robo\Task\Base\Exec|\Robo\Collection\CollectionBuilder
      */

src/Task/Development/loadTasks.php

Doc Comments +1 added lines, -1 removed lines

@@ -75,7 +75,7 @@
     }
 
     /**
-     * @param string|array $url
+     * @param string[] $url
      *
      * @return \Robo\Task\Development\OpenBrowser|\Robo\Collection\CollectionBuilder
      */

src/Task/Filesystem/loadTasks.php

Doc Comments +1 added lines, -1 removed lines

@@ -14,7 +14,7 @@
     }
 
     /**
-     * @param string|string[] $dirs
+     * @param string $dirs
      *
      * @return \Robo\Task\Filesystem\DeleteDir|\Robo\Collection\CollectionBuilder
      */

src/Task/Remote/Ssh.php

Doc Comments +2 added lines, -2 removed lines

@@ -186,7 +186,7 @@ 
     }
 
     /**
-     * @param string|string[]|CommandInterface $command
+     * @param string $command
      *
      * @return $this
      */
@@ -255,7 +255,7 @@ 
     /**
      * Returns an ssh command string running $command on the remote.
      *
-     * @param string|CommandInterface $command
+     * @param string $command
      *
      * @return string
      */

src/Task/Assets/loadTasks.php

Indentation +5 added lines, -5 removed lines

@@ -23,11 +23,11 @@
         return $this->task(ImageMinify::class, $input);
     }
 
-   /**
-    * @param array $input
-    *
-    * @return \Robo\Task\Assets\Less|\Robo\Collection\CollectionBuilder
-    */
+    /**
+     * @param array $input
+     *
+     * @return \Robo\Task\Assets\Less|\Robo\Collection\CollectionBuilder
+     */
     protected function taskLess($input)
     {
         return $this->task(Less::class, $input);

src/Collection/CollectionBuilder.php

Indentation +10 added lines, -10 removed lines

@@ -168,16 +168,16 @@
         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);