consolidation / robo

src/Task/Base/loadShortcuts.php

Doc Comments +1 added lines, -1 removed lines

@@ -6,7 +6,7 @@
     /**
      * Executes shell command
      *
-     * @param string|\Robo\Contract\CommandInterface $command
+     * @param string $command
      *
      * @return \Robo\Result
      */

src/Task/Docker/Exec.php

Doc Comments +2 added lines, -2 removed lines

@@ -47,7 +47,7 @@ 
     protected $run = '';
 
     /**
-     * @param string|\Robo\Result $cidOrResult
+     * @param string $cidOrResult
      */
     public function __construct($cidOrResult)
     {
@@ -73,7 +73,7 @@ 
     }
 
     /**
-     * @param string|\Robo\Contract\CommandInterface $command
+     * @param string $command
      *
      * @return $this
      */

Spacing +1 added lines, -1 removed lines

@@ -88,6 +88,6 @@
      */
     public function getCommand()
     {
-        return $this->command . ' ' . $this->arguments . ' ' . $this->cid.' '.$this->run;
+        return $this->command . ' ' . $this->arguments . ' ' . $this->cid . ' ' . $this->run;
     }
 }

src/Task/Docker/Run.php

Doc Comments +1 added lines, -1 removed lines

@@ -124,7 +124,7 @@
     }
 
     /**
-     * @param string|\Robo\Contract\CommandInterface $run
+     * @param string $run
      *
      * @return $this
      */

src/Task/Filesystem/FlattenDir.php

Doc Comments +1 added lines, -1 removed lines

@@ -198,7 +198,7 @@
     }
 
     /**
-     * @param array $dirs
+     * @param string[] $dirs
      *
      * @return array|\Robo\Result
      *

Spacing +6 added lines, -6 removed lines

@@ -127,7 +127,7 @@ 
      */
     public function dirPermissions($permission)
     {
-        $this->chmod = (int) $permission;
+        $this->chmod = (int)$permission;
 
         return $this;
     }
@@ -176,7 +176,7 @@ 
     {
         if (!$this->fs->isAbsolutePath($dir)) {
             // attach the relative path to current working directory
-            $dir = getcwd().'/'.$dir;
+            $dir = getcwd() . '/' . $dir;
         }
         $this->parentDir = $dir;
 
@@ -230,7 +230,7 @@ 
             } catch (\InvalidArgumentException $e) {
                 // if finder cannot handle it, try with in()->name()
                 if (strpos($dir, '/') === false) {
-                    $dir = './'.$dir;
+                    $dir = './' . $dir;
                 }
                 $parts = explode('/', $dir);
                 $new_dir = implode('/', array_slice($parts, 0, -1));
@@ -260,7 +260,7 @@ 
      */
     protected function getTarget($file, $to)
     {
-        $target = $to.'/'.basename($file);
+        $target = $to . '/' . basename($file);
         if ($this->parents !== array(0, 0)) {
             // if the parent is set, create additional directories inside target
             // get relative path to parentDir
@@ -268,10 +268,10 @@ 
             // get top parents and bottom parents
             $parts = explode('/', rtrim($rel_path, '/'));
             $prefix_dir = '';
-            $prefix_dir .= ($this->parents[0] > 0 ? implode('/', array_slice($parts, 0, $this->parents[0])).'/' : '');
+            $prefix_dir .= ($this->parents[0] > 0 ? implode('/', array_slice($parts, 0, $this->parents[0])) . '/' : '');
             $prefix_dir .= ($this->parents[1] > 0 ? implode('/', array_slice($parts, (0 - $this->parents[1]), $this->parents[1])) : '');
             $prefix_dir = rtrim($prefix_dir, '/');
-            $target = $to.'/'.$prefix_dir.'/'.basename($file);
+            $target = $to . '/' . $prefix_dir . '/' . basename($file);
         }
 
         return $target;

src/Common/ResourceExistenceChecker.php

Indentation +14 added lines, -14 removed lines

@@ -90,25 +90,25 @@
         $this->checkResources($targets);
     }
 
-   /**
-    * Wrapper method around phps is_dir()
-    *
-    * @param string $directory
-    *
-    * @return bool
-    */
+    /**
+     * Wrapper method around phps is_dir()
+     *
+     * @param string $directory
+     *
+     * @return bool
+     */
     protected function isDir($directory)
     {
         return is_dir($directory);
     }
 
-   /**
-    * Wrapper method around phps file_exists()
-    *
-    * @param string $file
-    *
-    * @return bool
-    */
+    /**
+     * Wrapper method around phps file_exists()
+     *
+     * @param string $file
+     *
+     * @return bool
+     */
     protected function isFile($file)
     {
         return file_exists($file);

Doc Comments +1 added lines, -1 removed lines

@@ -48,7 +48,7 @@
      * @param string $type
      *   Allowed values: "file", "dir", "fileAndDir".
      *
-     * @return bool
+     * @return boolean|null
      */
     protected function checkResource($resource, $type)
     {

src/Task/Remote/Rsync.php

Doc Comments +1 added lines, -1 removed lines

@@ -353,7 +353,7 @@
     }
 
     /**
-     * @param array|string $pattern
+     * @param string[] $pattern
      *
      * @return $this
      */

tests/unit/Task/HgTest.php

Unused Use Statements -2 removed lines

@@ -1,7 +1,5 @@
 <?php
 
-use AspectMock\Test as test;
-
 class HgTest extends \Codeception\TestCase\Test
 {
     /**

src/Common/ExecCommand.php

Doc Comments +2 added lines, -2 removed lines

@@ -151,7 +151,7 @@ 
      *
      * @param string $cmd
      *
-     * @return bool|string
+     * @return string|false
      */
     protected function findExecutable($cmd)
     {
@@ -180,7 +180,7 @@ 
     }
 
     /**
-     * @return bool|string
+     * @return string|false
      */
     protected function findProjectBin()
     {

Spacing +1 added lines, -1 removed lines

@@ -89,7 +89,7 @@
     protected function findProjectBin()
     {
         $cwd = getcwd();
-        $candidates = [ __DIR__ . '/../../vendor/bin', __DIR__ . '/../../bin', $cwd . '/vendor/bin' ];
+        $candidates = [__DIR__ . '/../../vendor/bin', __DIR__ . '/../../bin', $cwd . '/vendor/bin'];
 
         // If this project is inside a vendor directory, give highest priority
         // to that directory.

tests/unit/Common/CommandArgumentsTest.php

Doc Comments +1 added lines, -1 removed lines

@@ -79,7 +79,7 @@
      *
      * @covers ::args
      *
-     * @param string $expected
+     * @param string $expectedLinux
      * @param array $args
      */
     public function testArgs($expectedLinux, $expectedWindows, $args)