Roave / BackwardCompatibilityCheck

src/Comparator/BackwardsCompatibility/FunctionBased/ReturnTypeChanged.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\FunctionBased;
 
@@ -78,7 +78,7 @@ 
 
     private function typeToString(?ReflectionType $type) : string
     {
-        if (! $type) {
+        if (!$type) {
             return 'no type';
         }
 

test/unit/Formatter/MarkdownPipedToSymfonyConsoleFormatterTest.php

Spacing +2 added lines, -2 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-declare(strict_types=1);
+declare(strict_types = 1);
 
 namespace RoaveTest\ApiCompare\Formatter;
 
@@ -36,7 +36,7 @@ 
 
         $output->expects(self::any())
             ->method('writeln')
-            ->willReturnCallback(function (string $output) use ($changeToExpect) : void {
+            ->willReturnCallback(function(string $output) use ($changeToExpect) : void {
                 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) : void {
+            ->willReturnCallback(function(string $output) use ($changeToExpect) : void {
                 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;
             }))

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