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

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);
+        }
 
         ###
 

Spacing +3 added lines, -3 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;
                 }
@@ -378,7 +378,7 @@ 
         $streams = $this->streams;
         if ($args->hasOption('verbatim')) {
             $streams = new Streams();
-            $streams->copyHandle($this->streams,2);
+            $streams->copyHandle($this->streams, 2);
         }
 
         if ($option = $args->getFirstRemainingOption()) {
@@ -454,7 +454,7 @@ 
          * @param File $file
          * @return array|Step[]
          */
-        $iter = function (File $file) {
+        $iter = function(File $file) {
             $ids = $file->getPipelineIds();
             $return = array();
             foreach ($ids as $id) {

src/PharBuild/Builder.php

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(

Spacing +5 added lines, -5 removed lines

@@ -160,7 +160,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));
@@ -195,7 +195,7 @@ 
      */
     public function dropFirstLine()
     {
-        return function ($file) {
+        return function($file) {
             $lines = file($file);
             if (false === $lines) {
                 $this->err(sprintf('error reading file: %s', $file));
@@ -217,7 +217,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));
 
@@ -659,13 +659,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);

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(
                 '\\\\' => '\\\\',
                 '\\{' => '{', '\\}' => '}', '\\,' => ','

Indentation +6 added lines, -6 removed lines

@@ -119,12 +119,12 @@
             }
             // match
             foreach (array_unique(
-                     /** @scrutinizer ignore-type */
-                         preg_split(
-                             '~\\\\.(*SKIP)(*FAIL)|,~',
-                             $matches[2]
-                         )
-                     ) as $segment) {
+                        /** @scrutinizer ignore-type */
+                            preg_split(
+                                '~\\\\.(*SKIP)(*FAIL)|,~',
+                                $matches[2]
+                            )
+                        ) as $segment) {
                 $permutation = $matches[1] . $segment . $matches[3];
                 in_array($permutation, $stack, true) || $stack[] = $permutation;
             }

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");
             }

src/Cli/Vcs.php

Spacing +1 added lines, -1 removed lines

@@ -28,7 +28,7 @@
             return null;
         }
 
-        $path = substr($out, 0,-1);
+        $path = substr($out, 0, -1);
 
         if (!is_dir($path)) {
             return null;

src/Runner/Env.php

Spacing +1 added lines, -1 removed lines

@@ -118,7 +118,7 @@
      */
     public function setPipelinesId($id)
     {
-        $list = (string) $this->getValue('PIPELINES_IDS');
+        $list = (string)$this->getValue('PIPELINES_IDS');
         $hashes = preg_split('~\s+~', $list, -1, PREG_SPLIT_NO_EMPTY);
         $hashes = array_map('strtolower', $hashes);