consolidation / robo

src/ClassDiscovery/RelativeNamespaceDiscovery.php

Indentation +2 added lines, -2 removed lines

@@ -91,8 +91,8 @@
     {
         $finder = new Finder();
         $finder->files()
-          ->name($pattern)
-          ->in($directories);
+            ->name($pattern)
+            ->in($directories);
 
         return $finder;
     }

Spacing +1 added lines, -1 removed lines

@@ -53,7 +53,7 @@
         $relativePath = $this->convertNamespaceToPath($this->relativeNamespace);
 
         foreach ($this->classLoader->getPrefixesPsr4() as $baseNamespace => $directories) {
-            $directories = array_filter(array_map(function ($directory) use ($relativePath) {
+            $directories = array_filter(array_map(function($directory) use ($relativePath) {
                 return $directory . $relativePath;
             }, $directories), 'is_dir');
 

tests/unit/ClassDiscovery/RelativeNamespaceDiscoveryTest.php

Indentation +7 added lines, -7 removed lines

@@ -38,10 +38,10 @@ 
     public function testConvertPathToNamespaceData()
     {
         return [
-          ['/A/B/C', 'A\B\C'],
-          ['A/B/C', 'A\B\C'],
-          ['A/B/C', 'A\B\C'],
-          ['A/B/C.php', 'A\B\C'],
+            ['/A/B/C', 'A\B\C'],
+            ['A/B/C', 'A\B\C'],
+            ['A/B/C', 'A\B\C'],
+            ['A/B/C.php', 'A\B\C'],
         ];
     }
 
@@ -61,9 +61,9 @@ 
     public function testConvertNamespaceToPathData()
     {
         return [
-          ['A\B\C', '/A/B/C'],
-          ['\A\B\C\\', '/A/B/C'],
-          ['A\B\C\\', '/A/B/C'],
+            ['A\B\C', '/A/B/C'],
+            ['\A\B\C\\', '/A/B/C'],
+            ['A\B\C\\', '/A/B/C'],
         ];
     }
 

Spacing +2 added lines, -2 removed lines

@@ -11,7 +11,7 @@ 
     public function testGetClasses()
     {
         $classLoader = new ClassLoader();
-        $classLoader->addPsr4('\\Robo\\PluginTest\\', [realpath(__DIR__.'/../../plugins')]);
+        $classLoader->addPsr4('\\Robo\\PluginTest\\', [realpath(__DIR__ . '/../../plugins')]);
         $service = new RelativeNamespaceDiscovery($classLoader);
         $service->setRelativeNamespace('Robo\Plugin');
         $service->setSearchPattern('/.*Commands?\.php$/');
@@ -26,7 +26,7 @@ 
     public function testGetFile()
     {
         $classLoader = new ClassLoader();
-        $classLoader->addPsr4('\\Robo\\PluginTest\\', [realpath(__DIR__.'/../../plugins')]);
+        $classLoader->addPsr4('\\Robo\\PluginTest\\', [realpath(__DIR__ . '/../../plugins')]);
         $service = new RelativeNamespaceDiscovery($classLoader);
         $service->setRelativeNamespace('Robo\Plugin');
 

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

src/Config/Config.php

Unused Use Statements -1 removed lines

@@ -2,7 +2,6 @@
 namespace Robo\Config;
 
 use Consolidation\Config\Util\ConfigOverlay;
-use Consolidation\Config\ConfigInterface;
 
 class Config extends ConfigOverlay implements GlobalOptionDefaultValuesInterface
 {

tests/unit/Task/WatchTest.php

Spacing +2 added lines, -2 removed lines

@@ -38,7 +38,7 @@ 
 
         $task->monitor(
             'src',
-            function () {
+            function() {
                 //do nothing
             },
             1 // CREATE
@@ -53,7 +53,7 @@ 
 
         $task->monitor(
             'src',
-            function () {
+            function() {
                 //do nothing
             },
             [

tests/unit/Task/SemVerTest.php

Spacing +1 added lines, -1 removed lines

@@ -51,7 +51,7 @@
     public function testSemverParseFileWithWindowsLineEndings()
     {
         $fixturePath = tempnam(sys_get_temp_dir(), 'semver');
-        $semverFile = str_replace("\n", "\r\n", file_get_contents(codecept_data_dir().'.semver'));
+        $semverFile = str_replace("\n", "\r\n", file_get_contents(codecept_data_dir() . '.semver'));
         file_put_contents($fixturePath, $semverFile);
 
         $res = (new \Robo\Task\Development\SemVer($fixturePath))

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

Doc Comments -1 removed lines

@@ -11,7 +11,6 @@
 
     /**
      * @command cat
-     * @param string $file
      * @default $file -
      */
     public function cat(InputInterface $input)

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