Roave / BackwardCompatibilityCheck

test/unit/ChangeTest.php

Spacing +7 added lines, -7 removed lines

@@ -18,7 +18,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());
@@ -29,7 +29,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());
@@ -40,7 +40,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());
@@ -50,7 +50,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());
@@ -61,7 +61,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());
@@ -72,7 +72,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());
@@ -83,7 +83,7 @@ 
     {
         $failure = new Exception('changeText');
         $change  = Change::skippedDueToFailure($failure);
-        self::assertSame('[BC] SKIPPED: ' . $failure->getMessage(), (string) $change);
+        self::assertSame('[BC] SKIPPED: '.$failure->getMessage(), (string) $change);
         self::assertFalse($change->isAdded());
         self::assertFalse($change->isChanged());
         self::assertFalse($change->isRemoved());

bin/roave-backward-compatibility-check.php

Spacing +5 added lines, -5 removed lines

@@ -31,11 +31,11 @@ 
 use Symfony\Component\Console\Output\ConsoleOutput;
 use function file_exists;
 
-(function () : void {
-    (function () : void {
+(function() : void {
+    (function() : void {
         $autoloaderLocations = [
-            __DIR__ . '/../vendor/autoload.php', // Installed by cloning the project and running `composer install`
-            __DIR__ . '/../../../autoload.php',  // Installed via `composer require`
+            __DIR__.'/../vendor/autoload.php', // Installed by cloning the project and running `composer install`
+            __DIR__.'/../../../autoload.php', // Installed via `composer require`
         ];
 
         foreach ($autoloaderLocations as $autoload) {
@@ -64,7 +64,7 @@ 
         new GetVersionCollectionFromGitRepository(),
         new PickLastMinorVersionFromCollection(),
         new LocateDependenciesViaComposer(
-            function (string $installationPath) use ($composerIo) : Installer {
+            function(string $installationPath) use ($composerIo) : Installer {
                 return Installer::create(
                     $composerIo,
                     (new Factory())->createComposer(

test/unit/Command/AssertBackwardsCompatibleTest.php

Spacing +3 added lines, -3 removed lines

@@ -75,7 +75,7 @@ 
 
     public function setUp() : void
     {
-        $repositoryPath = realpath(__DIR__ . '/../../../');
+        $repositoryPath = realpath(__DIR__.'/../../../');
 
         self::assertInternalType('string', $repositoryPath);
 
@@ -274,7 +274,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);
             });
     }
@@ -358,7 +358,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/Git/GitCheckoutRevisionToTemporaryPathTest.php

Spacing +2 added lines, -2 removed lines

@@ -47,7 +47,7 @@ 
 
     public function testExceptionIsThrownWhenTwoPathsCollide() : void
     {
-        $git              = new GitCheckoutRevisionToTemporaryPath(function () : string {
+        $git              = new GitCheckoutRevisionToTemporaryPath(function() : string {
             return 'foo';
         });
         $sourceRepository = $this->sourceRepository();
@@ -75,7 +75,7 @@ 
 
     private function sourceRepository() : CheckedOutRepository
     {
-        $repositoryPath = realpath(__DIR__ . '/../../..');
+        $repositoryPath = realpath(__DIR__.'/../../..');
 
         self::assertInternalType('string', $repositoryPath);
 

test/unit/LocateDependencies/LocateDependenciesViaComposerTest.php

Spacing +11 added lines, -11 removed lines

@@ -53,7 +53,7 @@ 
         $this->originalCwd       = $originalCwd;
         $this->composerInstaller = $this->createMock(Installer::class);
         $this->astLocator        = (new BetterReflection())->astLocator();
-        $this->makeInstaller     = function (string $installationPath) : Installer {
+        $this->makeInstaller     = function(string $installationPath) : Installer {
             self::assertSame($this->expectedInstallatonPath, $installationPath);
 
             return $this->composerInstaller;
@@ -102,13 +102,13 @@ 
 
     public function testWillLocateDependencies() : void
     {
-        $this->expectedInstallatonPath = $this->realpath(__DIR__ . '/../../asset/composer-installation-structure');
+        $this->expectedInstallatonPath = $this->realpath(__DIR__.'/../../asset/composer-installation-structure');
 
         $this
             ->composerInstaller
             ->expects(self::once())
             ->method('run')
-            ->willReturnCallback(function () : void {
+            ->willReturnCallback(function() : void {
                 self::assertSame($this->expectedInstallatonPath, getcwd());
             });
 
@@ -128,8 +128,8 @@ 
         self::assertEquals(
             new StaticClassMapSourceLocator(
                 [
-                    'A\\ClassName' => $this->realpath(__DIR__ . '/../../asset/composer-installation-structure/AClassName.php'),
-                    'B\\ClassName' => $this->realpath(__DIR__ . '/../../asset/composer-installation-structure/BClassName.php'),
+                    'A\\ClassName' => $this->realpath(__DIR__.'/../../asset/composer-installation-structure/AClassName.php'),
+                    'B\\ClassName' => $this->realpath(__DIR__.'/../../asset/composer-installation-structure/BClassName.php'),
                 ],
                 $this->astLocator
             ),
@@ -138,11 +138,11 @@ 
         self::assertEquals(
             new AggregateSourceLocator([
                 new SingleFileSourceLocator(
-                    $this->realpath(__DIR__ . '/../../asset/composer-installation-structure/included-file-1.php'),
+                    $this->realpath(__DIR__.'/../../asset/composer-installation-structure/included-file-1.php'),
                     $this->astLocator
                 ),
                 new SingleFileSourceLocator(
-                    $this->realpath(__DIR__ . '/../../asset/composer-installation-structure/included-file-2.php'),
+                    $this->realpath(__DIR__.'/../../asset/composer-installation-structure/included-file-2.php'),
                     $this->astLocator
                 ),
             ]),
@@ -154,13 +154,13 @@ 
 
     public function testWillLocateDependenciesEvenWithoutAutoloadFiles() : void
     {
-        $this->expectedInstallatonPath = $this->realpath(__DIR__ . '/../../asset/composer-installation-structure-without-autoload-files');
+        $this->expectedInstallatonPath = $this->realpath(__DIR__.'/../../asset/composer-installation-structure-without-autoload-files');
 
         $this
             ->composerInstaller
             ->expects(self::once())
             ->method('run')
-            ->willReturnCallback(function () : void {
+            ->willReturnCallback(function() : void {
                 self::assertSame($this->expectedInstallatonPath, getcwd());
             });
 
@@ -180,8 +180,8 @@ 
         self::assertEquals(
             new StaticClassMapSourceLocator(
                 [
-                    'A\\ClassName' => $this->realpath(__DIR__ . '/../../asset/composer-installation-structure-without-autoload-files/AClassName.php'),
-                    'B\\ClassName' => $this->realpath(__DIR__ . '/../../asset/composer-installation-structure-without-autoload-files/BClassName.php'),
+                    'A\\ClassName' => $this->realpath(__DIR__.'/../../asset/composer-installation-structure-without-autoload-files/AClassName.php'),
+                    'B\\ClassName' => $this->realpath(__DIR__.'/../../asset/composer-installation-structure-without-autoload-files/BClassName.php'),
                 ],
                 $this->astLocator
             ),

test/unit/LocateSources/LocateSourcesViaComposerJsonTest.php

Spacing +13 added lines, -13 removed lines

@@ -45,37 +45,37 @@
     {
         return [
             'empty composer definition' => [
-                __DIR__ . '/../../asset/located-sources/empty',
+                __DIR__.'/../../asset/located-sources/empty',
                 new AggregateSourceLocator([
                     new DirectoriesSourceLocator([], $this->astLocator()),
                 ]),
             ],
             'composer definition with everything' => [
-                __DIR__ . '/../../asset/located-sources/composer-definition-with-everything',
+                __DIR__.'/../../asset/located-sources/composer-definition-with-everything',
                 new AggregateSourceLocator([
                     new DirectoriesSourceLocator(
                         [
-                            $this->realPath(__DIR__ . '/../../asset/located-sources/composer-definition-with-everything/foo0'),
-                            $this->realPath(__DIR__ . '/../../asset/located-sources/composer-definition-with-everything/bar4'),
-                            $this->realPath(__DIR__ . '/../../asset/located-sources/composer-definition-with-everything/baz4_0'),
-                            $this->realPath(__DIR__ . '/../../asset/located-sources/composer-definition-with-everything/baz4_1'),
-                            $this->realPath(__DIR__ . '/../../asset/located-sources/composer-definition-with-everything/baz4_2'),
-                            $this->realPath(__DIR__ . '/../../asset/located-sources/composer-definition-with-everything/baz4_3'),
-                            $this->realPath(__DIR__ . '/../../asset/located-sources/composer-definition-with-everything/classmap0'),
-                            $this->realPath(__DIR__ . '/../../asset/located-sources/composer-definition-with-everything/classmap1'),
+                            $this->realPath(__DIR__.'/../../asset/located-sources/composer-definition-with-everything/foo0'),
+                            $this->realPath(__DIR__.'/../../asset/located-sources/composer-definition-with-everything/bar4'),
+                            $this->realPath(__DIR__.'/../../asset/located-sources/composer-definition-with-everything/baz4_0'),
+                            $this->realPath(__DIR__.'/../../asset/located-sources/composer-definition-with-everything/baz4_1'),
+                            $this->realPath(__DIR__.'/../../asset/located-sources/composer-definition-with-everything/baz4_2'),
+                            $this->realPath(__DIR__.'/../../asset/located-sources/composer-definition-with-everything/baz4_3'),
+                            $this->realPath(__DIR__.'/../../asset/located-sources/composer-definition-with-everything/classmap0'),
+                            $this->realPath(__DIR__.'/../../asset/located-sources/composer-definition-with-everything/classmap1'),
                         ],
                         $this->astLocator()
                     ),
                     new SingleFileSourceLocator(
-                        $this->realPath(__DIR__ . '/../../asset/located-sources/composer-definition-with-everything/classmap2/file.php'),
+                        $this->realPath(__DIR__.'/../../asset/located-sources/composer-definition-with-everything/classmap2/file.php'),
                         $this->astLocator()
                     ),
                     new SingleFileSourceLocator(
-                        $this->realPath(__DIR__ . '/../../asset/located-sources/composer-definition-with-everything/files/foo.php'),
+                        $this->realPath(__DIR__.'/../../asset/located-sources/composer-definition-with-everything/files/foo.php'),
                         $this->astLocator()
                     ),
                     new SingleFileSourceLocator(
-                        $this->realPath(__DIR__ . '/../../asset/located-sources/composer-definition-with-everything/files/bar.php'),
+                        $this->realPath(__DIR__.'/../../asset/located-sources/composer-definition-with-everything/files/bar.php'),
                         $this->astLocator()
                     ),
                 ]),

src/LocateSources/LocateSourcesViaComposerJson.php

Indentation +2 added lines, -2 removed lines

@@ -43,8 +43,8 @@
         $composerJsonPath = $realInstallationPath . '/composer.json';
 
         Assert::that($composerJsonPath)
-              ->file()
-              ->readable();
+                ->file()
+                ->readable();
 
         $composerDefinitionString = file_get_contents($composerJsonPath);
 

Spacing +8 added lines, -8 removed lines

@@ -40,7 +40,7 @@ 
 
         Assert::that($realInstallationPath)->string();
 
-        $composerJsonPath = $realInstallationPath . '/composer.json';
+        $composerJsonPath = $realInstallationPath.'/composer.json';
 
         Assert::that($composerJsonPath)
               ->file()
@@ -75,7 +75,7 @@ 
                 ),
             ],
             array_map(
-                function (string $path) : SourceLocator {
+                function(string $path) : SourceLocator {
                     return new SingleFileSourceLocator($path, $this->astLocator);
                 },
                 array_map(
@@ -84,7 +84,7 @@ 
                 )
             ),
             array_map(
-                function (string $path) : SourceLocator {
+                function(string $path) : SourceLocator {
                     return new SingleFileSourceLocator($path, $this->astLocator);
                 },
                 array_map(
@@ -104,7 +104,7 @@ 
     {
         return array_values(array_filter(
             $autoloadDefinition['classmap'] ?? [],
-            function (string $path) use ($installationPath) : bool {
+            function(string $path) use ($installationPath) : bool {
                 $filePath = ($this->prependInstallationPath($installationPath))($path);
 
                 return is_dir($filePath);
@@ -121,7 +121,7 @@ 
     {
         return array_values(array_filter(
             $autoloadDefinition['classmap'] ?? [],
-            function (string $path) use ($installationPath) : bool {
+            function(string $path) use ($installationPath) : bool {
                 $filePath = ($this->prependInstallationPath($installationPath))($path);
 
                 return is_file($filePath);
@@ -168,12 +168,12 @@ 
     /** @return callable (string $path) : string */
     private function prependInstallationPath(string $installationPath) : callable
     {
-        return function (string $path) use ($installationPath) : string {
+        return function(string $path) use ($installationPath) : string {
             if (strpos($path, './') === 0) {
-                return $installationPath . '/' . substr($path, 2);
+                return $installationPath.'/'.substr($path, 2);
             }
 
-            return $installationPath . '/' . ltrim($path, '/');
+            return $installationPath.'/'.ltrim($path, '/');
         };
     }
 

src/Git/GitCheckoutRevisionToTemporaryPath.php

Spacing +1 added lines, -1 removed lines

@@ -51,7 +51,7 @@
     private function generateTemporaryPathFor(Revision $revision) : string
     {
         $uniquePathGenerator = $this->uniquenessFunction;
-        $checkoutDirectory   = sys_get_temp_dir() . '/api-compare-' . $uniquePathGenerator($revision . '_');
+        $checkoutDirectory   = sys_get_temp_dir().'/api-compare-'.$uniquePathGenerator($revision.'_');
 
         if (file_exists($checkoutDirectory)) {
             throw new RuntimeException(sprintf(

src/Formatter/MarkdownPipedToSymfonyConsoleFormatter.php

Spacing +5 added lines, -5 removed lines

@@ -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($change->__toString())) . "\n";
+            function(Change $change) : string {
+                return ' - '.str_replace(['ADDED: ', 'CHANGED: ', 'REMOVED: '], '', trim($change->__toString()))."\n";
             },
             array_filter($changes, $filterFunction)
         );