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

Spacing +3 added lines, -3 removed lines

@@ -66,7 +66,7 @@ 
         } elseif (is_array($command)) {
             $command = implode('; ', $command);
         } else {
-            throw new \Netresearch\Kite\Exception('Command must be string or array, ' . gettype($command) . ' given.', 1312454906);
+            throw new \Netresearch\Kite\Exception('Command must be string or array, '.gettype($command).' given.', 1312454906);
         }
 
         parent::__construct($command, $cwd, null, null, null);
@@ -141,14 +141,14 @@ 
     {
         $command = $this->getCommandLine();
 
-        $this->console->output('<cmd>' . $this->getWorkingDirectory() . ' > ' . $command . '</cmd>', OutputInterface::VERBOSITY_VERBOSE);
+        $this->console->output('<cmd>'.$this->getWorkingDirectory().' > '.$command.'</cmd>', OutputInterface::VERBOSITY_VERBOSE);
 
         if ($this->dryRun) {
             return null;
         }
 
         parent::run(
-            function ($type, $buffer) use ($callback) {
+            function($type, $buffer) use ($callback) {
                 if ($callback) {
                     call_user_func($callback, $type, $buffer);
                 }

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

Spacing +3 added lines, -3 removed lines

@@ -143,7 +143,7 @@ 
         $process = $this->console->createProcess($this->getCommand(), $this->get('cwd'));
         $process->setDryRun(!$this->shouldExecute());
         foreach ($this->get('processSettings') as $key => $value) {
-            $process->{'set' . ucfirst($key)}($value);
+            $process->{'set'.ucfirst($key)}($value);
         }
         return $process->run();
     }
@@ -169,7 +169,7 @@ 
                 if ($options || $arguments) {
                     throw new Exception('Can not combine argv with options or arguments');
                 }
-                $cmd .= ' ' . (is_array($argv) ? implode(' ', $argv) : $argv);
+                $cmd .= ' '.(is_array($argv) ? implode(' ', $argv) : $argv);
             } else {
                 $cmd .= $this->renderOptions($options);
                 $cmd .= $this->renderArguments($arguments);
@@ -224,7 +224,7 @@ 
         $argumentString = '';
         foreach ($arguments as $argument) {
             $value = $this->expand($argument);
-            $argumentString .= ' ' . escapeshellarg($value);
+            $argumentString .= ' '.escapeshellarg($value);
         }
         return $argumentString;
     }

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

Spacing +3 added lines, -3 removed lines

@@ -283,7 +283,7 @@ 
         if (!$name) {
             return 0;
         }
-        $key = $type . ':' . $name;
+        $key = $type.':'.$name;
         $tasksAdded = 0;
         if (array_key_exists($key, $this->job->deferredTasks)) {
             while ($task = array_shift($this->job->deferredTasks[$key])) {
@@ -310,7 +310,7 @@ 
                 $shouldRun = $if;
             } elseif (is_string($if)) {
                 $shouldRun = (boolean) $task->expand(
-                    '{' . $if . ' ? 1 : 0}'
+                    '{'.$if.' ? 1 : 0}'
                 );
             } else {
                 $shouldRun = call_user_func_array($if, array($task, $this->console));
@@ -349,7 +349,7 @@ 
                     if ($name === '@self') {
                         $name = $this->get('name');
                     }
-                    $key = $type . ':' . $name;
+                    $key = $type.':'.$name;
                     $this->job->deferredTasks[$key][] = $task;
                 }
             }

presets/common.php

Spacing +1 added lines, -1 removed lines

@@ -49,7 +49,7 @@
         ],
         'task' => [
             'type' => 'callback',
-            'callback' => function (\Netresearch\Kite\Job $job) {
+            'callback' => function(\Netresearch\Kite\Job $job) {
                 $git = $job->get('git');
                 $command = $job->get('cmd');
                 foreach ($job->get('composer.packages') as $package) {

src/ExpressionLanguage/Lexer.php

Spacing +2 added lines, -2 removed lines

@@ -172,10 +172,10 @@
                 }
                 if ($isTest) {
                     if ($isFunctionCall) {
-                        throw new SyntaxError('Can\'t use function return value in write context',  $stream->current->cursor);
+                        throw new SyntaxError('Can\'t use function return value in write context', $stream->current->cursor);
                     }
                     if (!$stream->current->test(Token::PUNCTUATION_TYPE, ')')) {
-                        throw new SyntaxError('Expected )',  $stream->current->cursor);
+                        throw new SyntaxError('Expected )', $stream->current->cursor);
                     }
                     $tokens[] = new Token(Token::STRING_TYPE, implode('.', $names), $token->cursor);
                 } else {

src/Task/IncludeTask.php

Spacing +1 added lines, -1 removed lines

@@ -51,7 +51,7 @@
     public function execute()
     {
         ob_start(
-            function ($output) {
+            function($output) {
                 $this->console->output($output, false);
                 return '';
             }, 2, 0

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

Spacing +5 added lines, -5 removed lines

@@ -57,7 +57,7 @@ 
     {
         if ($name === 'command') {
             parent::offsetSet('composerCommand', $value);
-            $value = 'composer ' .  $value;
+            $value = 'composer '.$value;
         }
         parent::offsetSet($name, $value);
     }
@@ -84,19 +84,19 @@ 
     protected function getCommand()
     {
         $cmd = $this->get('command');
-        $argv = substr(parent::getCommand(), strlen($cmd)) . ' ';
+        $argv = substr(parent::getCommand(), strlen($cmd)).' ';
         if (strpos($argv, ' --no-ansi ') === false
             && strpos($argv, ' --ansi ') === false
             && $this->console->getOutput()->isDecorated()
         ) {
-            $argv = ' --ansi' . $argv;
+            $argv = ' --ansi'.$argv;
         }
         if (!$this->shouldExecute()
             && strpos($argv, ' --dry-run ') === false
         ) {
-            $argv = ' --dry-run' . $argv;
+            $argv = ' --dry-run'.$argv;
         }
-        return $cmd . rtrim($argv);
+        return $cmd.rtrim($argv);
     }
 }
 ?>

src/Workflow/Deployment.php

Spacing +11 added lines, -11 removed lines

@@ -88,7 +88,7 @@ 
         if (!$rollback && !$restore) {
             $this->checkout();
             $this->release = date($this->get('releaseFormat', 'YmdHis'));
-            $this->set('releaseDir', 'releases/' . $this->release);
+            $this->set('releaseDir', 'releases/'.$this->release);
             $this->release();
             $this->shareResources();
         }
@@ -195,9 +195,9 @@ 
     protected function activate()
     {
         $sub = $this->iterate('{nodes}', 'node');
-        $sub->message('<step>Activating ' . ($this->release ? 'new' : 'latest') . ' release</step>');
+        $sub->message('<step>Activating '.($this->release ? 'new' : 'latest').' release</step>');
         $sub->callback(
-            function (Task\IterateTask $iterator) {
+            function(Task\IterateTask $iterator) {
                 $links = $iterator->remoteShell('echo "`readlink previous`;`readlink current`;`readlink next`"', '{node.deployPath}');
                 list($previous, $current, $next) = explode(';', $links);
 
@@ -213,7 +213,7 @@ 
 
                     $commands = array("ln -sfn $next current; rm next");
                     if ($current) {
-                        $from = '<comment>' . basename($current) . '</comment>';
+                        $from = '<comment>'.basename($current).'</comment>';
                         array_unshift($commands, "ln -s $current previous");
                         if ($previous) {
                             array_unshift($commands, "rm previous; rm -rf $previous");
@@ -252,7 +252,7 @@ 
         $sub = $this->iterate('{nodes}', 'node');
         $sub->message('<step>Restoring previous release</step>');
         $sub->callback(
-            function (Task\IterateTask $iterator) use (&$firstPreviousRelease) {
+            function(Task\IterateTask $iterator) use (&$firstPreviousRelease) {
                 $links = $iterator->remoteShell('echo "`readlink previous`;`readlink current`;`readlink next`"', '{node.deployPath}');
                 list($previous, $current, $next) = explode(';', $links);
                 if (!$previous) {
@@ -269,7 +269,7 @@ 
                     $commands = array("ln -sfn $previous current; rm previous");
                     if ($current) {
                         array_unshift($commands, "ln -s $current next");
-                        $from = '<comment>' . basename($current) . '</comment>';
+                        $from = '<comment>'.basename($current).'</comment>';
                     } else {
                         $from = '<warning>none</warning>';
                     }
@@ -292,7 +292,7 @@ 
         $sub = $this->iterate('{shared}', array('type' => 'entries'));
         $sub->message('<step>Linking shared resources</step>');
         $sub->callback(
-            function (Task\IterateTask $iterator) {
+            function(Task\IterateTask $iterator) {
                 $type = $iterator->get('type');
                 if (!in_array($type, array('dirs', 'files'), true)) {
                     $iterator->doExit('shared may only contain keys "dirs" or "files"', 1);
@@ -311,13 +311,13 @@ 
                     }
 
                     $commands[] = 'cd {releaseDir}';
-                    $commands[] = 'rm -rf ' . $entry;
+                    $commands[] = 'rm -rf '.$entry;
                     if ($dirName) {
-                        $commands[] = 'mkdir -p ' . $dirName;
-                        $commands[] = 'cd ' . $dirName;
+                        $commands[] = 'mkdir -p '.$dirName;
+                        $commands[] = 'cd '.$dirName;
                         $subDirCount += substr_count($dirName, '/') + 1;
                     }
-                    $commands[] = 'ln -s ' . str_repeat('../', $subDirCount) . $shareDir . '/' . $entry;
+                    $commands[] = 'ln -s '.str_repeat('../', $subDirCount).$shareDir.'/'.$entry;
 
                     $iterator->remoteShell($commands, '{node.deployPath}');
                 }