@@ -16,7 +16,7 @@ |
||
16 | 16 | */ |
17 | 17 | public function checkout(CheckedOutRepository $sourceRepository, Revision $revision) : CheckedOutRepository |
18 | 18 | { |
19 | - $checkoutDirectory = sys_get_temp_dir() . '/api-compare-' . (string) $revision; |
|
19 | + $checkoutDirectory = sys_get_temp_dir().'/api-compare-'.(string) $revision; |
|
20 | 20 | |
21 | 21 | (new Process(['git', 'clone', (string) $sourceRepository, $checkoutDirectory]))->mustRun(); |
22 | 22 | (new Process(['git', 'checkout', (string) $revision]))->setWorkingDirectory($checkoutDirectory)->mustRun(); |
@@ -31,7 +31,7 @@ |
||
31 | 31 | // @todo handle invalid versions more gracefully (drop them) |
32 | 32 | return VersionsCollection::fromArray(array_filter( |
33 | 33 | explode("\n", $output), |
34 | - function (string $maybeVersion) { |
|
34 | + function(string $maybeVersion) { |
|
35 | 35 | return trim($maybeVersion) !== ''; |
36 | 36 | } |
37 | 37 | )); |
@@ -18,7 +18,7 @@ |
||
18 | 18 | public static function fromPath(string $path) : self |
19 | 19 | { |
20 | 20 | Assert::that($path)->directory(); |
21 | - Assert::that($path . '/.git')->directory(); |
|
21 | + Assert::that($path.'/.git')->directory(); |
|
22 | 22 | $instance = new self(); |
23 | 23 | $instance->path = $path; |
24 | 24 | return $instance; |
@@ -21,7 +21,7 @@ discard block |
||
21 | 21 | $this->accessibleConstants($toClass) |
22 | 22 | ); |
23 | 23 | |
24 | - return Changes::fromArray(array_values(array_map(function (ReflectionClassConstant $constant) use ($fromClass) : Change { |
|
24 | + return Changes::fromArray(array_values(array_map(function(ReflectionClassConstant $constant) use ($fromClass) : Change { |
|
25 | 25 | return Change::removed( |
26 | 26 | sprintf('Constant %s::%s was removed', $fromClass->getName(), $constant->getName()), |
27 | 27 | true |
@@ -32,7 +32,7 @@ discard block |
||
32 | 32 | /** @return ReflectionClassConstant[] */ |
33 | 33 | private function accessibleConstants(ReflectionClass $class) : array |
34 | 34 | { |
35 | - return array_filter($class->getReflectionConstants(), function (ReflectionClassConstant $constant) : bool { |
|
35 | + return array_filter($class->getReflectionConstants(), function(ReflectionClassConstant $constant) : bool { |
|
36 | 36 | return $constant->isPublic() || $constant->isProtected(); |
37 | 37 | }); |
38 | 38 | } |
@@ -34,7 +34,7 @@ |
||
34 | 34 | */ |
35 | 35 | protected function createLocatedSource(Identifier $identifier) : ?LocatedSource |
36 | 36 | { |
37 | - if (! $identifier->isClass()) { |
|
37 | + if (!$identifier->isClass()) { |
|
38 | 38 | return null; |
39 | 39 | } |
40 | 40 |
@@ -17,7 +17,7 @@ discard block |
||
17 | 17 | { |
18 | 18 | $changeText = uniqid('changeText', true); |
19 | 19 | $change = Change::added($changeText, false); |
20 | - self::assertSame(' ADDED: ' . $changeText, (string) $change); |
|
20 | + self::assertSame(' ADDED: '.$changeText, (string) $change); |
|
21 | 21 | self::assertTrue($change->isAdded()); |
22 | 22 | self::assertFalse($change->isChanged()); |
23 | 23 | self::assertFalse($change->isRemoved()); |
@@ -27,7 +27,7 @@ discard block |
||
27 | 27 | { |
28 | 28 | $changeText = uniqid('changeText', true); |
29 | 29 | $change = Change::added($changeText, true); |
30 | - self::assertSame('[BC] ADDED: ' . $changeText, (string) $change); |
|
30 | + self::assertSame('[BC] ADDED: '.$changeText, (string) $change); |
|
31 | 31 | self::assertTrue($change->isAdded()); |
32 | 32 | self::assertFalse($change->isChanged()); |
33 | 33 | self::assertFalse($change->isRemoved()); |
@@ -37,7 +37,7 @@ discard block |
||
37 | 37 | { |
38 | 38 | $changeText = uniqid('changeText', true); |
39 | 39 | $change = Change::changed($changeText, false); |
40 | - self::assertSame(' CHANGED: ' . $changeText, (string) $change); |
|
40 | + self::assertSame(' CHANGED: '.$changeText, (string) $change); |
|
41 | 41 | self::assertFalse($change->isAdded()); |
42 | 42 | self::assertTrue($change->isChanged()); |
43 | 43 | self::assertFalse($change->isRemoved()); |
@@ -47,7 +47,7 @@ discard block |
||
47 | 47 | { |
48 | 48 | $changeText = uniqid('changeText', true); |
49 | 49 | $change = Change::changed($changeText, true); |
50 | - self::assertSame('[BC] CHANGED: ' . $changeText, (string) $change); |
|
50 | + self::assertSame('[BC] CHANGED: '.$changeText, (string) $change); |
|
51 | 51 | self::assertFalse($change->isAdded()); |
52 | 52 | self::assertTrue($change->isChanged()); |
53 | 53 | self::assertFalse($change->isRemoved()); |
@@ -57,7 +57,7 @@ discard block |
||
57 | 57 | { |
58 | 58 | $changeText = uniqid('changeText', true); |
59 | 59 | $change = Change::removed($changeText, false); |
60 | - self::assertSame(' REMOVED: ' . $changeText, (string) $change); |
|
60 | + self::assertSame(' REMOVED: '.$changeText, (string) $change); |
|
61 | 61 | self::assertFalse($change->isAdded()); |
62 | 62 | self::assertFalse($change->isChanged()); |
63 | 63 | self::assertTrue($change->isRemoved()); |
@@ -67,7 +67,7 @@ discard block |
||
67 | 67 | { |
68 | 68 | $changeText = uniqid('changeText', true); |
69 | 69 | $change = Change::removed($changeText, true); |
70 | - self::assertSame('[BC] REMOVED: ' . $changeText, (string) $change); |
|
70 | + self::assertSame('[BC] REMOVED: '.$changeText, (string) $change); |
|
71 | 71 | self::assertFalse($change->isAdded()); |
72 | 72 | self::assertFalse($change->isChanged()); |
73 | 73 | self::assertTrue($change->isRemoved()); |
@@ -235,7 +235,7 @@ discard block |
||
235 | 235 | return array_merge( |
236 | 236 | [[null]], |
237 | 237 | array_merge(...array_map( |
238 | - function (string $type) use ($reflector) : array { |
|
238 | + function(string $type) use ($reflector) : array { |
|
239 | 239 | return [ |
240 | 240 | [ReflectionType::createFromTypeAndReflector($type, false, $reflector)], |
241 | 241 | [ReflectionType::createFromTypeAndReflector($type, true, $reflector)], |
@@ -259,7 +259,7 @@ discard block |
||
259 | 259 | /** @dataProvider existingNullableTypeStrings */ |
260 | 260 | public function testCovarianceConsidersNullability(string $type) : void |
261 | 261 | { |
262 | - $reflector = new ClassReflector(new StringSourceLocator( |
|
262 | + $reflector = new ClassReflector(new StringSourceLocator( |
|
263 | 263 | <<<'PHP' |
264 | 264 | <?php |
265 | 265 |
@@ -224,7 +224,7 @@ discard block |
||
224 | 224 | return array_merge( |
225 | 225 | [[null]], |
226 | 226 | array_merge(...array_map( |
227 | - function (string $type) use ($reflector) : array { |
|
227 | + function(string $type) use ($reflector) : array { |
|
228 | 228 | return [ |
229 | 229 | [ReflectionType::createFromTypeAndReflector($type, false, $reflector)], |
230 | 230 | [ReflectionType::createFromTypeAndReflector($type, true, $reflector)], |
@@ -248,7 +248,7 @@ discard block |
||
248 | 248 | /** @dataProvider existingNullableTypeStrings */ |
249 | 249 | public function testContravarianceConsidersNullability(string $type) : void |
250 | 250 | { |
251 | - $reflector = new ClassReflector(new StringSourceLocator( |
|
251 | + $reflector = new ClassReflector(new StringSourceLocator( |
|
252 | 252 | <<<'PHP' |
253 | 253 | <?php |
254 | 254 |
@@ -34,7 +34,7 @@ discard block |
||
34 | 34 | |
35 | 35 | self::assertSame( |
36 | 36 | $expectedMessages, |
37 | - array_map(function (Change $change) : string { |
|
37 | + array_map(function(Change $change) : string { |
|
38 | 38 | return $change->__toString(); |
39 | 39 | }, iterator_to_array($changes)) |
40 | 40 | ); |
@@ -112,7 +112,7 @@ discard block |
||
112 | 112 | return array_combine( |
113 | 113 | array_keys($properties), |
114 | 114 | array_map( |
115 | - function (string $constant, array $errorMessages) use ($fromClass, $toClass) : array { |
|
115 | + function(string $constant, array $errorMessages) use ($fromClass, $toClass) : array { |
|
116 | 116 | return [ |
117 | 117 | $fromClass->getReflectionConstant($constant), |
118 | 118 | $toClass->getReflectionConstant($constant), |