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/WriteFileTest.php

Doc Comments +1 added lines, -4 removed lines

@@ -133,7 +133,4 @@
         $this->assertTrue($result->wasSuccessful());
         $this->assertFileExists('box/robo.txt');
         $contents = file_get_contents('box/robo.txt');
-        $this->assertContains('Hello robo.li!', $contents);
-    }
-}
-
+        $this->assertContains('Hello robo.li!', $contents
\ No newline at end of file

Unused Use Statements +1 added lines, -4 removed lines

@@ -133,7 +133,4 @@
         $this->assertTrue($result->wasSuccessful());
         $this->assertFileExists('box/robo.txt');
         $contents = file_get_contents('box/robo.txt');
-        $this->assertContains('Hello robo.li!', $contents);
-    }
-}
-
+        $this->assertContains('Hello robo.li!', $contents
\ No newline at end of file

Indentation +5 added lines, -8 removed lines

@@ -27,10 +27,10 @@ 
     {
         // write lines with WriteToFile task
         $result = $this->taskWriteToFile('blogpost.md')
-           ->line('****')
-           ->line('hello world')
-           ->line('****')
-           ->run();
+            ->line('****')
+            ->line('hello world')
+            ->line('****')
+            ->run();
         $this->assertTrue($result->wasSuccessful());
         $this->assertFileExists('blogpost.md');
         $contents = file_get_contents('blogpost.md');
@@ -133,7 +133,4 @@ 
         $this->assertTrue($result->wasSuccessful());
         $this->assertFileExists('box/robo.txt');
         $contents = file_get_contents('box/robo.txt');
-        $this->assertContains('Hello robo.li!', $contents);
-    }
-}
-
+        $this->assertContains('Hello robo.li!', $contents
\ No newline at end of file

Switch Indentation +1 added lines, -4 removed lines

@@ -133,7 +133,4 @@
         $this->assertTrue($result->wasSuccessful());
         $this->assertFileExists('box/robo.txt');
         $contents = file_get_contents('box/robo.txt');
-        $this->assertContains('Hello robo.li!', $contents);
-    }
-}
-
+        $this->assertContains('Hello robo.li!', $contents
\ No newline at end of file

Spacing +1 added lines, -4 removed lines

@@ -133,7 +133,4 @@
         $this->assertTrue($result->wasSuccessful());
         $this->assertFileExists('box/robo.txt');
         $contents = file_get_contents('box/robo.txt');
-        $this->assertContains('Hello robo.li!', $contents);
-    }
-}
-
+        $this->assertContains('Hello robo.li!', $contents
\ No newline at end of file

Braces +1 added lines, -4 removed lines

@@ -133,7 +133,4 @@
         $this->assertTrue($result->wasSuccessful());
         $this->assertFileExists('box/robo.txt');
         $contents = file_get_contents('box/robo.txt');
-        $this->assertContains('Hello robo.li!', $contents);
-    }
-}
-
+        $this->assertContains('Hello robo.li!', $contents
\ No newline at end of file

Upper-Lower-Casing +1 added lines, -4 removed lines

@@ -133,7 +133,4 @@
         $this->assertTrue($result->wasSuccessful());
         $this->assertFileExists('box/robo.txt');
         $contents = file_get_contents('box/robo.txt');
-        $this->assertContains('Hello robo.li!', $contents);
-    }
-}
-
+        $this->assertContains('Hello robo.li!', $contents
\ No newline at end of file