@@ -113,15 +113,15 @@ |
||
| 113 | 113 | $toPath = $this->git->checkout($sourceRepo, $toRevision); |
| 114 | 114 | |
| 115 | 115 | try { |
| 116 | - $fromSources = $fromPath . '/' . $sourcesPath; |
|
| 117 | - $toSources = $toPath . '/' . $sourcesPath; |
|
| 116 | + $fromSources = $fromPath.'/'.$sourcesPath; |
|
| 117 | + $toSources = $toPath.'/'.$sourcesPath; |
|
| 118 | 118 | |
| 119 | 119 | Assert::that($fromSources)->directory(); |
| 120 | 120 | Assert::that($toSources)->directory(); |
| 121 | 121 | |
| 122 | 122 | $changes = $this->comparator->compare( |
| 123 | - $this->reflectorFactory->__invoke((string) $fromPath . '/' . $sourcesPath), |
|
| 124 | - $this->reflectorFactory->__invoke((string) $toPath . '/' . $sourcesPath) |
|
| 123 | + $this->reflectorFactory->__invoke((string) $fromPath.'/'.$sourcesPath), |
|
| 124 | + $this->reflectorFactory->__invoke((string) $toPath.'/'.$sourcesPath) |
|
| 125 | 125 | ); |
| 126 | 126 | |
| 127 | 127 | (new SymfonyConsoleTextFormatter($stdErr))->write($changes); |
@@ -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 | } |
@@ -19,7 +19,7 @@ |
||
| 19 | 19 | |
| 20 | 20 | public function compare(ReflectionClassConstant $fromConstant, ReflectionClassConstant $toConstant) : Changes |
| 21 | 21 | { |
| 22 | - if (! $fromConstant->isProtected()) { |
|
| 22 | + if (!$fromConstant->isProtected()) { |
|
| 23 | 23 | return Changes::new(); |
| 24 | 24 | } |
| 25 | 25 | |
@@ -19,7 +19,7 @@ |
||
| 19 | 19 | |
| 20 | 20 | public function compare(ReflectionClassConstant $fromConstant, ReflectionClassConstant $toConstant) : Changes |
| 21 | 21 | { |
| 22 | - if (! $fromConstant->isPublic()) { |
|
| 22 | + if (!$fromConstant->isPublic()) { |
|
| 23 | 23 | return Changes::new(); |
| 24 | 24 | } |
| 25 | 25 | |
@@ -22,7 +22,7 @@ |
||
| 22 | 22 | { |
| 23 | 23 | return array_reduce( |
| 24 | 24 | $this->checks, |
| 25 | - function (Changes $changes, ClassConstantBased $check) use ($fromConstant, $toConstant) : Changes { |
|
| 25 | + function(Changes $changes, ClassConstantBased $check) use ($fromConstant, $toConstant) : Changes { |
|
| 26 | 26 | return $changes->mergeWith($check->compare($fromConstant, $toConstant)); |
| 27 | 27 | }, |
| 28 | 28 | Changes::new() |
@@ -74,7 +74,7 @@ |
||
| 74 | 74 | |
| 75 | 75 | private function typeToString(?ReflectionType $type) : string |
| 76 | 76 | { |
| 77 | - if (! $type) { |
|
| 77 | + if (!$type) { |
|
| 78 | 78 | return 'no type'; |
| 79 | 79 | } |
| 80 | 80 | |
@@ -71,13 +71,13 @@ |
||
| 71 | 71 | { |
| 72 | 72 | $optionalParameters = array_values(array_filter( |
| 73 | 73 | $function->getParameters(), |
| 74 | - function (ReflectionParameter $parameter) : bool { |
|
| 74 | + function(ReflectionParameter $parameter) : bool { |
|
| 75 | 75 | return $parameter->isDefaultValueAvailable(); |
| 76 | 76 | } |
| 77 | 77 | )); |
| 78 | 78 | |
| 79 | 79 | return array_combine( |
| 80 | - array_map(function (ReflectionParameter $parameter) : int { |
|
| 80 | + array_map(function(ReflectionParameter $parameter) : int { |
|
| 81 | 81 | return $parameter->getPosition(); |
| 82 | 82 | }, $optionalParameters), |
| 83 | 83 | $optionalParameters |
@@ -71,7 +71,7 @@ |
||
| 71 | 71 | |
| 72 | 72 | private function typeToString(?ReflectionType $type) : string |
| 73 | 73 | { |
| 74 | - if (! $type) { |
|
| 74 | + if (!$type) { |
|
| 75 | 75 | return 'no type'; |
| 76 | 76 | } |
| 77 | 77 | |