captainhook-git / captainhook

src/Hook/Util.php

Spacing +1 added lines, -1 removed lines

@@ -103,7 +103,7 @@
             $info = explode(' ', $stdIn[0]);
             // If we find a rewritten commit, we return the first commit before the rewritten one.
             // If we do not find any rewritten commits (awkward) we use the last ref-log position.
-            return isset($info[1]) ? trim($info[1]) . '^' :  'HEAD@{1}';
+            return isset($info[1]) ? trim($info[1]) . '^' : 'HEAD@{1}';
         }
 
         return $io->getArgument(Hooks::ARG_PREVIOUS_HEAD, 'HEAD@{1}');

src/Hook/PHP/CoverageResolver/CloverXML.php

Spacing +1 added lines, -1 removed lines

@@ -77,6 +77,6 @@
             );
         }
 
-        return $covered / ((int)$statements * 0.01);
+        return $covered / ((int) $statements * 0.01);
     }
 }

src/Hook/Message/Rule/UseImperativeMood.php

Indentation +1 added lines, -1 removed lines

@@ -31,7 +31,7 @@
         parent::__construct();
 
         $this->hint = 'A commit message subject should always complete the following sentence.'  . PHP_EOL .
-                      'This commit will [YOUR COMMIT MESSAGE].';
+                        'This commit will [YOUR COMMIT MESSAGE].';
 
         $this->setSubjectBlacklist(
             [

Spacing +2 added lines, -2 removed lines

@@ -30,7 +30,7 @@ 
     {
         parent::__construct();
 
-        $this->hint = 'A commit message subject should always complete the following sentence.'  . PHP_EOL .
+        $this->hint = 'A commit message subject should always complete the following sentence.' . PHP_EOL .
                       'This commit will [YOUR COMMIT MESSAGE].';
 
         $this->setSubjectBlacklist(
@@ -49,7 +49,7 @@ 
 
         if ($checkOnlyBeginning) {
             // overwrite the detection logic to only check the beginning og the string
-            $this->stringDetection = function (string $content, string $term): bool {
+            $this->stringDetection = function(string $content, string $term): bool {
                 return strpos($content, $term) === 0;
             };
         }

src/Hook/Message/Action/PrepareFromFile.php

Spacing +1 added lines, -1 removed lines

@@ -62,7 +62,7 @@
 
         // if there is a commit message don't do anything just delete the file
         if ($repository->getCommitMsg()->isEmpty()) {
-            $msg = (string)file_get_contents($cacheFile);
+            $msg = (string) file_get_contents($cacheFile);
             $repository->setCommitMsg(
                 new CommitMessage($msg, $repository->getCommitMsg()->getCommentCharacter())
             );

src/Hook/Condition/FileStaged/ThatIs.php

Spacing +3 added lines, -3 removed lines

@@ -69,8 +69,8 @@ 
      */
     public function __construct(array $options)
     {
-        $this->directories = (array)($options['inDirectory'] ?? []);
-        $this->suffixes    = (array)($options['ofType'] ?? []);
+        $this->directories = (array) ($options['inDirectory'] ?? []);
+        $this->suffixes    = (array) ($options['ofType'] ?? []);
 
         $diffFilter = $options['diffFilter'] ?? [];
         $this->diffFilter  = FilterUtil::sanitize(is_array($diffFilter) ? $diffFilter : str_split($diffFilter));
@@ -112,7 +112,7 @@ 
         if (empty($this->directories)) {
             return $files;
         }
-        return array_filter($files, function ($file) {
+        return array_filter($files, function($file) {
             foreach ($this->directories as $directory) {
                 if (str_starts_with($file, $directory)) {
                     return true;

src/Hook/Condition/Logic.php

Spacing +1 added lines, -1 removed lines

@@ -48,7 +48,7 @@
     {
         $realConditions = [];
         foreach ($conditions as $condition) {
-            if (! $condition instanceof Condition) {
+            if (!$condition instanceof Condition) {
                 continue;
             }
             $realConditions[] = $condition;

src/Hook/File/Action/DoesNotContainRegex.php

Indentation +7 added lines, -7 removed lines

@@ -122,9 +122,9 @@ 
     protected function errorDetails(string $file): string
     {
         return ' <comment>('
-               . $this->fileMatches[$file] . ' match'
-               . ($this->fileMatches[$file] > 1 ? 'es' : '')
-               . ')</comment>';
+                . $this->fileMatches[$file] . ' match'
+                . ($this->fileMatches[$file] > 1 ? 'es' : '')
+                . ')</comment>';
     }
 
     /**
@@ -136,10 +136,10 @@ 
     protected function errorMessage(int $filesFailed): string
     {
         return 'Regex \'' . $this->regexName . '\' failed: '
-               . 'found ' . $this->totalMatches . ' match'
-               . ($this->totalMatches > 1 ? 'es' : '')
-               . ' in ' . $filesFailed . ' file'
-               . ($filesFailed > 1 ? 's' : '' );
+                . 'found ' . $this->totalMatches . ' match'
+                . ($this->totalMatches > 1 ? 'es' : '')
+                . ' in ' . $filesFailed . ' file'
+                . ($filesFailed > 1 ? 's' : '' );
     }
 
     /**

Spacing +1 added lines, -1 removed lines

@@ -139,7 +139,7 @@
                . 'found ' . $this->totalMatches . ' match'
                . ($this->totalMatches > 1 ? 'es' : '')
                . ' in ' . $filesFailed . ' file'
-               . ($filesFailed > 1 ? 's' : '' );
+               . ($filesFailed > 1 ? 's' : '');
     }
 
     /**

tests/unit/Runner/Hook/PreCommitTest.php

Indentation +6 added lines, -6 removed lines

@@ -78,8 +78,8 @@ 
 
         // every action has to get executed
         $actionConfigFail->expects($this->atLeastOnce())
-                         ->method('getAction')
-                         ->willReturn(CH_PATH_FILES . '/bin/failure');
+                            ->method('getAction')
+                            ->willReturn(CH_PATH_FILES . '/bin/failure');
 
         // so even if the first actions fails this action has to get executed
         $actionConfigSuccess->expects($this->atLeastOnce())
@@ -88,8 +88,8 @@ 
 
         $hookConfig->method('isEnabled')->willReturn(true);
         $hookConfig->expects($this->once())
-                   ->method('getActions')
-                   ->willReturn([$actionConfigFail, $actionConfigSuccess]);
+                    ->method('getActions')
+                    ->willReturn([$actionConfigFail, $actionConfigSuccess]);
 
         $config->expects($this->once())->method('getHookConfigToExecute')->willReturn($hookConfig);
         $config->method('isHookEnabled')->willReturn(true);
@@ -125,8 +125,8 @@ 
 
         $hookConfig->method('isEnabled')->willReturn(true);
         $hookConfig->expects($this->once())
-                   ->method('getActions')
-                   ->willReturn([$actionConfigFail, $actionConfigSuccess]);
+                    ->method('getActions')
+                    ->willReturn([$actionConfigFail, $actionConfigSuccess]);
 
         $config->expects($this->once())->method('getHookConfigToExecute')->willReturn($hookConfig);
         $config->expects($this->atLeastOnce())->method('isHookEnabled')->willReturn(true);

src/Git/ChangedFiles/Detector/Factory.php

Spacing +1 added lines, -1 removed lines

@@ -50,7 +50,7 @@
         $command = $io->getArgument(Hooks::ARG_COMMAND);
 
         /** @var \CaptainHook\App\Git\ChangedFiles\Detecting $class */
-        $class    = self::$detectors[$command] ?? '\\CaptainHook\\App\\Git\\ChangedFiles\\Detector\\Fallback';
+        $class = self::$detectors[$command] ?? '\\CaptainHook\\App\\Git\\ChangedFiles\\Detector\\Fallback';
         return new $class($io, $repository);
     }
 }