Style fixes

Author: twodarek

ci/phan.php

@@ -1,12 +1,12 @@
 #!/usr/bin/env php
 <?php
+
 declare(strict_types=1);
 require realpath(dirname(__FILE__)).'/../vendor/autoload.php';
 use Expensify\Bedrock\CI\PhanAnalyzer;
 
 $analyzer = new PhanAnalyzer($_SERVER['GITHUB_REF']);
 if ($analyzer->analyze()) {
     exit(0);
-} else {
-    exit(1);
 }
+exit(1);

ci/src/PHPStyler.php

@@ -39,7 +39,7 @@ public function check()
         $PHPLintCommand = "find . -name '*.php' -not \\( -path './externalLib/*' -or -path './vendor/*' -or -path './build/*' \\) -print0 | xargs -0 -L 1 -n 1 -P 8 php -l 1>/dev/null";
 
         if ($this->branch === 'main') {
-            Travis::foldCall("lintmaster.php", $PHPLintCommand);
+            Travis::foldCall('lintmaster.php', $PHPLintCommand);
 
             echo 'Skipping style check for merge commits';
 
@@ -48,9 +48,9 @@ public function check()
 
         $this->checkoutBranch($this->branch);
 
-        Travis::foldCall("lint.php", $PHPLintCommand);
+        Travis::foldCall('lint.php', $PHPLintCommand);
 
-        Travis::fold("start", "style.php");
+        Travis::fold('start', 'style.php');
         Travis::timeStart();
         echo 'Enforce PHP style'.PHP_EOL;
         $output = $this->getModifiedFiles($this->branch);
@@ -83,13 +83,13 @@ public function check()
         }
 
         Travis::timeFinish();
-        Travis::fold("end", "style.php");
+        Travis::fold('end', 'style.php');
 
         if (!$lintOK) {
             return false;
         }
 
-        Travis::foldCall("git.checkout2", "git checkout {$this->commit} 2>&1");
+        Travis::foldCall('git.checkout2', "git checkout {$this->commit} 2>&1");
 
         return true;
     }

ci/src/PhanAnalyzer.php

@@ -35,12 +35,12 @@ public function analyze()
 
         $this->checkoutBranch($this->branch);
 
-        Travis::fold("start", "phan.analyze");
+        Travis::fold('start', 'phan.analyze');
         Travis::timeStart();
         echo 'Analyze PHP using Phan'.PHP_EOL;
         $changedFiles = $this->eexec("git diff main...{$this->branch} --name-status | egrep \"^[A|M].*\\.php$\" | cut -f 2");
-        echo "Analyzing files:".PHP_EOL;
-        $lintErrors = $this->eexec("./vendor/bin/phan -p -z --processes 5", false);
+        echo 'Analyzing files:'.PHP_EOL;
+        $lintErrors = $this->eexec('./vendor/bin/phan -p -z --processes 5', false);
         $lintOK = true;
         foreach ($lintErrors as $lintError) {
             foreach ($changedFiles as $file) {
@@ -52,7 +52,7 @@ public function analyze()
             }
         }
         Travis::timeFinish();
-        Travis::fold("end", "phan.analyze");
+        Travis::fold('end', 'phan.analyze');
 
         return $lintOK;
     }