@@ -22,7 +22,7 @@ |
||
| 22 | 22 | { |
| 23 | 23 | return array_reduce( |
| 24 | 24 | $this->checks, |
| 25 | - function (Changes $changes, PropertyBased $check) use ($fromProperty, $toProperty) : Changes { |
|
| 25 | + function(Changes $changes, PropertyBased $check) use ($fromProperty, $toProperty) : Changes { |
|
| 26 | 26 | return $changes->mergeWith($check->__invoke($fromProperty, $toProperty)); |
| 27 | 27 | }, |
| 28 | 28 | Changes::empty() |
@@ -30,7 +30,7 @@ discard block |
||
| 30 | 30 | return false; |
| 31 | 31 | } |
| 32 | 32 | |
| 33 | - if ($comparedType->allowsNull() && ! $type->allowsNull()) { |
|
| 33 | + if ($comparedType->allowsNull() && !$type->allowsNull()) { |
|
| 34 | 34 | return false; |
| 35 | 35 | } |
| 36 | 36 | |
@@ -46,7 +46,7 @@ discard block |
||
| 46 | 46 | return false; |
| 47 | 47 | } |
| 48 | 48 | |
| 49 | - if (strtolower($typeAsString) === 'object' && ! $comparedType->isBuiltin()) { |
|
| 49 | + if (strtolower($typeAsString) === 'object' && !$comparedType->isBuiltin()) { |
|
| 50 | 50 | // `object` is not covariant to a defined class type |
| 51 | 51 | return true; |
| 52 | 52 | } |
@@ -56,7 +56,7 @@ discard block |
||
| 56 | 56 | return true; |
| 57 | 57 | } |
| 58 | 58 | |
| 59 | - if (strtolower($typeAsString) === 'iterable' && ! $comparedType->isBuiltin()) { |
|
| 59 | + if (strtolower($typeAsString) === 'iterable' && !$comparedType->isBuiltin()) { |
|
| 60 | 60 | if ($comparedType->targetReflectionClass()->implementsInterface(\Traversable::class)) { |
| 61 | 61 | // `iterable` can be restricted via any `Iterator` implementation |
| 62 | 62 | return true; |
@@ -29,7 +29,7 @@ discard block |
||
| 29 | 29 | return false; |
| 30 | 30 | } |
| 31 | 31 | |
| 32 | - if ($type->allowsNull() && ! $comparedType->allowsNull()) { |
|
| 32 | + if ($type->allowsNull() && !$comparedType->allowsNull()) { |
|
| 33 | 33 | return false; |
| 34 | 34 | } |
| 35 | 35 | |
@@ -45,7 +45,7 @@ discard block |
||
| 45 | 45 | return true; |
| 46 | 46 | } |
| 47 | 47 | |
| 48 | - if (strtolower($comparedTypeAsString) === 'object' && ! $type->isBuiltin()) { |
|
| 48 | + if (strtolower($comparedTypeAsString) === 'object' && !$type->isBuiltin()) { |
|
| 49 | 49 | // `object` is always contravariant to any object type |
| 50 | 50 | return true; |
| 51 | 51 | } |
@@ -44,7 +44,7 @@ |
||
| 44 | 44 | |
| 45 | 45 | public function mergeWith(self $other) : self |
| 46 | 46 | { |
| 47 | - if (! $other->changes) { |
|
| 47 | + if (!$other->changes) { |
|
| 48 | 48 | return $this; |
| 49 | 49 | } |
| 50 | 50 | |
@@ -26,10 +26,10 @@ discard block |
||
| 26 | 26 | |
| 27 | 27 | public function setUp() : void |
| 28 | 28 | { |
| 29 | - $tmpGitRepo = sys_get_temp_dir() . '/api-compare-' . uniqid('tmpGitRepo', true); |
|
| 29 | + $tmpGitRepo = sys_get_temp_dir().'/api-compare-'.uniqid('tmpGitRepo', true); |
|
| 30 | 30 | mkdir($tmpGitRepo, 0777, true); |
| 31 | 31 | (new Process(['git', 'init']))->setWorkingDirectory($tmpGitRepo)->mustRun(); |
| 32 | - file_put_contents($tmpGitRepo . '/test', uniqid('testContent', true)); |
|
| 32 | + file_put_contents($tmpGitRepo.'/test', uniqid('testContent', true)); |
|
| 33 | 33 | (new Process(['git', 'add', '.']))->setWorkingDirectory($tmpGitRepo)->mustRun(); |
| 34 | 34 | (new Process(['git', 'commit', '-m', '"whatever"']))->setWorkingDirectory($tmpGitRepo)->mustRun(); |
| 35 | 35 | $this->repoPath = CheckedOutRepository::fromPath($tmpGitRepo); |
@@ -49,7 +49,7 @@ discard block |
||
| 49 | 49 | private function getTags() : array |
| 50 | 50 | { |
| 51 | 51 | return array_map( |
| 52 | - function (Version $version) { |
|
| 52 | + function(Version $version) { |
|
| 53 | 53 | return $version->getVersionString(); |
| 54 | 54 | }, |
| 55 | 55 | iterator_to_array((new GetVersionCollectionFromGitRepository())->fromRepository($this->repoPath)) |
@@ -22,7 +22,7 @@ |
||
| 22 | 22 | { |
| 23 | 23 | return array_reduce( |
| 24 | 24 | $this->checks, |
| 25 | - function (Changes $changes, FunctionBased $check) use ($fromFunction, $toFunction) : Changes { |
|
| 25 | + function(Changes $changes, FunctionBased $check) use ($fromFunction, $toFunction) : Changes { |
|
| 26 | 26 | return $changes->mergeWith($check->__invoke($fromFunction, $toFunction)); |
| 27 | 27 | }, |
| 28 | 28 | Changes::empty() |
@@ -37,7 +37,7 @@ |
||
| 37 | 37 | self::assertSame( |
| 38 | 38 | $expectedVersion, |
| 39 | 39 | (new PickLastMinorVersionFromCollection())->forVersions( |
| 40 | - new VersionsCollection(...array_map(function (string $version) : Version { |
|
| 40 | + new VersionsCollection(...array_map(function(string $version) : Version { |
|
| 41 | 41 | return Version::fromString($version); |
| 42 | 42 | }, $collectionOfVersions)) |
| 43 | 43 | )->getVersionString() |
@@ -31,7 +31,7 @@ |
||
| 31 | 31 | ->getOutput(); |
| 32 | 32 | |
| 33 | 33 | return new VersionsCollection(...array_filter( |
| 34 | - array_map(function (string $maybeVersion) : ?Version { |
|
| 34 | + array_map(function(string $maybeVersion) : ?Version { |
|
| 35 | 35 | try { |
| 36 | 36 | return Version::fromString($maybeVersion); |
| 37 | 37 | } catch (InvalidVersionStringException $e) { |
@@ -46,13 +46,13 @@ |
||
| 46 | 46 | $changelog = Changes::empty(); |
| 47 | 47 | |
| 48 | 48 | $definedApiClassNames = array_map( |
| 49 | - function (ReflectionClass $class) : string { |
|
| 49 | + function(ReflectionClass $class) : string { |
|
| 50 | 50 | return $class->getName(); |
| 51 | 51 | }, |
| 52 | 52 | array_filter( |
| 53 | 53 | $definedSymbols->getAllClasses(), |
| 54 | - function (ReflectionClass $class) : bool { |
|
| 55 | - return ! $class->isAnonymous(); |
|
| 54 | + function(ReflectionClass $class) : bool { |
|
| 55 | + return !$class->isAnonymous(); |
|
| 56 | 56 | } |
| 57 | 57 | ) |
| 58 | 58 | ); |