Roave / BackwardCompatibilityCheck

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

src/Formatter/ReflectionFunctionAbstractName.php

Spacing +3 added lines, -3 removed lines

@@ -13,12 +13,12 @@
     {
         if ($function instanceof ReflectionMethod) {
             if ($function->isStatic()) {
-                return $function->getDeclaringClass()->getName() . '::' . $function->getName() . '()';
+                return $function->getDeclaringClass()->getName().'::'.$function->getName().'()';
             }
 
-            return $function->getDeclaringClass()->getName() . '#' . $function->getName() . '()';
+            return $function->getDeclaringClass()->getName().'#'.$function->getName().'()';
         }
 
-        return $function->getName() . '()';
+        return $function->getName().'()';
     }
 }

test/unit/LocateDependencies/LocateDependenciesViaComposerTest.php

Spacing +10 added lines, -10 removed lines

@@ -93,7 +93,7 @@ 
 
     public function testWillLocateDependencies() : void
     {
-        $composerInstallationStructure = realpath(__DIR__ . '/../../asset/composer-installation-structure');
+        $composerInstallationStructure = realpath(__DIR__.'/../../asset/composer-installation-structure');
 
         $this
             ->makeInstaller
@@ -106,7 +106,7 @@ 
             ->composerInstaller
             ->expects(self::once())
             ->method('run')
-            ->willReturnCallback(function () use ($composerInstallationStructure) : void {
+            ->willReturnCallback(function() use ($composerInstallationStructure) : void {
                 self::assertSame($composerInstallationStructure, getcwd());
             });
 
@@ -126,8 +126,8 @@ 
         self::assertEquals(
             new StaticClassMapSourceLocator(
                 [
-                    'A\\ClassName' => realpath(__DIR__ . '/../../asset/composer-installation-structure/AClassName.php'),
-                    'B\\ClassName' => realpath(__DIR__ . '/../../asset/composer-installation-structure/BClassName.php'),
+                    'A\\ClassName' => realpath(__DIR__.'/../../asset/composer-installation-structure/AClassName.php'),
+                    'B\\ClassName' => realpath(__DIR__.'/../../asset/composer-installation-structure/BClassName.php'),
                 ],
                 $this->astLocator
             ),
@@ -136,11 +136,11 @@ 
         self::assertEquals(
             new AggregateSourceLocator([
                 new SingleFileSourceLocator(
-                    realpath(__DIR__ . '/../../asset/composer-installation-structure/included-file-1.php'),
+                    realpath(__DIR__.'/../../asset/composer-installation-structure/included-file-1.php'),
                     $this->astLocator
                 ),
                 new SingleFileSourceLocator(
-                    realpath(__DIR__ . '/../../asset/composer-installation-structure/included-file-2.php'),
+                    realpath(__DIR__.'/../../asset/composer-installation-structure/included-file-2.php'),
                     $this->astLocator
                 ),
             ]),
@@ -151,7 +151,7 @@ 
 
     public function testWillLocateDependenciesEvenWithoutAutoloadFiles() : void
     {
-        $composerInstallationStructure = realpath(__DIR__ . '/../../asset/composer-installation-structure-without-autoload-files');
+        $composerInstallationStructure = realpath(__DIR__.'/../../asset/composer-installation-structure-without-autoload-files');
 
         $this
             ->makeInstaller
@@ -164,7 +164,7 @@ 
             ->composerInstaller
             ->expects(self::once())
             ->method('run')
-            ->willReturnCallback(function () use ($composerInstallationStructure) : void {
+            ->willReturnCallback(function() use ($composerInstallationStructure) : void {
                 self::assertSame($composerInstallationStructure, getcwd());
             });
 
@@ -184,8 +184,8 @@ 
         self::assertEquals(
             new StaticClassMapSourceLocator(
                 [
-                    'A\\ClassName' => realpath(__DIR__ . '/../../asset/composer-installation-structure-without-autoload-files/AClassName.php'),
-                    'B\\ClassName' => realpath(__DIR__ . '/../../asset/composer-installation-structure-without-autoload-files/BClassName.php'),
+                    'A\\ClassName' => realpath(__DIR__.'/../../asset/composer-installation-structure-without-autoload-files/AClassName.php'),
+                    'B\\ClassName' => realpath(__DIR__.'/../../asset/composer-installation-structure-without-autoload-files/BClassName.php'),
                 ],
                 $this->astLocator
             ),

test/asset/composer-installation-structure-without-autoload-files/vendor/composer/autoload_static.php

Spacing +15 added lines, -15 removed lines

@@ -6,40 +6,40 @@
 
 class ComposerStaticInitaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa
 {
-    public static $prefixLengthsPsr4 = array (
+    public static $prefixLengthsPsr4 = array(
         'p' =>
-            array (
+            array(
                 'phpDocumentor\\Reflection\\' => 25,
             ),
     );
 
-    public static $prefixDirsPsr4 = array (
+    public static $prefixDirsPsr4 = array(
         'phpDocumentor\\Reflection\\' =>
-            array (
-                0 => __DIR__ . '/..' . '/phpdocumentor/reflection-common/src',
-                1 => __DIR__ . '/..' . '/phpdocumentor/reflection-docblock/src',
-                2 => __DIR__ . '/..' . '/phpdocumentor/type-resolver/src',
+            array(
+                0 => __DIR__.'/..'.'/phpdocumentor/reflection-common/src',
+                1 => __DIR__.'/..'.'/phpdocumentor/reflection-docblock/src',
+                2 => __DIR__.'/..'.'/phpdocumentor/type-resolver/src',
             ),
     );
 
-    public static $prefixesPsr0 = array (
+    public static $prefixesPsr0 = array(
         'P' =>
-            array (
+            array(
                 'Prophecy\\' =>
-                    array (
-                        0 => __DIR__ . '/..' . '/phpspec/prophecy/src',
+                    array(
+                        0 => __DIR__.'/..'.'/phpspec/prophecy/src',
                     ),
             ),
     );
 
-    public static $classMap = array (
-        'A\\ClassName' => __DIR__ . '/../..' . '/AClassName.php',
-        'B\\ClassName' => __DIR__ . '/../..' . '/BClassName.php',
+    public static $classMap = array(
+        'A\\ClassName' => __DIR__.'/../..'.'/AClassName.php',
+        'B\\ClassName' => __DIR__.'/../..'.'/BClassName.php',
     );
 
     public static function getInitializer(ClassLoader $loader)
     {
-        return \Closure::bind(function () use ($loader) {
+        return \Closure::bind(function() use ($loader) {
             $loader->prefixLengthsPsr4 = ComposerStaticInitaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa::$prefixLengthsPsr4;
             $loader->prefixDirsPsr4 = ComposerStaticInitaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa::$prefixDirsPsr4;
             $loader->prefixesPsr0 = ComposerStaticInitaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa::$prefixesPsr0;

src/LocateDependencies/LocateDependenciesViaComposer.php

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

test/unit/DetectChanges/BCBreak/MethodBased/MethodBecameFinalTest.php

Spacing +2 added lines, -2 removed lines

@@ -36,7 +36,7 @@ 
 
         self::assertSame(
             $expectedMessages,
-            array_map(function (Change $change) : string {
+            array_map(function(Change $change) : string {
                 return $change->__toString();
             }, iterator_to_array($changes))
         );
@@ -122,7 +122,7 @@ 
         return array_combine(
             array_keys($properties),
             array_map(
-                function (string $methodName, array $errorMessages) use ($fromClass, $toClass) : array {
+                function(string $methodName, array $errorMessages) use ($fromClass, $toClass) : array {
                     return [
                         $fromClass->getMethod($methodName),
                         $toClass->getMethod($methodName),

test/unit/DetectChanges/BCBreak/MethodBased/MethodVisibilityReducedTest.php

Spacing +2 added lines, -2 removed lines

@@ -36,7 +36,7 @@ 
 
         self::assertSame(
             $expectedMessages,
-            array_map(function (Change $change) : string {
+            array_map(function(Change $change) : string {
                 return $change->__toString();
             }, iterator_to_array($changes))
         );
@@ -108,7 +108,7 @@ 
         return array_combine(
             array_keys($properties),
             array_map(
-                function (string $method, array $errorMessages) use ($fromClass, $toClass) : array {
+                function(string $method, array $errorMessages) use ($fromClass, $toClass) : array {
                     return [
                         $fromClass->getMethod($method),
                         $toClass->getMethod($method),

test/unit/DetectChanges/BCBreak/MethodBased/MethodConcretenessChangedTest.php

Spacing +2 added lines, -2 removed lines

@@ -36,7 +36,7 @@ 
 
         self::assertSame(
             $expectedMessages,
-            array_map(function (Change $change) : string {
+            array_map(function(Change $change) : string {
                 return $change->__toString();
             }, iterator_to_array($changes))
         );
@@ -122,7 +122,7 @@ 
         return array_combine(
             array_keys($properties),
             array_map(
-                function (string $methodName, array $errorMessages) use ($fromClass, $toClass) : array {
+                function(string $methodName, array $errorMessages) use ($fromClass, $toClass) : array {
                     return [
                         $fromClass->getMethod($methodName),
                         $toClass->getMethod($methodName),

test/unit/DetectChanges/BCBreak/MethodBased/MethodScopeChangedTest.php

Spacing +2 added lines, -2 removed lines

@@ -36,7 +36,7 @@ 
 
         self::assertSame(
             $expectedMessages,
-            array_map(function (Change $change) : string {
+            array_map(function(Change $change) : string {
                 return $change->__toString();
             }, iterator_to_array($changes))
         );
@@ -122,7 +122,7 @@ 
         return array_combine(
             array_keys($properties),
             array_map(
-                function (string $methodName, array $errorMessages) use ($fromClass, $toClass) : array {
+                function(string $methodName, array $errorMessages) use ($fromClass, $toClass) : array {
                     return [
                         $fromClass->getMethod($methodName),
                         $toClass->getMethod($methodName),