consolidation / robo

src/Task/Base/Exec.php

Unused Use Statements -1 removed lines

@@ -2,7 +2,6 @@
 
 namespace Robo\Task\Base;
 
-use Robo\Common\ExecTrait;
 use Robo\Contract\CommandInterface;
 use Robo\Contract\PrintedInterface;
 use Robo\Contract\SimulatedInterface;

src/Runner.php

Spacing +1 added lines, -1 removed lines

@@ -81,7 +81,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();
     }

src/Application.php

Spacing +1 added lines, -1 removed lines

@@ -40,7 +40,7 @@
     {
         $createRoboFile = new Command('init');
         $createRoboFile->setDescription("Intitalizes basic RoboFile in current dir");
-        $createRoboFile->setCode(function () use ($roboClass, $roboFile) {
+        $createRoboFile->setCode(function() use ($roboClass, $roboFile) {
             $output = Robo::output();
             $output->writeln("<comment>  ~~~ Welcome to Robo! ~~~~ </comment>");
             $output->writeln("<comment>  " . basename($roboFile) . " will be created in the current directory </comment>");

src/Task/Development/GenerateMarkdownDoc.php

Spacing +2 added lines, -2 removed lines

@@ -705,7 +705,7 @@ 
         return implode(
             "\n",
             array_map(
-                function ($line) use ($indent) {
+                function($line) use ($indent) {
                     return substr($line, $indent);
                 },
                 explode("\n", $doc)
@@ -727,7 +727,7 @@ 
         $params = implode(
             ', ',
             array_map(
-                function ($p) {
+                function($p) {
                     return $this->documentParam($p);
                 },
                 $reflectedMethod->getParameters()

src/ClassDiscovery/RelativeNamespaceDiscovery.php

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

src/Collection/CollectionBuilder.php

Doc Comments +1 added lines, -1 removed lines

@@ -309,7 +309,7 @@
 
     /**
      * @param string $functionName
-     * @param int|string $stateKey
+     * @param string $stateKey
      *
      * @return $this
      */

tests/functional/ForEachTest.php

Spacing +7 added lines, -7 removed lines

@@ -59,8 +59,8 @@ 
         // set iterable in the __constructor
         $result = $this
             ->taskForEach($items)
-            ->withBuilder(function (CollectionBuilder $builder, $key, $value) use (&$actual) {
-                $builder->addCode(function () use ($key, $value, &$actual) {
+            ->withBuilder(function(CollectionBuilder $builder, $key, $value) use (&$actual) {
+                $builder->addCode(function() use ($key, $value, &$actual) {
                     $actual[] = "$key = $value";
 
                     return 0;
@@ -82,7 +82,7 @@ 
         // set iterable with deferTaskConfiguration()
         $result = $this
             ->collectionBuilder()
-            ->addCode(function (Data $data) use ($items) {
+            ->addCode(function(Data $data) use ($items) {
                 $data['items'] = $items;
 
                 return 0;
@@ -91,8 +91,8 @@ 
                 $this
                     ->taskForEach()
                     ->deferTaskConfiguration('setIterable', 'items')
-                    ->withBuilder(function (CollectionBuilder $builder, $key, $value) use (&$actual) {
-                        $builder->addCode(function () use ($key, $value, &$actual) {
+                    ->withBuilder(function(CollectionBuilder $builder, $key, $value) use (&$actual) {
+                        $builder->addCode(function() use ($key, $value, &$actual) {
                             $actual[] = "$key = $value";
 
                             return 0;
@@ -111,8 +111,8 @@ 
         // call the __constructor() without argument
         $result = $this
             ->taskForEach()
-            ->withBuilder(function (CollectionBuilder $builder, $key, $value) use (&$actual) {
-                $builder->addCode(function () use ($key, $value, &$actual) {
+            ->withBuilder(function(CollectionBuilder $builder, $key, $value) use (&$actual) {
+                $builder->addCode(function() use ($key, $value, &$actual) {
                     $actual++;
 
                     return 0;

tests/functional/GenerateTaskTest.php

Indentation +2 added lines, -2 removed lines

@@ -20,8 +20,8 @@
         $result = $this->taskGenTask('Symfony\Component\Filesystem\Filesystem', 'FilesystemStack')->run();
         $this->assertTrue($result->wasSuccessful());
         $this->assertContains(
-          'protected function _chgrp($files, $group, $recursive = false)',
-          $result->getMessage());
+            'protected function _chgrp($files, $group, $recursive = false)',
+            $result->getMessage());
     }
 
 }

tests/functional/GenerateMarkdownDocTest.php

Spacing +7 added lines, -7 removed lines

@@ -33,13 +33,13 @@ 
 
         $collection = $this->collectionBuilder();
         $taskGenerator = $collection->taskGenDoc("TestedRoboTask.md");
-        $taskGenerator->filterClasses(function (\ReflectionClass $r) {
+        $taskGenerator->filterClasses(function(\ReflectionClass $r) {
             return !($r->isAbstract() || $r->isTrait()) && $r->implementsInterface('Robo\Contract\TaskInterface');
         })->prepend("# TestedRoboTask Tasks");
         $taskGenerator->docClass('TestedRoboTask');
 
         $taskGenerator->filterMethods(
-            function (\ReflectionMethod $m) {
+            function(\ReflectionMethod $m) {
                 if ($m->isConstructor() || $m->isDestructor() || $m->isStatic()) {
                     return false;
                 }
@@ -65,21 +65,21 @@ 
                 return !in_array($m->name, $undocumentedMethods) && $m->isPublic(); // methods are not documented
             }
         )->processClassSignature(
-            function ($c) {
+            function($c) {
                 return "## " . preg_replace('~Task$~', '', $c->getShortName()) . "\n";
             }
         )->processClassDocBlock(
-            function (\ReflectionClass $c, $doc) {
+            function(\ReflectionClass $c, $doc) {
                 $doc = preg_replace('~@method .*?(.*?)\)~', '* `$1)` ', $doc);
-                $doc = str_replace('\\'.$c->name, '', $doc);
+                $doc = str_replace('\\' . $c->name, '', $doc);
                 return $doc;
             }
         )->processMethodSignature(
-            function (\ReflectionMethod $m, $text) {
+            function(\ReflectionMethod $m, $text) {
                 return str_replace('#### *public* ', '* `', $text) . '`';
             }
         )->processMethodDocBlock(
-            function (\ReflectionMethod $m, $text) {
+            function(\ReflectionMethod $m, $text) {
 
                 return $text ? ' ' . trim(strtok($text, "\n"), "\n") : '';
             }