Roave / BackwardCompatibilityCheck

src/Comparator/BackwardsCompatibility/ClassBased/ConstantVisibilityReduced.php

Spacing +5 added lines, -5 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-declare(strict_types=1);
+declare(strict_types = 1);
 
 namespace Roave\ApiCompare\Comparator\BackwardsCompatibility\ClassBased;
 
@@ -30,19 +30,19 @@ 
             array_combine(
                 $sharedKeys,
                 array_map(
-                    function (string $propertyName) use ($visibilitiesFrom, $visibilitiesTo) : array {
+                    function(string $propertyName) use ($visibilitiesFrom, $visibilitiesTo) : array {
                         return [$visibilitiesFrom[$propertyName], $visibilitiesTo[$propertyName]];
                     },
                     $sharedKeys
                 )
             ),
-            function (array $visibilities) : bool {
+            function(array $visibilities) : bool {
                 // Note: works because public, protected and private are (luckily) sortable
                 return $visibilities[0] > $visibilities[1];
             }
         );
 
-        return Changes::fromArray(array_values(array_map(function (
+        return Changes::fromArray(array_values(array_map(function(
             string $propertyName,
             array $visibilities
         ) use (
@@ -64,7 +64,7 @@ 
     /** @return string[] indexed by constant name */
     private function constantVisibilities(ReflectionClass $class) : array
     {
-        return array_map(function (ReflectionClassConstant $constant) : string {
+        return array_map(function(ReflectionClassConstant $constant) : string {
             if ($constant->isPublic()) {
                 return self::VISIBILITY_PUBLIC;
             }

src/Comparator/BackwardsCompatibility/ClassBased/ConstantValueChanged.php

Spacing +5 added lines, -5 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-declare(strict_types=1);
+declare(strict_types = 1);
 
 namespace Roave\ApiCompare\Comparator\BackwardsCompatibility\ClassBased;
 
@@ -19,7 +19,7 @@ 
         $fromValues = $this->accessibleConstantValues($fromClass);
         $toValues   = $this->accessibleConstantValues($toClass);
 
-        $changedConstants = array_keys(array_filter($fromValues, function (
+        $changedConstants = array_keys(array_filter($fromValues, function(
             $constantValue,
             string $constantName
         ) use (
@@ -28,7 +28,7 @@ 
             return array_key_exists($constantName, $toValues) && $constantValue !== $toValues[$constantName];
         }, \ARRAY_FILTER_USE_BOTH));
 
-        return Changes::fromArray(array_values(array_map(function (string $constantName) use ($fromClass) : Change {
+        return Changes::fromArray(array_values(array_map(function(string $constantName) use ($fromClass) : Change {
             return Change::changed(
                 sprintf('Value of constant %s::%s changed', $fromClass->getName(), $constantName),
                 true
@@ -41,12 +41,12 @@ 
     {
         $accessibleConstants = array_filter(
             $class->getReflectionConstants(),
-            function (ReflectionClassConstant $constant) : bool {
+            function(ReflectionClassConstant $constant) : bool {
                 return $constant->isPublic() || $constant->isProtected();
             }
         );
 
-        return array_map(function (ReflectionClassConstant $constant) {
+        return array_map(function(ReflectionClassConstant $constant) {
             return $constant->getValue();
         }, $accessibleConstants);
     }

test/unit/Formatter/MarkdownPipedToSymfonyConsoleFormatterTest.php

Spacing +2 added lines, -2 removed lines

@@ -1,5 +1,5 @@ 
 <?php
-declare(strict_types=1);
+declare(strict_types = 1);
 
 namespace RoaveTest\ApiCompare\Formatter;
 
@@ -35,7 +35,7 @@ 
 
         $output->expects(self::any())
             ->method('writeln')
-            ->willReturnCallback(function (string $output) use ($changeToExpect) {
+            ->willReturnCallback(function(string $output) use ($changeToExpect) {
                 self::assertContains($changeToExpect, $output);
             });
 

test/unit/Command/ApiCompareTest.php

Spacing +4 added lines, -4 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-declare(strict_types=1);
+declare(strict_types = 1);
 
 namespace RoaveTest\ApiCompare\Command;
 
@@ -65,7 +65,7 @@ 
 
     public function setUp() : void
     {
-        $this->sourceRepository = CheckedOutRepository::fromPath(realpath(__DIR__ . '/../../../'));
+        $this->sourceRepository = CheckedOutRepository::fromPath(realpath(__DIR__.'/../../../'));
         chdir((string) $this->sourceRepository);
 
         $this->input   = $this->createMock(InputInterface::class);
@@ -225,7 +225,7 @@ 
 
         $this->output->expects(self::any())
             ->method('writeln')
-            ->willReturnCallback(function (string $output) use ($changeToExpect) {
+            ->willReturnCallback(function(string $output) use ($changeToExpect) {
                 self::assertContains($changeToExpect, $output);
             });
     }
@@ -272,7 +272,7 @@ 
 
         $this->getVersions->expects(self::once())
             ->method('fromRepository')
-            ->with(self::callback(function (CheckedOutRepository $checkedOutRepository) : bool {
+            ->with(self::callback(function(CheckedOutRepository $checkedOutRepository) : bool {
                 self::assertEquals($this->sourceRepository, $checkedOutRepository);
                 return true;
             }))

src/Formatter/MarkdownPipedToSymfonyConsoleFormatter.php

Spacing +6 added lines, -6 removed lines

@@ -1,5 +1,5 @@ 
 <?php
-declare(strict_types=1);
+declare(strict_types = 1);
 
 namespace Roave\ApiCompare\Formatter;
 
@@ -27,21 +27,21 @@ 
         $this->output->writeln(
             "# Added\n"
             . implode('', $this->convertFilteredChangesToMarkdownBulletList(
-                function (Change $change) : bool {
+                function(Change $change) : bool {
                     return $change->isAdded();
                 },
                 ...$arrayOfChanges
             ))
             . "\n# Changed\n"
             . implode('', $this->convertFilteredChangesToMarkdownBulletList(
-                function (Change $change) : bool {
+                function(Change $change) : bool {
                     return $change->isChanged();
                 },
                 ...$arrayOfChanges
             ))
             . "\n# Removed\n"
             . implode('', $this->convertFilteredChangesToMarkdownBulletList(
-                function (Change $change) : bool {
+                function(Change $change) : bool {
                     return $change->isRemoved();
                 },
                 ...$arrayOfChanges
@@ -52,8 +52,8 @@ 
     private function convertFilteredChangesToMarkdownBulletList(callable $filterFunction, Change ...$changes) : array
     {
         return array_map(
-            function (Change $change) : string {
-                return ' - ' . str_replace(['ADDED: ', 'CHANGED: ', 'REMOVED: '], '', trim((string)$change)) . "\n";
+            function(Change $change) : string {
+                return ' - '.str_replace(['ADDED: ', 'CHANGED: ', 'REMOVED: '], '', trim((string) $change))."\n";
             },
             array_filter($changes, $filterFunction)
         );

src/Command/ApiCompare.php

Spacing +6 added lines, -6 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-declare(strict_types=1);
+declare(strict_types = 1);
 
 namespace Roave\ApiCompare\Command;
 
@@ -113,21 +113,21 @@ 
         $toRevision  = $this->parseRevision->fromStringForRepository($input->getOption('to'), $sourceRepo);
         $sourcesPath = $input->getArgument('sources-path');
 
-        $stdErr->writeln(sprintf('Comparing from %s to %s...', (string)$fromRevision, (string)$toRevision));
+        $stdErr->writeln(sprintf('Comparing from %s to %s...', (string) $fromRevision, (string) $toRevision));
 
         $fromPath = $this->git->checkout($sourceRepo, $fromRevision);
         $toPath   = $this->git->checkout($sourceRepo, $toRevision);
 
         try {
-            $fromSources = $fromPath . '/' . $sourcesPath;
-            $toSources   = $toPath . '/' . $sourcesPath;
+            $fromSources = $fromPath.'/'.$sourcesPath;
+            $toSources   = $toPath.'/'.$sourcesPath;
 
             Assert::that($fromSources)->directory();
             Assert::that($toSources)->directory();
 
             $changes = $this->comparator->compare(
-                $this->reflectorFactory->__invoke((string) $fromPath . '/' . $sourcesPath),
-                $this->reflectorFactory->__invoke((string) $toPath . '/' . $sourcesPath)
+                $this->reflectorFactory->__invoke((string) $fromPath.'/'.$sourcesPath),
+                $this->reflectorFactory->__invoke((string) $toPath.'/'.$sourcesPath)
             );
 
             (new SymfonyConsoleTextFormatter($stdErr))->write($changes);