thecodingmachine / washingmachine

src/Clover/CloverFile.php

Spacing +1 added lines, -1 removed lines

@@ -1,5 +1,5 @@
 <?php
-declare(strict_types=1);
+declare(strict_types = 1);
 
 namespace TheCodingMachine\WashingMachine\Clover;
 

src/Clover/EmptyCloverFile.php

Spacing +1 added lines, -1 removed lines

@@ -1,5 +1,5 @@
 <?php
-declare(strict_types=1);
+declare(strict_types = 1);
 
 namespace TheCodingMachine\WashingMachine\Clover;
 

src/Clover/DiffService.php

Indentation +1 added lines, -1 removed lines

@@ -60,7 +60,7 @@
 
         // Now, let's order the differences by crap order.
         usort($differences, function(Difference $d1, Difference $d2) {
-           return $d2->getCrapScore() <=> $d1->getCrapScore();
+            return $d2->getCrapScore() <=> $d1->getCrapScore();
         });
 
         // Now, let's limit the number of returned differences

src/Clover/Crap4JFile.php

Spacing +1 added lines, -1 removed lines

@@ -1,5 +1,5 @@
 <?php
-declare(strict_types=1);
+declare(strict_types = 1);
 
 namespace TheCodingMachine\WashingMachine\Clover;
 

src/Gitlab/Message.php

Spacing +1 added lines, -1 removed lines

@@ -125,7 +125,7 @@
         $url = $this->getArtifactFileUrl($file->getFilename(), $gitlabUrl, $projectName, $buildId);
 
         $this->msg .= sprintf("\n<strong>[%s](%s)</strong>\n", $file->getFilename(), $url);
-        $this->msg .= sprintf("```\n%s%s```\n", $text, $isComplete?'':"... (file truncated)\n");
+        $this->msg .= sprintf("```\n%s%s```\n", $text, $isComplete ? '' : "... (file truncated)\n");
 
         if (!$isComplete) {
             $this->msg .= sprintf("[Download complete file](%s)\n", $url);

src/Gitlab/BuildService.php

Unused Use Statements -1 removed lines

@@ -1,7 +1,6 @@
 <?php
 namespace TheCodingMachine\WashingMachine\Gitlab;
 use Gitlab\Client;
-use GuzzleHttp\Psr7\Stream;
 use GuzzleHttp\Psr7\StreamWrapper;
 use Symfony\Component\Filesystem\Filesystem;
 

Doc Comments +2 added lines, -2 removed lines

@@ -71,7 +71,7 @@ 
      * @param string $projectName
      * @param string $commitId
      * @param int $numIter
-     * @return array
+     * @return string
      * @throws BuildNotFoundException
      */
     public function getLatestPipelineFromCommitId(string $projectName, string $commitId, int $numIter = 0) : array
@@ -108,7 +108,7 @@ 
     /**
      * @param string $projectName
      * @param string $branchName
-     * @return array
+     * @return string
      * @throws BuildNotFoundException
      */
     public function getLatestPipelineFromBranch(string $projectName, string $branchName) : array

Spacing +1 added lines, -1 removed lines

@@ -76,7 +76,7 @@
         return $this->pipelines[$projectName];
     }
 
-    public function findPipelineByCommit(string $projectName, string $commitId) : ?array
+    public function findPipelineByCommit(string $projectName, string $commitId) : ? array
     {
         $pipelines = $this->getPipelines($projectName);
 

src/Commands/RunCommand.php

Doc Comments +1 added lines, -1 removed lines

@@ -235,7 +235,7 @@
      * Returns the user id of the committer.
      *
      * @param Project $project
-     * @param $commitRef
+     * @param string $commitRef
      * @return int|null
      */
     private function getCommiterId(Project $project, $commitRef)

Spacing +3 added lines, -3 removed lines

@@ -259,7 +259,7 @@ 
 
         $commit = $project->commit($commitRef);
 
-        return $commit->committer ? $commit->committer->id :  null;
+        return $commit->committer ? $commit->committer->id : null;
     }
 
     /**
@@ -277,7 +277,7 @@ 
 
             $buildService->dumpArtifactFromBranch($projectName, $targetBranch, $jobStage, $tmpFile);
             $zipFile = new \ZipArchive();
-            if ($zipFile->open($tmpFile)!==true) {
+            if ($zipFile->open($tmpFile) !== true) {
                 throw new \RuntimeException('Invalid ZIP archive '.$tmpFile);
             }
             return $this->getMeasuresFromZipFile($zipFile, $cloverPath, $crap4JPath);
@@ -302,7 +302,7 @@ 
             $pipeline = $buildService->getLatestPipelineFromCommitId($projectName, $commitId);
             $buildService->dumpArtifact($projectName, $pipeline['id'], $jobStage, $tmpFile);
             $zipFile = new \ZipArchive();
-            if ($zipFile->open($tmpFile)!==true) {
+            if ($zipFile->open($tmpFile) !== true) {
                 throw new \RuntimeException('Invalid ZIP archive '.$tmpFile);
             }
             return $this->getMeasuresFromZipFile($zipFile, $cloverPath, $crap4JPath);

src/Commands/Config.php

Spacing +2 added lines, -2 removed lines

@@ -53,9 +53,9 @@
                 throw new \RuntimeException('Could not find the Gitlab URL in the "CI_REPOSITORY_URL" environment variable (usually set by Gitlab CI). Either set this environment variable or pass the URL via the --gitlab-url command line option.');
             }
             $parsed_url = parse_url($ciProjectUrl);
-            $scheme   = isset($parsed_url['scheme']) ? $parsed_url['scheme'] . '://' : '';
+            $scheme   = isset($parsed_url['scheme']) ? $parsed_url['scheme'].'://' : '';
             $host     = isset($parsed_url['host']) ? $parsed_url['host'] : '';
-            $port     = isset($parsed_url['port']) ? ':' . $parsed_url['port'] : '';
+            $port     = isset($parsed_url['port']) ? ':'.$parsed_url['port'] : '';
             $gitlabUrl = $scheme.$host.$port;
         }
         return rtrim($gitlabUrl, '/');

src/Git/GitRepository.php

Spacing +3 added lines, -3 removed lines

@@ -11,7 +11,7 @@ 
     public function getMergeBase(string $commit1, string $commit2) : string
     {
         try {
-            $this->extractFromCommand('git merge-base --is-ancestor ' . escapeshellarg($commit1) . ' ' . escapeshellarg($commit2));
+            $this->extractFromCommand('git merge-base --is-ancestor '.escapeshellarg($commit1).' '.escapeshellarg($commit2));
         } catch (GitException $e) {
             // The command will return exit code 1 if $commit1 is an ancestor of $commit2
             // Exit code one triggers an exception. We catch it.
@@ -19,14 +19,14 @@ 
         }
 
 
-        $results = $this->extractFromCommand('git merge-base ' . escapeshellarg($commit1). ' '. escapeshellarg($commit2));
+        $results = $this->extractFromCommand('git merge-base '.escapeshellarg($commit1).' '.escapeshellarg($commit2));
 
         return $results[0];
     }
 
     public function getLatestCommitForBranch(string $branch) : string
     {
-        $results = $this->extractFromCommand('git log -n 1 --pretty=format:"%H" ' . escapeshellarg($branch));
+        $results = $this->extractFromCommand('git log -n 1 --pretty=format:"%H" '.escapeshellarg($branch));
 
         return $results[0];
     }