@@ -19,7 +19,7 @@ |
||
| 19 | 19 | |
| 20 | 20 | public function __invoke(ReflectionProperty $fromProperty, ReflectionProperty $toProperty) : Changes |
| 21 | 21 | { |
| 22 | - if (! $fromProperty->isProtected()) { |
|
| 22 | + if (!$fromProperty->isProtected()) { |
|
| 23 | 23 | return Changes::empty(); |
| 24 | 24 | } |
| 25 | 25 | |
@@ -19,7 +19,7 @@ |
||
| 19 | 19 | |
| 20 | 20 | public function __invoke(ReflectionProperty $fromProperty, ReflectionProperty $toProperty) : Changes |
| 21 | 21 | { |
| 22 | - if (! $fromProperty->isPublic()) { |
|
| 22 | + if (!$fromProperty->isPublic()) { |
|
| 23 | 23 | return Changes::empty(); |
| 24 | 24 | } |
| 25 | 25 | |
@@ -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 | |
@@ -161,23 +161,23 @@ |
||
| 161 | 161 | $toPath = $this->git->checkout($sourceRepo, $toRevision); |
| 162 | 162 | |
| 163 | 163 | try { |
| 164 | - $fromSources = $fromPath . '/' . $sourcesPath; |
|
| 165 | - $toSources = $toPath . '/' . $sourcesPath; |
|
| 164 | + $fromSources = $fromPath.'/'.$sourcesPath; |
|
| 165 | + $toSources = $toPath.'/'.$sourcesPath; |
|
| 166 | 166 | |
| 167 | 167 | Assert::that($fromSources)->directory(); |
| 168 | 168 | Assert::that($toSources)->directory(); |
| 169 | 169 | |
| 170 | 170 | $changes = $this->compareApi->__invoke( |
| 171 | 171 | $this->reflectorFactory->__invoke( |
| 172 | - $fromPath . '/' . $sourcesPath, |
|
| 172 | + $fromPath.'/'.$sourcesPath, |
|
| 173 | 173 | new AggregateSourceLocator() // no dependencies |
| 174 | 174 | ), |
| 175 | 175 | $this->reflectorFactory->__invoke( |
| 176 | - $fromPath . '/' . $sourcesPath, |
|
| 176 | + $fromPath.'/'.$sourcesPath, |
|
| 177 | 177 | $this->locateDependencies->__invoke((string) $fromPath) |
| 178 | 178 | ), |
| 179 | 179 | $this->reflectorFactory->__invoke( |
| 180 | - $toPath . '/' . $sourcesPath, |
|
| 180 | + $toPath.'/'.$sourcesPath, |
|
| 181 | 181 | $this->locateDependencies->__invoke((string) $toPath) |
| 182 | 182 | ) |
| 183 | 183 | ); |
@@ -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)) |
@@ -20,7 +20,7 @@ discard block |
||
| 20 | 20 | |
| 21 | 21 | public function __construct(?callable $uniquenessFunction = null) |
| 22 | 22 | { |
| 23 | - $this->uniquenessFunction = $uniquenessFunction ?? function (string $nonUniqueThing) : string { |
|
| 23 | + $this->uniquenessFunction = $uniquenessFunction ?? function(string $nonUniqueThing) : string { |
|
| 24 | 24 | return uniqid($nonUniqueThing, true); |
| 25 | 25 | }; |
| 26 | 26 | } |
@@ -54,7 +54,7 @@ discard block |
||
| 54 | 54 | private function generateTemporaryPathFor(Revision $revision) : string |
| 55 | 55 | { |
| 56 | 56 | $uniquePathGenerator = $this->uniquenessFunction; |
| 57 | - $checkoutDirectory = sys_get_temp_dir() . '/api-compare-' . $uniquePathGenerator((string) $revision . '_'); |
|
| 57 | + $checkoutDirectory = sys_get_temp_dir().'/api-compare-'.$uniquePathGenerator((string) $revision.'_'); |
|
| 58 | 58 | |
| 59 | 59 | if (file_exists($checkoutDirectory) || is_dir($checkoutDirectory)) { |
| 60 | 60 | throw new RuntimeException(sprintf( |