Roave / BackwardCompatibilityCheck

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/LocateDependencies/LocateDependenciesViaComposer.php

Indentation +6 added lines, -6 removed lines

@@ -76,9 +76,9 @@ 
         $classMap = $classMapProperty->getDefaultValue();
 
         Assert::that($classMap)
-              ->isArray()
-              ->all()
-              ->file();
+                ->isArray()
+                ->all()
+                ->file();
 
         return new StaticClassMapSourceLocator($classMap, $this->astLocator);
     }
@@ -92,9 +92,9 @@ 
         $filesMap = $filesMapProperty->getDefaultValue();
 
         Assert::that($filesMap)
-              ->isArray()
-              ->all()
-              ->file();
+                ->isArray()
+                ->all()
+                ->file();
 
         return new AggregateSourceLocator(array_values(array_map(
             function (string $path) : SourceLocator {

Spacing +5 added lines, -5 removed lines

@@ -43,9 +43,9 @@ 
     public function __invoke(string $installationPath) : SourceLocator
     {
         Assert::that($installationPath)->directory();
-        Assert::that($installationPath . '/composer.json')->file();
+        Assert::that($installationPath.'/composer.json')->file();
 
-        $this->runInDirectory(function () use ($installationPath) : void {
+        $this->runInDirectory(function() use ($installationPath) : void {
             $installer = ($this->makeComposerInstaller)($installationPath);
 
             Assert::that($installer)->isInstanceOf(Installer::class);
@@ -62,7 +62,7 @@ 
             $installer->run();
         }, $installationPath);
 
-        $autoloadStatic = $installationPath . '/vendor/composer/autoload_static.php';
+        $autoloadStatic = $installationPath.'/vendor/composer/autoload_static.php';
 
         Assert::that($autoloadStatic)->file();
 
@@ -104,7 +104,7 @@ 
     {
         $filesMapProperty = $autoloadStatic->getProperty('files');
 
-        if (! $filesMapProperty) {
+        if (!$filesMapProperty) {
             return new AggregateSourceLocator();
         }
 
@@ -119,7 +119,7 @@ 
               ->file();
 
         return new AggregateSourceLocator(array_values(array_map(
-            function (string $path) : SourceLocator {
+            function(string $path) : SourceLocator {
                 return new SingleFileSourceLocator(
                     realpath($path),
                     $this->astLocator

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

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
 

test/unit/Comparator/Variance/TypeIsContravariantTest.php

Spacing +2 added lines, -2 removed lines

@@ -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