Completed
Pull Request — master (#38)
by Marco
03:32
created
src/Comparator/BackwardsCompatibility/ClassBased/MultiClassBased.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -1,6 +1,6 @@  discard block
 block discarded – undo
1 1
 <?php
2 2
 
3
-declare(strict_types=1);
3
+declare(strict_types = 1);
4 4
 
5 5
 namespace Roave\ApiCompare\Comparator\BackwardsCompatibility\ClassBased;
6 6
 
@@ -21,7 +21,7 @@  discard block
 block discarded – undo
21 21
     {
22 22
         return array_reduce(
23 23
             $this->checks,
24
-            function (Changes $changes, ClassBased $check) use ($fromClass, $toClass) : Changes {
24
+            function(Changes $changes, ClassBased $check) use ($fromClass, $toClass) : Changes {
25 25
                 return $changes->mergeWith($check->compare($fromClass, $toClass));
26 26
             },
27 27
             Changes::new()
Please login to merge, or discard this patch.
test/unit/ComparatorTest.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -1,5 +1,5 @@
 block discarded – undo
1 1
 <?php
2
-declare(strict_types=1);
2
+declare(strict_types = 1);
3 3
 
4 4
 namespace RoaveTest\ApiCompare;
5 5
 
Please login to merge, or discard this patch.
unit/Comparator/BackwardsCompatibility/ClassBased/ConstantRemovedTest.php 1 patch
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -1,6 +1,6 @@  discard block
 block discarded – undo
1 1
 <?php
2 2
 
3
-declare(strict_types=1);
3
+declare(strict_types = 1);
4 4
 
5 5
 namespace RoaveTest\ApiCompare\Comparator\BackwardsCompatibility\ClassBased;
6 6
 
@@ -29,7 +29,7 @@  discard block
 block discarded – undo
29 29
 
30 30
         self::assertSame(
31 31
             $expectedMessages,
32
-            array_map(function (Change $change) : string {
32
+            array_map(function(Change $change) : string {
33 33
                 return $change->__toString();
34 34
             }, iterator_to_array($changes))
35 35
         );
@@ -43,11 +43,11 @@  discard block
 block discarded – undo
43 43
         return [
44 44
             'RoaveTestAsset\\ClassWithConstantsBeingRemoved' => [
45 45
                 (new ClassReflector(new SingleFileSourceLocator(
46
-                    __DIR__ . '/../../../../asset/api/old/ClassWithConstantsBeingRemoved.php',
46
+                    __DIR__.'/../../../../asset/api/old/ClassWithConstantsBeingRemoved.php',
47 47
                     $locator
48 48
                 )))->reflect('RoaveTestAsset\\ClassWithConstantsBeingRemoved'),
49 49
                 (new ClassReflector(new SingleFileSourceLocator(
50
-                    __DIR__ . '/../../../../asset/api/new/ClassWithConstantsBeingRemoved.php',
50
+                    __DIR__.'/../../../../asset/api/new/ClassWithConstantsBeingRemoved.php',
51 51
                     $locator
52 52
                 )))->reflect('RoaveTestAsset\\ClassWithConstantsBeingRemoved'),
53 53
                 [
Please login to merge, or discard this patch.
test/asset/api/new/ClassWithConstantsBeingRemoved.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -1,5 +1,5 @@
 block discarded – undo
1 1
 <?php
2
-declare(strict_types=1);
2
+declare(strict_types = 1);
3 3
 
4 4
 namespace RoaveTestAsset;
5 5
 
Please login to merge, or discard this patch.
test/asset/api/old/ClassWithConstantsBeingRemoved.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -1,5 +1,5 @@
 block discarded – undo
1 1
 <?php
2
-declare(strict_types=1);
2
+declare(strict_types = 1);
3 3
 
4 4
 namespace RoaveTestAsset;
5 5
 
Please login to merge, or discard this patch.
src/Comparator/BackwardsCompatibility/ClassBased/ConstantRemoved.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -1,6 +1,6 @@  discard block
 block discarded – undo
1 1
 <?php
2 2
 
3
-declare(strict_types=1);
3
+declare(strict_types = 1);
4 4
 
5 5
 namespace Roave\ApiCompare\Comparator\BackwardsCompatibility\ClassBased;
6 6
 
@@ -21,7 +21,7 @@  discard block
 block discarded – undo
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
 block discarded – undo
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
     }
Please login to merge, or discard this patch.
test/unit/ChangeTest.php 1 patch
Spacing   +7 added lines, -7 removed lines patch added patch discarded remove patch
@@ -1,5 +1,5 @@  discard block
 block discarded – undo
1 1
 <?php
2
-declare(strict_types=1);
2
+declare(strict_types = 1);
3 3
 
4 4
 namespace RoaveTest\ApiCompare;
5 5
 
@@ -15,7 +15,7 @@  discard block
 block discarded – undo
15 15
     {
16 16
         $changeText = uniqid('changeText', true);
17 17
         $change = Change::added($changeText, false);
18
-        self::assertSame('     ADDED: ' . $changeText, (string) $change);
18
+        self::assertSame('     ADDED: '.$changeText, (string) $change);
19 19
         self::assertTrue($change->isAdded());
20 20
         self::assertFalse($change->isChanged());
21 21
         self::assertFalse($change->isRemoved());
@@ -25,7 +25,7 @@  discard block
 block discarded – undo
25 25
     {
26 26
         $changeText = uniqid('changeText', true);
27 27
         $change = Change::added($changeText, true);
28
-        self::assertSame('[BC] ADDED: ' . $changeText, (string) $change);
28
+        self::assertSame('[BC] ADDED: '.$changeText, (string) $change);
29 29
         self::assertTrue($change->isAdded());
30 30
         self::assertFalse($change->isChanged());
31 31
         self::assertFalse($change->isRemoved());
@@ -35,7 +35,7 @@  discard block
 block discarded – undo
35 35
     {
36 36
         $changeText = uniqid('changeText', true);
37 37
         $change = Change::changed($changeText, false);
38
-        self::assertSame('     CHANGED: ' . $changeText, (string) $change);
38
+        self::assertSame('     CHANGED: '.$changeText, (string) $change);
39 39
         self::assertFalse($change->isAdded());
40 40
         self::assertTrue($change->isChanged());
41 41
         self::assertFalse($change->isRemoved());
@@ -45,7 +45,7 @@  discard block
 block discarded – undo
45 45
     {
46 46
         $changeText = uniqid('changeText', true);
47 47
         $change = Change::changed($changeText, true);
48
-        self::assertSame('[BC] CHANGED: ' . $changeText, (string) $change);
48
+        self::assertSame('[BC] CHANGED: '.$changeText, (string) $change);
49 49
         self::assertFalse($change->isAdded());
50 50
         self::assertTrue($change->isChanged());
51 51
         self::assertFalse($change->isRemoved());
@@ -55,7 +55,7 @@  discard block
 block discarded – undo
55 55
     {
56 56
         $changeText = uniqid('changeText', true);
57 57
         $change = Change::removed($changeText, false);
58
-        self::assertSame('     REMOVED: ' . $changeText, (string) $change);
58
+        self::assertSame('     REMOVED: '.$changeText, (string) $change);
59 59
         self::assertFalse($change->isAdded());
60 60
         self::assertFalse($change->isChanged());
61 61
         self::assertTrue($change->isRemoved());
@@ -65,7 +65,7 @@  discard block
 block discarded – undo
65 65
     {
66 66
         $changeText = uniqid('changeText', true);
67 67
         $change = Change::removed($changeText, true);
68
-        self::assertSame('[BC] REMOVED: ' . $changeText, (string) $change);
68
+        self::assertSame('[BC] REMOVED: '.$changeText, (string) $change);
69 69
         self::assertFalse($change->isAdded());
70 70
         self::assertFalse($change->isChanged());
71 71
         self::assertTrue($change->isRemoved());
Please login to merge, or discard this patch.
src/Git/CheckedOutRepository.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -1,6 +1,6 @@  discard block
 block discarded – undo
1 1
 <?php
2 2
 
3
-declare(strict_types=1);
3
+declare(strict_types = 1);
4 4
 
5 5
 namespace Roave\ApiCompare\Git;
6 6
 
@@ -18,7 +18,7 @@  discard block
 block discarded – undo
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;
Please login to merge, or discard this patch.
src/Git/GetVersionCollectionFromGitRepository.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -1,6 +1,6 @@  discard block
 block discarded – undo
1 1
 <?php
2 2
 
3
-declare(strict_types=1);
3
+declare(strict_types = 1);
4 4
 
5 5
 namespace Roave\ApiCompare\Git;
6 6
 
@@ -31,7 +31,7 @@  discard block
 block discarded – undo
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
         ));
Please login to merge, or discard this patch.