Roave / BackwardCompatibilityCheck

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

src/DetectChanges/Variance/TypeIsContravariant.php

Spacing +2 added lines, -2 removed lines

@@ -29,7 +29,7 @@ 
             return false;
         }
 
-        if ($type->allowsNull() && ! $comparedType->allowsNull()) {
+        if ($type->allowsNull() && !$comparedType->allowsNull()) {
             return false;
         }
 
@@ -45,7 +45,7 @@ 
             return true;
         }
 
-        if ($comparedTypeAsString === 'object' && ! $type->isBuiltin()) {
+        if ($comparedTypeAsString === 'object' && !$type->isBuiltin()) {
             // `object` is always contravariant to any object type
             return true;
         }

src/DetectChanges/BCBreak/FunctionBased/ParameterDefaultValueChanged.php

Spacing +1 added lines, -1 removed lines

@@ -66,7 +66,7 @@
     {
         return array_filter(
             $function->getParameters(),
-            static function (ReflectionParameter $parameter) : bool {
+            static function(ReflectionParameter $parameter) : bool {
                 return $parameter->isDefaultValueAvailable();
             }
         );

test/unit/Git/RevisionTest.php

Spacing +3 added lines, -3 removed lines

@@ -27,7 +27,7 @@ 
     {
         $sha1 = sha1(uniqid('sha1', true));
 
-        self::assertSame($sha1, (string) Revision::fromSha1($sha1 . "\n"));
+        self::assertSame($sha1, (string) Revision::fromSha1($sha1."\n"));
     }
 
     /**
@@ -40,8 +40,8 @@ 
             ['a'],
             [str_repeat('a', 39)],
             [str_repeat('a', 41)],
-            [' ' . str_repeat('a', 42)],
-            [str_repeat('a', 42) . ' '],
+            [' '.str_repeat('a', 42)],
+            [str_repeat('a', 42).' '],
         ];
     }
 

test/unit/SourceLocator/StaticClassMapSourceLocatorTest.php

Spacing +2 added lines, -2 removed lines

@@ -77,7 +77,7 @@ 
             ->astLocator
             ->expects(self::once())
             ->method('findReflection')
-            ->with($this->reflector, self::callback(static function (LocatedSource $source) : bool {
+            ->with($this->reflector, self::callback(static function(LocatedSource $source) : bool {
                 self::assertSame(file_get_contents(__FILE__), $source->getSource());
                 self::assertSame(__FILE__, $source->getFileName());
                 self::assertNull($source->getExtensionName());
@@ -97,7 +97,7 @@ 
     {
         return [
             [__FILE__],
-            [__DIR__ . '/../SourceLocator/StaticClassMapSourceLocatorTest.php'],
+            [__DIR__.'/../SourceLocator/StaticClassMapSourceLocatorTest.php'],
         ];
     }
 

test/unit/DetectChanges/BCBreak/TraitBased/ExcludeInternalTraitTest.php

Indentation +3 added lines, -3 removed lines

@@ -32,9 +32,9 @@
 
         $check = $this->createMock(TraitBased::class);
         $check->expects(self::once())
-              ->method('__invoke')
-              ->with($reflection, $reflection)
-              ->willReturn(Changes::fromList(Change::removed('foo', true)));
+                ->method('__invoke')
+                ->with($reflection, $reflection)
+                ->willReturn(Changes::fromList(Change::removed('foo', true)));
 
         self::assertEquals(
             Changes::fromList(Change::removed('foo', true)),

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

Spacing +2 added lines, -2 removed lines

@@ -31,7 +31,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))
         );
@@ -53,7 +53,7 @@ 
             ,
             $locator
         ));
-        $toReflector   = new ClassReflector(new StringSourceLocator(
+        $toReflector = new ClassReflector(new StringSourceLocator(
             <<<'PHP'
 <?php
 

test/unit/DetectChanges/BCBreak/InterfaceBased/ExcludeInternalInterfaceTest.php

Indentation +3 added lines, -3 removed lines

@@ -32,9 +32,9 @@
 
         $check = $this->createMock(InterfaceBased::class);
         $check->expects(self::once())
-              ->method('__invoke')
-              ->with($reflection, $reflection)
-              ->willReturn(Changes::fromList(Change::removed('foo', true)));
+                ->method('__invoke')
+                ->with($reflection, $reflection)
+                ->willReturn(Changes::fromList(Change::removed('foo', true)));
 
         self::assertEquals(
             Changes::fromList(Change::removed('foo', true)),

test/unit/DetectChanges/BCBreak/FunctionBased/ExcludeInternalFunctionTest.php

Indentation +4 added lines, -4 removed lines

@@ -33,9 +33,9 @@ 
 
         $check = $this->createMock(FunctionBased::class);
         $check->expects(self::once())
-              ->method('__invoke')
-              ->with($function, $function)
-              ->willReturn(Changes::fromList(Change::removed('foo', true)));
+                ->method('__invoke')
+                ->with($function, $function)
+                ->willReturn(Changes::fromList(Change::removed('foo', true)));
 
         self::assertEquals(
             Changes::fromList(Change::removed('foo', true)),
@@ -61,7 +61,7 @@ 
 
         $check = $this->createMock(FunctionBased::class);
         $check->expects(self::never())
-              ->method('__invoke');
+                ->method('__invoke');
 
         self::assertEquals(
             Changes::empty(),

Spacing +2 added lines, -2 removed lines

@@ -19,7 +19,7 @@ 
 {
     public function testNormalFunctionsAreNotExcluded() : void
     {
-        $source   = new StringSourceLocator(
+        $source = new StringSourceLocator(
             <<<'PHP'
 <?php
 
@@ -46,7 +46,7 @@ 
 
     public function testInternalFunctionsAreExcluded() : void
     {
-        $source   = new StringSourceLocator(
+        $source = new StringSourceLocator(
             <<<'PHP'
 <?php