Roave / BackwardCompatibilityCheck

test/unit/ChangeTest.php

Spacing +7 added lines, -7 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-declare(strict_types=1);
+declare(strict_types = 1);
 
 namespace RoaveTest\ApiCompare;
 
@@ -17,7 +17,7 @@ 
     {
         $changeText = uniqid('changeText', true);
         $change     = Change::added($changeText, false);
-        self::assertSame('     ADDED: ' . $changeText, (string) $change);
+        self::assertSame('     ADDED: '.$changeText, (string) $change);
         self::assertTrue($change->isAdded());
         self::assertFalse($change->isChanged());
         self::assertFalse($change->isRemoved());
@@ -27,7 +27,7 @@ 
     {
         $changeText = uniqid('changeText', true);
         $change     = Change::added($changeText, true);
-        self::assertSame('[BC] ADDED: ' . $changeText, (string) $change);
+        self::assertSame('[BC] ADDED: '.$changeText, (string) $change);
         self::assertTrue($change->isAdded());
         self::assertFalse($change->isChanged());
         self::assertFalse($change->isRemoved());
@@ -37,7 +37,7 @@ 
     {
         $changeText = uniqid('changeText', true);
         $change     = Change::changed($changeText, false);
-        self::assertSame('     CHANGED: ' . $changeText, (string) $change);
+        self::assertSame('     CHANGED: '.$changeText, (string) $change);
         self::assertFalse($change->isAdded());
         self::assertTrue($change->isChanged());
         self::assertFalse($change->isRemoved());
@@ -47,7 +47,7 @@ 
     {
         $changeText = uniqid('changeText', true);
         $change     = Change::changed($changeText, true);
-        self::assertSame('[BC] CHANGED: ' . $changeText, (string) $change);
+        self::assertSame('[BC] CHANGED: '.$changeText, (string) $change);
         self::assertFalse($change->isAdded());
         self::assertTrue($change->isChanged());
         self::assertFalse($change->isRemoved());
@@ -57,7 +57,7 @@ 
     {
         $changeText = uniqid('changeText', true);
         $change     = Change::removed($changeText, false);
-        self::assertSame('     REMOVED: ' . $changeText, (string) $change);
+        self::assertSame('     REMOVED: '.$changeText, (string) $change);
         self::assertFalse($change->isAdded());
         self::assertFalse($change->isChanged());
         self::assertTrue($change->isRemoved());
@@ -67,7 +67,7 @@ 
     {
         $changeText = uniqid('changeText', true);
         $change     = Change::removed($changeText, true);
-        self::assertSame('[BC] REMOVED: ' . $changeText, (string) $change);
+        self::assertSame('[BC] REMOVED: '.$changeText, (string) $change);
         self::assertFalse($change->isAdded());
         self::assertFalse($change->isChanged());
         self::assertTrue($change->isRemoved());

test/unit/Comparator/BackwardsCompatibility/ClassBased/ClassBecameFinalTest.php

Spacing +3 added lines, -3 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-declare(strict_types=1);
+declare(strict_types = 1);
 
 namespace RoaveTest\ApiCompare\Comparator\BackwardsCompatibility\ClassBased;
 
@@ -31,7 +31,7 @@ 
 
         self::assertSame(
             $expectedMessages,
-            array_map(function (Change $change) : string {
+            array_map(function(Change $change) : string {
                 return $change->__toString();
             }, iterator_to_array($changes))
         );
@@ -53,7 +53,7 @@ 
             ,
             $locator
         ));
-        $toReflector   = new ClassReflector(new StringSourceLocator(
+        $toReflector = new ClassReflector(new StringSourceLocator(
             <<<'PHP'
 <?php
 

test/unit/Comparator/Variance/TypeIsContravariantTest.php

Spacing +3 added lines, -3 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-declare(strict_types=1);
+declare(strict_types = 1);
 
 namespace RoaveTest\ApiCompare\Comparator\BackwardsCompatibility\ClassBased;
 
@@ -224,7 +224,7 @@ 
         return array_merge(
             [[null]],
             array_merge(...array_map(
-                function (string $type) use ($reflector) : array {
+                function(string $type) use ($reflector) : array {
                     return [
                         [ReflectionType::createFromTypeAndReflector($type, false, $reflector)],
                         [ReflectionType::createFromTypeAndReflector($type, true, $reflector)],
@@ -248,7 +248,7 @@ 
     /** @dataProvider existingNullableTypeStrings */
     public function testContravarianceConsidersNullability(string $type) : void
     {
-        $reflector   = new ClassReflector(new StringSourceLocator(
+        $reflector = new ClassReflector(new StringSourceLocator(
             <<<'PHP'
 <?php
 

test/unit/Comparator/Variance/TypeIsCovariantTest.php

Spacing +3 added lines, -3 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-declare(strict_types=1);
+declare(strict_types = 1);
 
 namespace RoaveTest\ApiCompare\Comparator\BackwardsCompatibility\ClassBased;
 
@@ -225,7 +225,7 @@ 
         return array_merge(
             [[null]],
             array_merge(...array_map(
-                function (string $type) use ($reflector) : array {
+                function(string $type) use ($reflector) : array {
                     return [
                         [ReflectionType::createFromTypeAndReflector($type, false, $reflector)],
                         [ReflectionType::createFromTypeAndReflector($type, true, $reflector)],
@@ -249,7 +249,7 @@ 
     /** @dataProvider existingNullableTypeStrings */
     public function testCovarianceConsidersNullability(string $type) : void
     {
-        $reflector   = new ClassReflector(new StringSourceLocator(
+        $reflector = new ClassReflector(new StringSourceLocator(
             <<<'PHP'
 <?php
 

src/Formatter/MarkdownPipedToSymfonyConsoleFormatter.php

Spacing +6 added lines, -6 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-declare(strict_types=1);
+declare(strict_types = 1);
 
 namespace Roave\ApiCompare\Formatter;
 
@@ -30,21 +30,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
@@ -56,8 +56,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 +5 added lines, -5 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-declare(strict_types=1);
+declare(strict_types = 1);
 
 namespace Roave\ApiCompare\Command;
 
@@ -116,15 +116,15 @@ 
         $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/MethodChanged.php

Spacing +3 added lines, -3 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-declare(strict_types=1);
+declare(strict_types = 1);
 
 namespace Roave\ApiCompare\Comparator\BackwardsCompatibility\ClassBased;
 
@@ -33,7 +33,7 @@ 
 
         return array_reduce(
             array_keys($commonMethods),
-            function (Changes $accumulator, string $methodName) use ($methodsFrom, $methodsTo) : Changes {
+            function(Changes $accumulator, string $methodName) use ($methodsFrom, $methodsTo) : Changes {
                 return $accumulator->mergeWith($this->checkMethod->compare(
                     $methodsFrom[$methodName],
                     $methodsTo[$methodName]
@@ -49,7 +49,7 @@ 
         $methods = $class->getMethods();
 
         return array_combine(
-            array_map(function (ReflectionMethod $method) : string {
+            array_map(function(ReflectionMethod $method) : string {
                 return strtolower($method->getName());
             }, $methods),
             $methods

src/Comparator/BackwardsCompatibility/ClassBased/PropertyRemoved.php

Spacing +3 added lines, -3 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-declare(strict_types=1);
+declare(strict_types = 1);
 
 namespace Roave\ApiCompare\Comparator\BackwardsCompatibility\ClassBased;
 
@@ -37,7 +37,7 @@ 
             array_keys($this->accessibleProperties($toClass))
         );
 
-        return Changes::fromArray(array_values(array_map(function (string $property) use ($fromProperties) : Change {
+        return Changes::fromArray(array_values(array_map(function(string $property) use ($fromProperties) : Change {
             return Change::removed(
                 sprintf('Property %s was removed', $this->formatProperty->__invoke($fromProperties[$property])),
                 true
@@ -48,7 +48,7 @@ 
     /** @return ReflectionProperty[] */
     private function accessibleProperties(ReflectionClass $class) : array
     {
-        return array_filter($class->getProperties(), function (ReflectionProperty $property) : bool {
+        return array_filter($class->getProperties(), function(ReflectionProperty $property) : bool {
             return $property->isPublic() || $property->isProtected();
         });
     }

src/Comparator/BackwardsCompatibility/ClassBased/PropertyChanged.php

Spacing +2 added lines, -2 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-declare(strict_types=1);
+declare(strict_types = 1);
 
 namespace Roave\ApiCompare\Comparator\BackwardsCompatibility\ClassBased;
 
@@ -29,7 +29,7 @@ 
 
         return array_reduce(
             array_keys($commonProperties),
-            function (Changes $accumulator, string $propertyName) use ($propertiesFrom, $propertiesTo) : Changes {
+            function(Changes $accumulator, string $propertyName) use ($propertiesFrom, $propertiesTo) : Changes {
                 return $accumulator->mergeWith($this->checkProperty->compare(
                     $propertiesFrom[$propertyName],
                     $propertiesTo[$propertyName]