ktomk / pipelines

src/Runner.php

Spacing +1 added lines, -1 removed lines

@@ -111,7 +111,7 @@
                 $name
             ));
 
-            $copy = (bool) ($this->flags & self::FLAG_DEPLOY_COPY);
+            $copy = (bool)($this->flags & self::FLAG_DEPLOY_COPY);
 
             // docker client inside docker
             // FIXME give controlling options, this is serious /!\

src/Utility/App.php

Spacing +2 added lines, -2 removed lines

@@ -91,7 +91,7 @@ 
             // @codeCoverageIgnoreStart
             $buffer = file_get_contents($installedFile);
             $struct = json_decode($buffer);
-            foreach ((array) $struct as $package) {
+            foreach ((array)$struct as $package) {
                 if (!isset($package->name) || $package->name !== 'ktomk/pipelines') {
                     continue;
                 }
@@ -447,7 +447,7 @@ 
          * @param File $file
          * @return array|Step[]
          */
-        $iter = function (File $file) {
+        $iter = function(File $file) {
             $ids = $file->getPipelineIds();
             $return = array();
             foreach ($ids as $id) {

Braces +9 added lines, -3 removed lines

@@ -357,9 +357,15 @@
 
         $pipelines = File::createFromFile($path);
 
-        if ($images) return $this->showImages($pipelines);
-        if ($show) return $this->showPipelines($pipelines);
-        if ($list) return $this->showPipelineIds($pipelines);
+        if ($images) {
+            return $this->showImages($pipelines);
+        }
+        if ($show) {
+            return $this->showPipelines($pipelines);
+        }
+        if ($list) {
+            return $this->showPipelineIds($pipelines);
+        }
 
         ###
 

src/PharBuild/Timestamps.php

Spacing +1 added lines, -1 removed lines

@@ -127,7 +127,7 @@
         );
 
         if (!isset($algos[$signatureAlgo])) {
-            throw new \UnexpectedValueException('Invalid hash algorithm given: '.$signatureAlgo.' expected one of Phar::MD5, Phar::SHA1, Phar::SHA256 or Phar::SHA512'); // @codeCoverageIgnore
+            throw new \UnexpectedValueException('Invalid hash algorithm given: ' . $signatureAlgo . ' expected one of Phar::MD5, Phar::SHA1, Phar::SHA256 or Phar::SHA512'); // @codeCoverageIgnore
         }
         $algo = $algos[$signatureAlgo];
 

src/PharBuild/Builder.php

Spacing +5 added lines, -5 removed lines

@@ -156,7 +156,7 @@ 
      */
     public function snapShot()
     {
-        return function ($file) {
+        return function($file) {
             $source = fopen($file, 'r');
             if (false === $source) {
                 $this->err(sprintf('failed to open for reading: %s', $file));
@@ -191,7 +191,7 @@ 
      */
     public function dropFirstLine()
     {
-        return function ($file) {
+        return function($file) {
             $lines = file($file);
             array_shift($lines);
             $buffer = implode("", $lines);
@@ -209,7 +209,7 @@ 
      */
     public function replace($that, $with)
     {
-        return function ($file) use ($that, $with) {
+        return function($file) use ($that, $with) {
             $buffer = file_get_contents($file);
             $buffer = strtr($buffer, array($that => $with));
 
@@ -648,13 +648,13 @@ 
     private function _bfiles(Phar $phar, array $files)
     {
         $builders = array(
-            'fil' => function (array $nodes) use ($phar) {
+            'fil' => function(array $nodes) use ($phar) {
                 $result = $phar->buildFromIterator(
                     new \ArrayIterator($nodes)
                 );
                 return count($result);
             },
-            'str' => function (array $nodes) use ($phar) {
+            'str' => function(array $nodes) use ($phar) {
                 $count = 0;
                 foreach ($nodes as $localName => $contents) {
                     $phar->addFromString($localName, $contents);

Braces +3 added lines, -1 removed lines

@@ -320,12 +320,14 @@
             return array();
             // @codeCoverageIgnoreEnd
         }
-        if ($result === array()) $this->err(sprintf(
+        if ($result === array()) {
+            $this->err(sprintf(
             "ineffective pattern: %s",
             $pattern === $glob
                 ? $pattern
                 : sprintf("'%s' -> '%s'", $pattern, $glob)
         ));
+        }
         if (!is_array($result)) {
             // @codeCoverageIgnoreStart
             throw new \UnexpectedValueException(

src/Cli/Args.php

Spacing +3 added lines, -3 removed lines

@@ -63,7 +63,7 @@ 
 
     public function getFirstRemainingOption()
     {
-        foreach($this->arguments as $argument) {
+        foreach ($this->arguments as $argument) {
             if (strlen($argument) < 2) {
                 continue;
             }
@@ -122,13 +122,13 @@ 
             if ($argument !== $compare) {
                 continue;
             }
-            if (!isset($this->arguments[$index+1]) || $this->arguments[$index+1] === '--') {
+            if (!isset($this->arguments[$index + 1]) || $this->arguments[$index + 1] === '--') {
                 ArgsException::__(
                     sprintf("error: option '%s' requires an argument", $option)
                 );
             }
 
-            $result = $this->arguments[$index+1];
+            $result = $this->arguments[$index + 1];
             $consume = array($index, $index + 1);
         }
 

src/Lib.php

Spacing +2 added lines, -2 removed lines

@@ -94,7 +94,7 @@ 
 
         foreach ($arrays as $key => $value) {
             if (!is_array($value)) {
-                $arrays[$key] = (array) $value;
+                $arrays[$key] = (array)$value;
             }
         }
 
@@ -126,7 +126,7 @@ 
         }
 
         // inline escaped brace characters
-        $stack = array_map(function ($str) {
+        $stack = array_map(function($str) {
             return strtr($str, array(
                 '\\\\' => '\\\\',
                 '\\{' => '{', '\\}' => '}', '\\,' => ','

src/bootstrap.php

Spacing +1 added lines, -1 removed lines

@@ -2,7 +2,7 @@
 
 /* this file is part of pipelines */
 
-$inc = function ($file) {
+$inc = function($file) {
     return is_file($path = __DIR__ . '/../' . $file) ? include $path : false;
 };
 

src/File.php

Spacing +1 added lines, -1 removed lines

@@ -58,7 +58,7 @@
         if (array_key_exists('image', $array) && !is_string($array['image'])) {
             ParseException::__("'image' requires a Docker image name");
         }
-        if (isset($array['image']) && ! Lib::validDockerImage($array['image'])) {
+        if (isset($array['image']) && !Lib::validDockerImage($array['image'])) {
             ParseException::__(
                 sprintf("'image' invalid Docker image name: '%s'", $array['image'])
             );

Braces +3 added lines, -1 removed lines

@@ -272,7 +272,9 @@
         }
 
         foreach ($array as $section => $refs) {
-            if (!in_array($section, $sections, true)) continue;
+            if (!in_array($section, $sections, true)) {
+                continue;
+            }
             if (!is_array($refs)) {
                 ParseException::__("'$section' requires a list");
             }