Roave / BackwardCompatibilityCheck

src/DetectChanges/BCBreak/ClassBased/MethodRemoved.php

Spacing +3 added lines, -3 removed lines

@@ -35,7 +35,7 @@ 
             array_change_key_case($this->accessibleMethods($toClass), CASE_UPPER)
         );
 
-        return Changes::fromList(...array_values(array_map(function (ReflectionMethod $method) : Change {
+        return Changes::fromList(...array_values(array_map(function(ReflectionMethod $method) : Change {
             return Change::removed(
                 sprintf('Method %s was removed', $this->formatFunction->__invoke($method)),
                 true
@@ -46,12 +46,12 @@ 
     /** @return ReflectionMethod[] */
     private function accessibleMethods(ReflectionClass $class) : array
     {
-        $methods = array_filter($class->getMethods(), static function (ReflectionMethod $method) : bool {
+        $methods = array_filter($class->getMethods(), static function(ReflectionMethod $method) : bool {
             return $method->isPublic() || $method->isProtected();
         });
 
         return array_combine(
-            array_map(static function (ReflectionMethod $method) : string {
+            array_map(static function(ReflectionMethod $method) : string {
                 return $method->getName();
             }, $methods),
             $methods

src/DetectChanges/BCBreak/InterfaceBased/MethodAdded.php

Spacing +3 added lines, -3 removed lines

@@ -23,11 +23,11 @@ 
         $toMethods   = $this->methods($toInterface);
         $newMethods  = array_diff_key($toMethods, $fromMethods);
 
-        if (! $newMethods) {
+        if (!$newMethods) {
             return Changes::empty();
         }
 
-        return Changes::fromList(...array_values(array_map(static function (ReflectionMethod $method) use (
+        return Changes::fromList(...array_values(array_map(static function(ReflectionMethod $method) use (
             $fromInterface
         ) : Change {
             return Change::added(
@@ -47,7 +47,7 @@ 
         $methods = $interface->getMethods();
 
         return array_combine(
-            array_map(static function (ReflectionMethod $method) : string {
+            array_map(static function(ReflectionMethod $method) : string {
                 return strtolower($method->getName());
             }, $methods),
             $methods

bin/roave-backward-compatibility-check.php

Spacing +5 added lines, -5 removed lines

@@ -32,11 +32,11 @@ 
 use Symfony\Component\Console\Output\ConsoleOutput;
 use function file_exists;
 
-(static function () : void {
-    (static function () : void {
+(static function() : void {
+    (static 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) {
@@ -65,7 +65,7 @@ 
         new GetVersionCollectionFromGitRepository(),
         new PickLastMinorVersionFromCollection(),
         new LocateDependenciesViaComposer(
-            static function (string $installationPath) use ($composerIo) : Installer {
+            static function(string $installationPath) use ($composerIo) : Installer {
                 return Installer::create(
                     $composerIo,
                     (new Factory())->createComposer(

test/unit/LocateSources/LocateSourcesViaComposerJsonTest.php

Spacing +1 added lines, -1 removed lines

@@ -29,7 +29,7 @@
     {
         $reflector = new ClassReflector(
             $this->locateSources
-                ->__invoke(__DIR__ . '/../../asset/located-sources/composer-definition-with-everything')
+                ->__invoke(__DIR__.'/../../asset/located-sources/composer-definition-with-everything')
         );
 
         self::assertSame(

test/unit/DetectChanges/BCBreak/PropertyBased/PropertyScopeChangedTest.php

Spacing +2 added lines, -2 removed lines

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

test/unit/DetectChanges/BCBreak/PropertyBased/PropertyDefaultValueChangedTest.php

Spacing +2 added lines, -2 removed lines

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

test/unit/DetectChanges/BCBreak/PropertyBased/PropertyDocumentedTypeChangedTest.php

Spacing +2 added lines, -2 removed lines

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

test/unit/DetectChanges/BCBreak/ClassBased/ClassBecameTraitTest.php

Spacing +3 added lines, -3 removed lines

@@ -33,7 +33,7 @@ 
 
         self::assertSame(
             $expectedMessages,
-            array_map(static function (Change $change) : string {
+            array_map(static function(Change $change) : string {
                 return $change->__toString();
             }, iterator_to_array($changes))
         );
@@ -63,7 +63,7 @@ 
             ,
             $locator
         ));
-        $toReflector   = new ClassReflector(new StringSourceLocator(
+        $toReflector = new ClassReflector(new StringSourceLocator(
             <<<'PHP'
 <?php
 
@@ -102,7 +102,7 @@ 
         return array_combine(
             array_keys($classes),
             array_map(
-                static function (string $className, array $errors) use ($fromReflector, $toReflector) : array {
+                static function(string $className, array $errors) use ($fromReflector, $toReflector) : array {
                     return [
                         $fromReflector->reflect($className),
                         $toReflector->reflect($className),

test/unit/DetectChanges/BCBreak/ClassBased/ClassBecameAbstractTest.php

Spacing +3 added lines, -3 removed lines

@@ -33,7 +33,7 @@ 
 
         self::assertSame(
             $expectedMessages,
-            array_map(static function (Change $change) : string {
+            array_map(static function(Change $change) : string {
                 return $change->__toString();
             }, iterator_to_array($changes))
         );
@@ -63,7 +63,7 @@ 
             ,
             $locator
         ));
-        $toReflector   = new ClassReflector(new StringSourceLocator(
+        $toReflector = new ClassReflector(new StringSourceLocator(
             <<<'PHP'
 <?php
 
@@ -102,7 +102,7 @@ 
         return array_combine(
             array_keys($classes),
             array_map(
-                static function (string $className, array $errors) use ($fromReflector, $toReflector) : array {
+                static function(string $className, array $errors) use ($fromReflector, $toReflector) : array {
                     return [
                         $fromReflector->reflect($className),
                         $toReflector->reflect($className),