Roave / BackwardCompatibilityCheck

src/DetectChanges/BCBreak/FunctionBased/ParameterDefaultValueChanged.php

Spacing +2 added lines, -2 removed lines

@@ -69,13 +69,13 @@
     {
         $optionalParameters = array_values(array_filter(
             $function->getParameters(),
-            static function (ReflectionParameter $parameter) : bool {
+            static function(ReflectionParameter $parameter) : bool {
                 return $parameter->isDefaultValueAvailable();
             }
         ));
 
         return array_combine(
-            array_map(static function (ReflectionParameter $parameter) : int {
+            array_map(static function(ReflectionParameter $parameter) : int {
                 return $parameter->getPosition();
             }, $optionalParameters),
             $optionalParameters

src/DetectChanges/BCBreak/ClassBased/MethodChanged.php

Spacing +1 added lines, -1 removed lines

@@ -49,7 +49,7 @@
         $methods = $class->getMethods();
 
         return array_combine(
-            array_map(static function (ReflectionMethod $method) : string {
+            array_map(static function(ReflectionMethod $method) : string {
                 return strtolower($method->getName());
             }, $methods),
             $methods

src/DetectChanges/BCBreak/ClassBased/ConstantRemoved.php

Spacing +2 added lines, -2 removed lines

@@ -23,7 +23,7 @@ 
             $this->accessibleConstants($toClass)
         );
 
-        return Changes::fromList(...array_values(array_map(static function (ReflectionClassConstant $constant) use ($fromClass) : Change {
+        return Changes::fromList(...array_values(array_map(static function(ReflectionClassConstant $constant) use ($fromClass) : Change {
             return Change::removed(
                 sprintf('Constant %s::%s was removed', $fromClass->getName(), $constant->getName()),
                 true
@@ -34,7 +34,7 @@ 
     /** @return ReflectionClassConstant[] */
     private function accessibleConstants(ReflectionClass $class) : array
     {
-        return array_filter($class->getReflectionConstants(), static function (ReflectionClassConstant $constant) : bool {
+        return array_filter($class->getReflectionConstants(), static function(ReflectionClassConstant $constant) : bool {
             return $constant->isPublic() || $constant->isProtected();
         });
     }

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/ClassBased/PropertyRemoved.php

Spacing +2 added lines, -2 removed lines

@@ -34,7 +34,7 @@ 
             array_keys($this->accessibleProperties($toClass))
         );
 
-        return Changes::fromList(...array_values(array_map(function (string $property) use ($fromProperties) : Change {
+        return Changes::fromList(...array_values(array_map(function(string $property) use ($fromProperties) : Change {
             return Change::removed(
                 sprintf('Property %s was removed', $this->formatProperty->__invoke($fromProperties[$property])),
                 true
@@ -45,7 +45,7 @@ 
     /** @return ReflectionProperty[] */
     private function accessibleProperties(ReflectionClass $class) : array
     {
-        return array_filter($class->getProperties(), static function (ReflectionProperty $property) : bool {
+        return array_filter($class->getProperties(), static function(ReflectionProperty $property) : bool {
             return $property->isPublic() || $property->isProtected();
         });
     }

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

src/DetectChanges/Variance/TypeIsCovariant.php

Spacing +3 added lines, -3 removed lines

@@ -31,7 +31,7 @@ 
             return false;
         }
 
-        if ($comparedType->allowsNull() && ! $type->allowsNull()) {
+        if ($comparedType->allowsNull() && !$type->allowsNull()) {
             return false;
         }
 
@@ -47,7 +47,7 @@ 
             return false;
         }
 
-        if (strtolower($typeAsString) === 'object' && ! $comparedType->isBuiltin()) {
+        if (strtolower($typeAsString) === 'object' && !$comparedType->isBuiltin()) {
             // `object` is not covariant to a defined class type
             return true;
         }
@@ -57,7 +57,7 @@ 
             return true;
         }
 
-        if (strtolower($typeAsString) === 'iterable' && ! $comparedType->isBuiltin()) {
+        if (strtolower($typeAsString) === 'iterable' && !$comparedType->isBuiltin()) {
             if ($comparedType->targetReflectionClass()->implementsInterface(Traversable::class)) {
                 // `iterable` can be restricted via any `Iterator` implementation
                 return true;

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/e2e/Command/AssertBackwardsCompatibleTest.php

Spacing +16 added lines, -16 removed lines

@@ -121,20 +121,20 @@ 
 
         unlink($this->sourcesRepository);
         mkdir($this->sourcesRepository);
-        mkdir($this->sourcesRepository . '/src');
+        mkdir($this->sourcesRepository.'/src');
 
         self::assertDirectoryExists($this->sourcesRepository);
-        self::assertDirectoryExists($this->sourcesRepository . '/src');
+        self::assertDirectoryExists($this->sourcesRepository.'/src');
 
         (new Process('git init', $this->sourcesRepository))->mustRun();
 
-        file_put_contents($this->sourcesRepository . '/composer.json', self::COMPOSER_MANIFEST);
+        file_put_contents($this->sourcesRepository.'/composer.json', self::COMPOSER_MANIFEST);
 
         (new Process('git add -A', $this->sourcesRepository))->mustRun();
         (new Process('git commit -am "Initial commit with composer manifest"', $this->sourcesRepository))->mustRun();
 
         foreach (self::CLASS_VERSIONS as $key => $classCode) {
-            file_put_contents($this->sourcesRepository . '/src/TheClass.php', $classCode);
+            file_put_contents($this->sourcesRepository.'/src/TheClass.php', $classCode);
 
             (new Process('git add -A', $this->sourcesRepository))->mustRun();
             (new Process(sprintf('git commit -am "Class sources v%d"', $key + 1), $this->sourcesRepository))->mustRun();
@@ -159,9 +159,9 @@ 
     {
         $check = new Process(
             [
-                __DIR__ . '/../../../bin/roave-backward-compatibility-check',
-                '--from=' . $this->versions[0],
-                '--to=' . $this->versions[1],
+                __DIR__.'/../../../bin/roave-backward-compatibility-check',
+                '--from='.$this->versions[0],
+                '--to='.$this->versions[1],
             ],
             $this->sourcesRepository
         );
@@ -181,7 +181,7 @@ 
     public function testWillNotRunWithoutTagsNorSpecifiedVersions() : void
     {
         $check = new Process(
-            __DIR__ . '/../../../bin/roave-backward-compatibility-check',
+            __DIR__.'/../../../bin/roave-backward-compatibility-check',
             $this->sourcesRepository
         );
 
@@ -196,9 +196,9 @@ 
     {
         $check = new Process(
             [
-                __DIR__ . '/../../../bin/roave-backward-compatibility-check',
-                '--from=' . $this->versions[0],
-                '--to=' . $this->versions[3],
+                __DIR__.'/../../../bin/roave-backward-compatibility-check',
+                '--from='.$this->versions[0],
+                '--to='.$this->versions[3],
                 '-vvv',
             ],
             $this->sourcesRepository
@@ -219,8 +219,8 @@ 
 
         $check = new Process(
             [
-                __DIR__ . '/../../../bin/roave-backward-compatibility-check',
-                '--to=' . $this->versions[2],
+                __DIR__.'/../../../bin/roave-backward-compatibility-check',
+                '--to='.$this->versions[2],
             ],
             $this->sourcesRepository
         );
@@ -248,8 +248,8 @@ 
 
         $check = new Process(
             [
-                __DIR__ . '/../../../bin/roave-backward-compatibility-check',
-                '--to=' . $this->versions[2],
+                __DIR__.'/../../../bin/roave-backward-compatibility-check',
+                '--to='.$this->versions[2],
             ],
             $this->sourcesRepository
         );
@@ -287,7 +287,7 @@ 
                 'tag',
                 $tagName,
                 '-m',
-                'A tag for version ' . $version,
+                'A tag for version '.$version,
             ],
             $this->sourcesRepository
         ))->mustRun();