Roave / BackwardCompatibilityCheck

src/Git/GetVersionCollectionFromGitRepository.php

Spacing +1 added lines, -1 removed lines

@@ -31,7 +31,7 @@
             ->getOutput();
 
         return VersionsCollection::fromArray(array_filter(
-            array_map(function (string $maybeVersion) : ?Version {
+            array_map(function(string $maybeVersion) : ?Version {
                 try {
                     return Version::fromString($maybeVersion);
                 } catch (InvalidVersionStringException $e) {

test/unit/Git/GitCheckoutRevisionToTemporaryPathTest.php

Spacing +4 added lines, -4 removed lines

@@ -23,7 +23,7 @@ 
         $revision = Revision::fromSha1(self::TEST_REVISION_TO_CHECKOUT);
 
         $temporaryClone = $git->checkout(
-            CheckedOutRepository::fromPath(realpath(__DIR__ . '/../../../')),
+            CheckedOutRepository::fromPath(realpath(__DIR__.'/../../../')),
             $revision
         );
 
@@ -35,7 +35,7 @@ 
     public function testCanCheckOutSameRevisionTwice() : void
     {
         $git              = new GitCheckoutRevisionToTemporaryPath();
-        $sourceRepository = CheckedOutRepository::fromPath(realpath(__DIR__ . '/../../../'));
+        $sourceRepository = CheckedOutRepository::fromPath(realpath(__DIR__.'/../../../'));
         $revision         = Revision::fromSha1(self::TEST_REVISION_TO_CHECKOUT);
 
         $first  = $git->checkout($sourceRepository, $revision);
@@ -50,10 +50,10 @@ 
 
     public function testExceptionIsThrownWhenTwoPathsCollide() : void
     {
-        $git              = new GitCheckoutRevisionToTemporaryPath(function () : string {
+        $git              = new GitCheckoutRevisionToTemporaryPath(function() : string {
             return 'foo';
         });
-        $sourceRepository = CheckedOutRepository::fromPath(realpath(__DIR__ . '/../../../'));
+        $sourceRepository = CheckedOutRepository::fromPath(realpath(__DIR__.'/../../../'));
         $revision         = Revision::fromSha1(self::TEST_REVISION_TO_CHECKOUT);
 
         $first = $git->checkout($sourceRepository, $revision);

test/unit/Git/GetVersionCollectionFromGitRepositoryTest.php

Spacing +3 added lines, -3 removed lines

@@ -26,10 +26,10 @@ 
 
     public function setUp() : void
     {
-        $tmpGitRepo = sys_get_temp_dir() . '/api-compare-' . uniqid('tmpGitRepo', true);
+        $tmpGitRepo = sys_get_temp_dir().'/api-compare-'.uniqid('tmpGitRepo', true);
         mkdir($tmpGitRepo, 0777, true);
         (new Process(['git', 'init']))->setWorkingDirectory($tmpGitRepo)->mustRun();
-        file_put_contents($tmpGitRepo . '/test', uniqid('testContent', true));
+        file_put_contents($tmpGitRepo.'/test', uniqid('testContent', true));
         (new Process(['git', 'add', '.']))->setWorkingDirectory($tmpGitRepo)->mustRun();
         (new Process(['git', 'commit', '-m', '"whatever"']))->setWorkingDirectory($tmpGitRepo)->mustRun();
         $this->repoPath = CheckedOutRepository::fromPath($tmpGitRepo);
@@ -49,7 +49,7 @@ 
     private function getTags() : array
     {
         return array_map(
-            function (Version $version) {
+            function(Version $version) {
                 return $version->getVersionString();
             },
             iterator_to_array((new GetVersionCollectionFromGitRepository())->fromRepository($this->repoPath))

src/Git/GitCheckoutRevisionToTemporaryPath.php

Spacing +2 added lines, -2 removed lines

@@ -20,7 +20,7 @@ 
 
     public function __construct(?callable $uniquenessFunction = null)
     {
-        $this->uniquenessFunction = $uniquenessFunction ?? function (string $nonUniqueThing) : string {
+        $this->uniquenessFunction = $uniquenessFunction ?? function(string $nonUniqueThing) : string {
                 return uniqid($nonUniqueThing, true);
         };
     }
@@ -54,7 +54,7 @@ 
     private function generateTemporaryPathFor(Revision $revision) : string
     {
         $uniquePathGenerator = $this->uniquenessFunction;
-        $checkoutDirectory   = sys_get_temp_dir() . '/api-compare-' . $uniquePathGenerator((string) $revision . '_');
+        $checkoutDirectory   = sys_get_temp_dir().'/api-compare-'.$uniquePathGenerator((string) $revision.'_');
 
         if (file_exists($checkoutDirectory) || is_dir($checkoutDirectory)) {
             throw new RuntimeException(sprintf(

src/Command/AssertBackwardsCompatible.php

Spacing +5 added lines, -5 removed lines

@@ -161,23 +161,23 @@
         $toPath   = $this->git->checkout($sourceRepo, $toRevision);
 
         try {
-            $fromSources = $fromPath . '/' . $sourcesPath;
-            $toSources   = $toPath . '/' . $sourcesPath;
+            $fromSources = $fromPath.'/'.$sourcesPath;
+            $toSources   = $toPath.'/'.$sourcesPath;
 
             Assert::that($fromSources)->directory();
             Assert::that($toSources)->directory();
 
             $changes = ($this->compareApi)(
                 ($this->reflectorFactory)(
-                    $fromPath . '/' . $sourcesPath,
+                    $fromPath.'/'.$sourcesPath,
                     new AggregateSourceLocator() // no dependencies
                 ),
                 ($this->reflectorFactory)(
-                    $fromPath . '/' . $sourcesPath,
+                    $fromPath.'/'.$sourcesPath,
                     ($this->locateDependencies)((string) $fromPath)
                 ),
                 ($this->reflectorFactory)(
-                    $toPath . '/' . $sourcesPath,
+                    $toPath.'/'.$sourcesPath,
                     ($this->locateDependencies)((string) $toPath)
                 )
             );

src/DetectChanges/BCBreak/InterfaceBased/MultipleChecksOnAnInterface.php

Spacing +1 added lines, -1 removed lines

@@ -22,7 +22,7 @@
     {
         return array_reduce(
             $this->checks,
-            function (Changes $changes, InterfaceBased $check) use ($fromClass, $toClass) : Changes {
+            function(Changes $changes, InterfaceBased $check) use ($fromClass, $toClass) : Changes {
                 return $changes->mergeWith($check($fromClass, $toClass));
             },
             Changes::empty()

src/DetectChanges/BCBreak/ClassBased/ConstantChanged.php

Spacing +1 added lines, -1 removed lines

@@ -29,7 +29,7 @@
 
         return array_reduce(
             array_keys($commonConstants),
-            function (Changes $accumulator, string $constantName) use ($constantsFrom, $constantsTo) : Changes {
+            function(Changes $accumulator, string $constantName) use ($constantsFrom, $constantsTo) : Changes {
                 return $accumulator->mergeWith(($this->checkConstant)(
                     $constantsFrom[$constantName],
                     $constantsTo[$constantName]

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)($method)),
                 true
@@ -46,12 +46,12 @@ 
     /** @return ReflectionMethod[] */
     private function accessibleMethods(ReflectionClass $class) : array
     {
-        $methods = array_filter($class->getMethods(), function (ReflectionMethod $method) : bool {
+        $methods = array_filter($class->getMethods(), function(ReflectionMethod $method) : bool {
             return $method->isPublic() || $method->isProtected();
         });
 
         return array_combine(
-            array_map(function (ReflectionMethod $method) : string {
+            array_map(function(ReflectionMethod $method) : string {
                 return $method->getName();
             }, $methods),
             $methods

src/DetectChanges/BCBreak/ClassBased/MultipleChecksOnAClass.php

Spacing +1 added lines, -1 removed lines

@@ -22,7 +22,7 @@
     {
         return array_reduce(
             $this->checks,
-            function (Changes $changes, ClassBased $check) use ($fromClass, $toClass) : Changes {
+            function(Changes $changes, ClassBased $check) use ($fromClass, $toClass) : Changes {
                 return $changes->mergeWith($check($fromClass, $toClass));
             },
             Changes::empty()