Roave / BackwardCompatibilityCheck

src/DetectChanges/BCBreak/FunctionBased/ReturnTypeChanged.php

Spacing +1 added lines, -1 removed lines

@@ -74,7 +74,7 @@
 
     private function typeToString(?ReflectionType $type) : string
     {
-        if (! $type) {
+        if (!$type) {
             return 'no type';
         }
 

src/DetectChanges/BCBreak/TraitBased/TraitBecameInterface.php

Spacing +1 added lines, -1 removed lines

@@ -17,7 +17,7 @@
 {
     public function __invoke(ReflectionClass $fromTrait, ReflectionClass $toTrait) : Changes
     {
-        if ($toTrait->isTrait() || ! $toTrait->isInterface() || ! $fromTrait->isTrait()) {
+        if ($toTrait->isTrait() || !$toTrait->isInterface() || !$fromTrait->isTrait()) {
             return Changes::empty();
         }
 

src/DetectChanges/BCBreak/TraitBased/TraitBecameClass.php

Spacing +2 added lines, -2 removed lines

@@ -17,7 +17,7 @@ 
 {
     public function __invoke(ReflectionClass $fromTrait, ReflectionClass $toTrait) : Changes
     {
-        if ($this->isClass($fromTrait) || ! $this->isClass($toTrait)) {
+        if ($this->isClass($fromTrait) || !$this->isClass($toTrait)) {
             return Changes::empty();
         }
 
@@ -32,6 +32,6 @@ 
      */
     private function isClass(ReflectionClass $class) : bool
     {
-        return ! ($class->isTrait() || $class->isInterface());
+        return !($class->isTrait() || $class->isInterface());
     }
 }

src/DetectChanges/BCBreak/PropertyBased/OnlyProtectedPropertyChanged.php

Spacing +1 added lines, -1 removed lines

@@ -19,7 +19,7 @@
 
     public function __invoke(ReflectionProperty $fromProperty, ReflectionProperty $toProperty) : Changes
     {
-        if (! $fromProperty->isProtected()) {
+        if (!$fromProperty->isProtected()) {
             return Changes::empty();
         }
 

src/DetectChanges/BCBreak/PropertyBased/OnlyPublicPropertyChanged.php

Spacing +1 added lines, -1 removed lines

@@ -19,7 +19,7 @@
 
     public function __invoke(ReflectionProperty $fromProperty, ReflectionProperty $toProperty) : Changes
     {
-        if (! $fromProperty->isPublic()) {
+        if (!$fromProperty->isPublic()) {
             return Changes::empty();
         }
 

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 (strtolower($comparedTypeAsString) === 'object' && ! $type->isBuiltin()) {
+        if (strtolower($comparedTypeAsString) === 'object' && !$type->isBuiltin()) {
             // `object` is always contravariant to any object type
             return true;
         }

src/SourceLocator/StubClassSourceLocator.php

Spacing +1 added lines, -1 removed lines

@@ -34,7 +34,7 @@
      */
     protected function createLocatedSource(Identifier $identifier) : ?LocatedSource
     {
-        if (! $identifier->isClass()) {
+        if (!$identifier->isClass()) {
             return null;
         }
 

test/e2e/Command/AssertBackwardsCompatibleTest.php

Indentation +1 added lines, -1 removed lines

@@ -120,7 +120,7 @@
             (new Process('git add -A', $this->sourcesRepository))->mustRun();
             (new Process(sprintf('git commit -am "Class sources v%d"', $key + 1), $this->sourcesRepository))->mustRun();
             $this->versions[$key] = trim((new Process('git rev-parse HEAD', $this->sourcesRepository))->mustRun()
-                                                                                                      ->getOutput());
+                                                                                                        ->getOutput());
         }
     }
 

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();

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());