Roave / BackwardCompatibilityCheck

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

src/Comparator/BackwardsCompatibility/ClassBased/PropertyVisibilityReduced.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;
 
@@ -37,19 +37,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 the strings "public", "protected" and "private" can be compared directly as they sort correctly for our purposes
                 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 (
@@ -71,7 +71,7 @@ 
     /** @return string[] */
     private function propertyVisibilities(ReflectionClass $class) : array
     {
-        return array_map(function (ReflectionProperty $property) : string {
+        return array_map(function(ReflectionProperty $property) : string {
             if ($property->isPublic()) {
                 return self::VISIBILITY_PUBLIC;
             }

src/Comparator/BackwardsCompatibility/ClassBased/MethodVisibilityReduced.php

Spacing +6 added lines, -6 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-declare(strict_types=1);
+declare(strict_types = 1);
 
 namespace Roave\ApiCompare\Comparator\BackwardsCompatibility\ClassBased;
 
@@ -37,19 +37,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 the strings "public", "protected" and "private" can be compared directly as they sort correctly for our purposes
                 return $visibilities[0] > $visibilities[1];
             }
         );
 
-        return Changes::fromArray(array_values(array_map(function (
+        return Changes::fromArray(array_values(array_map(function(
             string $methodName,
             array $visibilities
         ) use (
@@ -74,10 +74,10 @@ 
         $methods = $class->getMethods();
 
         return array_combine(
-            array_map(function (ReflectionMethod $method) : string {
+            array_map(function(ReflectionMethod $method) : string {
                 return $method->getName();
             }, $methods),
-            array_map(function (ReflectionMethod $method) : string {
+            array_map(function(ReflectionMethod $method) : string {
                 if ($method->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_map(function (string $constantName) use ($fromClass) : Change {
+        return Changes::fromArray(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);
     }

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 the strings "public", "protected" and "private" can be compared directly as they sort correctly for our purposes
                 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;
             }