consolidation / robo

src/Runner.php

Doc Comments +7 added lines, -4 removed lines

@@ -68,6 +68,9 @@ 
         $this->dir = getcwd();
     }
 
+    /**
+     * @param string $errorType
+     */
     protected function errorCondition($msg, $errorType)
     {
         $this->errorConditions[$msg] = $errorType;
@@ -115,7 +118,7 @@ 
     }
 
     /**
-     * @param array $argv
+     * @param string[] $argv
      * @param null|string $appName
      * @param null|string $appVersion
      * @param null|\Symfony\Component\Console\Output\OutputInterface $output
@@ -138,7 +141,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
@@ -231,9 +234,9 @@ 
     }
 
     /**
-     * @param $relativeNamespace
+     * @param string $relativeNamespace
      *
-     * @return array|string[]
+     * @return string[]
      */
     protected function discoverCommandClasses($relativeNamespace)
     {

Spacing +5 added lines, -5 removed lines

@@ -63,7 +63,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();
     }
@@ -239,7 +239,7 @@ 
     {
         /** @var \Robo\ClassDiscovery\RelativeNamespaceDiscovery $discovery */
         $discovery = Robo::service('relativeNamespaceDiscovery');
-        $discovery->setRelativeNamespace($relativeNamespace.'\Commands')
+        $discovery->setRelativeNamespace($relativeNamespace . '\Commands')
             ->setSearchPattern('*Commands.php');
         return $discovery->getClasses();
     }
@@ -413,9 +413,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.

examples/src/Robo/Plugin/Commands/ExampleCommands.php

Doc Comments +3 added lines

@@ -348,6 +348,9 @@
 {
     protected $message;
 
+    /**
+     * @param string $message
+     */
     public function __construct($message)
     {
         $this->message = $message;

Unused Use Statements -1 removed lines

@@ -2,7 +2,6 @@
 namespace RoboExample\Robo\Plugin\Commands;
 
 use Robo\Result;
-
 use Consolidation\AnnotatedCommand\CommandData;
 use Consolidation\OutputFormatters\Options\FormatterOptions;
 use Consolidation\OutputFormatters\StructuredData\RowsOfFields;

Spacing +11 added lines, -11 removed lines

@@ -29,7 +29,7 @@ 
      */
     public function tryWatch()
     {
-        $this->taskWatch()->monitor(['composer.json', 'composer.lock'], function () {
+        $this->taskWatch()->monitor(['composer.json', 'composer.lock'], function() {
             $this->taskComposerUpdate()->run();
         })->run();
     }
@@ -41,7 +41,7 @@ 
     {
         $this->say('The <b>expression</b> <bogus>is</bogus> <info>a < b</> it even works');
         $answer = $this->ask('how are you?');
-        $this->say('You are '.$answer);
+        $this->say('You are ' . $answer);
         $yes = $this->confirm('Do you want one more question?');
         if (!$yes) {
             return Result::cancelled();
@@ -49,7 +49,7 @@ 
         $lang = $this->askDefault('what is your favorite scripting language?', 'PHP');
         $this->say($lang);
         $pin = $this->askHidden('Ok, now tell your PIN code (it is hidden)');
-        $this->yell('Ha-ha, your pin code is: '.$pin);
+        $this->yell('Ha-ha, your pin code is: ' . $pin);
         $this->say('Bye!');
     }
 
@@ -173,7 +173,7 @@ 
         // using yaml or json output formats.
         $data->addRendererFunction(
             // n.b. There is a fourth parameter $rowData that may be added here.
-            function ($key, $cellData, FormatterOptions $options) {
+            function($key, $cellData, FormatterOptions $options) {
                 if ($key == 'name') {
                     return "<info>$cellData</>";
                 }
@@ -203,10 +203,10 @@ 
     public function tryFormatters($somthing = 'default', $options = ['format' => 'table', 'fields' => ''])
     {
         $outputData = [
-            'en' => [ 'first' => 'One',  'second' => 'Two',  'third' => 'Three' ],
-            'de' => [ 'first' => 'Eins', 'second' => 'Zwei', 'third' => 'Drei'  ],
-            'jp' => [ 'first' => 'Ichi', 'second' => 'Ni',   'third' => 'San'   ],
-            'es' => [ 'first' => 'Uno',  'second' => 'Dos',  'third' => 'Tres'  ],
+            'en' => ['first' => 'One', 'second' => 'Two', 'third' => 'Three'],
+            'de' => ['first' => 'Eins', 'second' => 'Zwei', 'third' => 'Drei'],
+            'jp' => ['first' => 'Ichi', 'second' => 'Ni', 'third' => 'San'],
+            'es' => ['first' => 'Uno', 'second' => 'Dos', 'third' => 'Tres'],
         ];
         return new RowsOfFields($outputData);
     }
@@ -241,7 +241,7 @@ 
     public function alterFormatters($result, CommandData $commandData)
     {
         if ($commandData->input()->getOption('french')) {
-            $result['fr'] = [ 'first' => 'Un',  'second' => 'Deux',  'third' => 'Trois'  ];
+            $result['fr'] = ['first' => 'Un', 'second' => 'Deux', 'third' => 'Trois'];
         }
 
         return $result;
@@ -396,7 +396,7 @@ 
             ->taskForEach($processList)
                 ->iterationMessage('Processing {value}')
                 ->call(
-                    function ($value) use($delay) {
+                    function($value) use($delay) {
                         // TaskForEach::call should only be used to do
                         // non-Robo operations. To use Robo tasks in an
                         // iterator, @see TaskForEach::withBuilder.
@@ -418,7 +418,7 @@ 
             ->taskCleanDir($workdir)
             ->taskForEach($processList)
                 ->withBuilder(
-                    function ($builder, $key, $value) use ($workdir) {
+                    function($builder, $key, $value) use ($workdir) {
                         return $builder
                             ->taskFilesystemStack()
                                 ->mkdir("$workdir/$value");