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/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 +4 added lines, -4 removed lines

@@ -48,13 +48,13 @@ 
     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 () : void {
+        $this->runInDirectory(function() : void {
             $this->installer->run();
         }, $installationPath);
 
-        $autoloadStatic = $installationPath . '/vendor/composer/autoload_static.php';
+        $autoloadStatic = $installationPath.'/vendor/composer/autoload_static.php';
 
         Assert::that($autoloadStatic)->file();
 
@@ -107,7 +107,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;
         }
 

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