Roave / BackwardCompatibilityCheck

src/Command/ApiCompare.php

Spacing +5 added lines, -5 removed lines

@@ -124,23 +124,23 @@
         $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(
-                    $fromPath . '/' . $sourcesPath,
+                    $fromPath.'/'.$sourcesPath,
                     new AggregateSourceLocator() // no dependencies
                 ),
                 $this->reflectorFactory->__invoke(
-                    $fromPath . '/' . $sourcesPath,
+                    $fromPath.'/'.$sourcesPath,
                     $this->locateDependencies->__invoke((string) $fromPath)
                 ),
                 $this->reflectorFactory->__invoke(
-                    $toPath . '/' . $sourcesPath,
+                    $toPath.'/'.$sourcesPath,
                     $this->locateDependencies->__invoke((string) $toPath)
                 )
             );

src/Git/GitCheckoutRevisionToTemporaryPath.php

Spacing +1 added lines, -1 removed lines

@@ -16,7 +16,7 @@
      */
     public function checkout(CheckedOutRepository $sourceRepository, Revision $revision) : CheckedOutRepository
     {
-        $checkoutDirectory = sys_get_temp_dir() . '/api-compare-' . (string) $revision;
+        $checkoutDirectory = sys_get_temp_dir().'/api-compare-'.(string) $revision;
 
         (new Process(['git', 'clone', (string) $sourceRepository, $checkoutDirectory]))->mustRun();
         (new Process(['git', 'checkout', (string) $revision]))->setWorkingDirectory($checkoutDirectory)->mustRun();

src/Git/GetVersionCollectionFromGitRepository.php

Spacing +1 added lines, -1 removed lines

@@ -31,7 +31,7 @@
         // @todo handle invalid versions more gracefully (drop them)
         return VersionsCollection::fromArray(array_filter(
             explode("\n", $output),
-            function (string $maybeVersion) {
+            function(string $maybeVersion) {
                 return trim($maybeVersion) !== '';
             }
         ));

src/Git/CheckedOutRepository.php

Spacing +1 added lines, -1 removed lines

@@ -18,7 +18,7 @@
     public static function fromPath(string $path) : self
     {
         Assert::that($path)->directory();
-        Assert::that($path . '/.git')->directory();
+        Assert::that($path.'/.git')->directory();
         $instance       = new self();
         $instance->path = $path;
         return $instance;

src/Comparator/BackwardsCompatibility/ClassBased/ConstantRemoved.php

Spacing +2 added lines, -2 removed lines

@@ -21,7 +21,7 @@ 
             $this->accessibleConstants($toClass)
         );
 
-        return Changes::fromArray(array_values(array_map(function (ReflectionClassConstant $constant) use ($fromClass) : Change {
+        return Changes::fromArray(array_values(array_map(function(ReflectionClassConstant $constant) use ($fromClass) : Change {
             return Change::removed(
                 sprintf('Constant %s::%s was removed', $fromClass->getName(), $constant->getName()),
                 true
@@ -32,7 +32,7 @@ 
     /** @return ReflectionClassConstant[] */
     private function accessibleConstants(ReflectionClass $class) : array
     {
-        return array_filter($class->getReflectionConstants(), function (ReflectionClassConstant $constant) : bool {
+        return array_filter($class->getReflectionConstants(), function(ReflectionClassConstant $constant) : bool {
             return $constant->isPublic() || $constant->isProtected();
         });
     }

src/SourceLocator/StaticClassMapSourceLocator.php

Spacing +1 added lines, -1 removed lines

@@ -34,7 +34,7 @@
      */
     protected function createLocatedSource(Identifier $identifier) : ?LocatedSource
     {
-        if (! $identifier->isClass()) {
+        if (!$identifier->isClass()) {
             return null;
         }
 

bin/api-compare.php

Spacing +3 added lines, -3 removed lines

@@ -20,9 +20,9 @@
 use Symfony\Component\Console\Application;
 use function file_exists;
 
-(function () : void {
-    foreach ([__DIR__ . '/../vendor/autoload.php', __DIR__ . '/../autoload.php'] as $autoload) {
-        if (! file_exists($autoload)) {
+(function() : void {
+    foreach ([__DIR__.'/../vendor/autoload.php', __DIR__.'/../autoload.php'] as $autoload) {
+        if (!file_exists($autoload)) {
             continue;
         }
 

test/unit/ChangeTest.php

Spacing +6 added lines, -6 removed lines

@@ -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/Variance/TypeIsCovariantTest.php

Spacing +2 added lines, -2 removed lines

@@ -235,7 +235,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)],
@@ -259,7 +259,7 @@ 
     /** @dataProvider existingNullableTypeStrings */
     public function testCovarianceConsidersNullability(string $type) : void
     {
-        $reflector   = new ClassReflector(new StringSourceLocator(
+        $reflector = new ClassReflector(new StringSourceLocator(
             <<<'PHP'
 <?php