netresearch / kite

src/Console/Output/ConsoleOutput.php

Doc Comments +1 added lines, -1 removed lines

@@ -28,7 +28,7 @@
     /**
      * ConsoleOutput constructor.
      *
-     * @param int|mixed                     $verbosity The verbosity
+     * @param integer                     $verbosity The verbosity
      * @param boolean                       $decorated If output should be decorated
      * @param OutputFormatterInterface|null $formatter Formatter
      */

src/Console/Output/Output.php

Doc Comments +2 added lines, -2 removed lines

@@ -52,7 +52,7 @@ 
     /**
      * Constructor.
      *
-     * @param mixed                         $stream    A stream resource
+     * @param resource                         $stream    A stream resource
      * @param int                           $verbosity The verbosity level (one of the VERBOSITY constants in OutputInterface)
      * @param bool|null                     $decorated Whether to decorate messages (null for auto-guessing)
      * @param OutputFormatterInterface|null $formatter Output formatter instance (null to use default OutputFormatter)
@@ -130,7 +130,7 @@ 
     /**
      * Write to output
      *
-     * @param array|string $messages Messages
+     * @param string|null $messages Messages
      * @param bool         $newline  Whether to append a newline
      * @param int          $type     The output type
      *

src/Service/Process.php

Doc Comments +1 added lines, -1 removed lines

@@ -135,7 +135,7 @@
      * @param callable|null $callback A PHP callback to run whenever there is some
      *                                output available on STDOUT or STDERR
      *
-     * @return mixed The output of the shell command or FALSE if the command returned a non-zero exit code and $ignoreErrors was enabled.
+     * @return null|string The output of the shell command or FALSE if the command returned a non-zero exit code and $ignoreErrors was enabled.
      */
     public function run($callback = NULL)
     {

src/Task/ShellTask.php

Doc Comments +1 added lines, -1 removed lines

@@ -136,7 +136,7 @@
     /**
      * Execute the command
      *
-     * @return mixed
+     * @return null|string
      */
     protected function executeCommand()
     {

src/Tasks.php

Doc Comments +7 added lines, -7 removed lines

@@ -426,7 +426,7 @@ 
     /**
      * Execute the next fetched task after given $task
      *
-     * @param Task|string $task Task or task name
+     * @param string $task Task or task name
      *
      * @return $this
      */
@@ -444,7 +444,7 @@ 
      * @param string $question The question
      * @param string $default  The default value
      *
-     * @return mixed|\Netresearch\Kite\Task
+     * @return string
      */
     public function answer($question, $default = null)
     {
@@ -506,11 +506,11 @@ 
      * Run a composer command
      *
      * @param string            $command         The command to execute
-     * @param array|string|null $optArg          Options and arguments
+     * @param string $optArg          Options and arguments
      *                                           {@see \Netresearch\Kite\Task\ShellTask}
      * @param array             $processSettings Settings for symfony process class
      *
-     * @return string|\Netresearch\Kite\Task\ComposerTask
+     * @return string
      */
     public function composer($command, $optArg = null, array $processSettings = array())
     {
@@ -566,7 +566,7 @@ 
      *                                           {@see \Netresearch\Kite\Task\ShellTask}
      * @param array             $processSettings Settings for symfony process class
      *
-     * @return Task\GitTask|string
+     * @return string
      */
     public function git($command, $cwd = null, $optArg = null, array $processSettings = array())
     {
@@ -576,7 +576,7 @@ 
     /**
      * Run a workflow for each of the $array's values.
      *
-     * @param array|\Traversable $array    The object to iterate over
+     * @param string $array    The object to iterate over
      * @param string|array       $as       Either string for
      *                                     foreach ($array as $as)
      *                                     or array for
@@ -633,7 +633,7 @@ 
      *                                           {@see \Netresearch\Kite\Task\ShellTask}
      * @param array             $processSettings Settings for symfony process class
      *
-     * @return Task\RemoteShellTask|string
+     * @return string
      */
     public function remoteShell($command, $cwd = null, $optArg = null, array $processSettings = array())
     {

src/Variables.php

Doc Comments +2 added lines, -2 removed lines

@@ -143,7 +143,7 @@ 
     /**
      * Get a variable from this very object (unexpanded)
      *
-     * @param mixed $offset Variable name
+     * @param string $offset Variable name
      *
      * @internal Required set/get/has/remove in order to access entries of Variables
      *           without looking up parents.
@@ -174,7 +174,7 @@ 
     /**
      * Set a variable on this very object
      *
-     * @param mixed $offset Variable name
+     * @param string $offset Variable name
      * @param mixed $value  The value
      *
      * @internal See {@see Variables::offsetGet()}

src/Workflow/Composer/Diagnose.php

Switch Indentation +61 added lines, -61 removed lines

@@ -189,25 +189,25 @@ 
             )
         );
         switch ($fix) {
-        case 1:
-            $this->git('add', $package->path, array('N' => true, 'A' => true));
-            $this->console->output('');
-            $this->git('diff', $package->path, 'HEAD', array('tty' => true));
-            $this->console->output('');
-            $this->fixUnstagedChanges($package);
-            break;
-        case 2:
-            $this->git('reset', $package->path, array('hard' => true));
-            $this->git('clean', $package->path, array('i' => true, 'd' => true), array('tty' => true));
-            break;
-        case 3:
-            $this->git('reset', $package->path);
-            $args = 'save -u';
-            if ($message = $this->answer('Message for stash:')) {
-                $args .= ' ' . escapeshellarg($message);
-            }
-            $this->git('stash', $package->path, $args);
-            break;
+            case 1:
+                $this->git('add', $package->path, array('N' => true, 'A' => true));
+                $this->console->output('');
+                $this->git('diff', $package->path, 'HEAD', array('tty' => true));
+                $this->console->output('');
+                $this->fixUnstagedChanges($package);
+                break;
+            case 2:
+                $this->git('reset', $package->path, array('hard' => true));
+                $this->git('clean', $package->path, array('i' => true, 'd' => true), array('tty' => true));
+                break;
+            case 3:
+                $this->git('reset', $package->path);
+                $args = 'save -u';
+                if ($message = $this->answer('Message for stash:')) {
+                    $args .= ' ' . escapeshellarg($message);
+                }
+                $this->git('stash', $package->path, $args);
+                break;
         }
     }
 
@@ -265,24 +265,24 @@ 
             $fixes[3] = 'Push with <comment>--force</comment>';
         }
         switch ($this->selectFixes($fixes)) {
-        case 1:
-            $this->gitRevDiff($package, '@\{u\}', 'Remote', 'Local');
-            $this->fixRemoteSynchronicity($package);
-            break;
-        case 3:
-            $commands = array('push');
-            $options = '--force';
-        case 2:
-            foreach ($commands as $command) {
-                $pck = "<comment>{$package->name}</comment>";
-                $this->console->output($msg = ucfirst($command) . "ing $pck...", false);
-                $this->git($command, $package->path, isset($options) ? $options : null);
-                $this->console->output(
-                    str_repeat(chr(8), strlen(strip_tags($msg)))
-                    . "<fg=green>Sucessfully {$command}ed $pck</>"
-                );
-            }
-            break;
+            case 1:
+                $this->gitRevDiff($package, '@\{u\}', 'Remote', 'Local');
+                $this->fixRemoteSynchronicity($package);
+                break;
+            case 3:
+                $commands = array('push');
+                $options = '--force';
+            case 2:
+                foreach ($commands as $command) {
+                    $pck = "<comment>{$package->name}</comment>";
+                    $this->console->output($msg = ucfirst($command) . "ing $pck...", false);
+                    $this->git($command, $package->path, isset($options) ? $options : null);
+                    $this->console->output(
+                        str_repeat(chr(8), strlen(strip_tags($msg)))
+                        . "<fg=green>Sucessfully {$command}ed $pck</>"
+                    );
+                }
+                break;
         }
     }
 
@@ -371,19 +371,19 @@ 
             }
         }
         switch ($this->selectFixes($actions)) {
-        case 1:
-            $this->gitRevDiff($package, $package->requiredBranch, $requiredConstraint, $currentConstraint);
-            $this->fixRequirementsMatch($package);
-            break;
-        case 2:
-            $this->checkoutPackage($package, $package->requiredBranch);
-            $this->reloadRequires($package);
-            break;
-        case 3:
-            foreach ($package->unsatisfiedDependentPackages as $dependentPackage) {
-                $this->rewriteRequirement($dependentPackage, $package->name, $currentConstraint);
-            }
-            break;
+            case 1:
+                $this->gitRevDiff($package, $package->requiredBranch, $requiredConstraint, $currentConstraint);
+                $this->fixRequirementsMatch($package);
+                break;
+            case 2:
+                $this->checkoutPackage($package, $package->requiredBranch);
+                $this->reloadRequires($package);
+                break;
+            case 3:
+                foreach ($package->unsatisfiedDependentPackages as $dependentPackage) {
+                    $this->rewriteRequirement($dependentPackage, $package->name, $currentConstraint);
+                }
+                break;
         }
     }
 
@@ -438,17 +438,17 @@ 
         );
 
         switch ($fix) {
-        case 1:
-            $this->gitRevDiff($package, $package->source->reference, 'Locked', 'Current');
-            $this->fixDivergeFromLock($package);
-            break;
-        case 2:
-            $this->composerUpdateRequired = true;
-            break;
-        case 3:
-            $this->git('checkout', $package->path, $package->source->reference);
-            $this->reloadRequires($package);
-            break;
+            case 1:
+                $this->gitRevDiff($package, $package->source->reference, 'Locked', 'Current');
+                $this->fixDivergeFromLock($package);
+                break;
+            case 2:
+                $this->composerUpdateRequired = true;
+                break;
+            case 3:
+                $this->git('checkout', $package->path, $package->source->reference);
+                $this->reloadRequires($package);
+                break;
         }
     }
 

src/Task/ComposerTask.php

Doc Comments +1 added lines, -1 removed lines

@@ -136,7 +136,7 @@
     /**
      * Execute the command
      *
-     * @return mixed
+     * @return null|string
      */
     protected function executeCommand()
     {