Completed
Pull Request — master (#30)
by Marco
01:47
created
test/asset/api/old/ClassWithMethodsBeingRemoved.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/ClassWithPropertyVisibilitiesBeingChanged.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/ClassWithMethodVisibilitiesBeingChanged.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.
unit/Comparator/BackwardsCompatibility/ClassBased/MethodRemovedTest.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\\ClassWithMethodsBeingRemoved' => [
45 45
                 (new ClassReflector(new SingleFileSourceLocator(
46
-                    __DIR__ . '/../../../../asset/api/old/ClassWithMethodsBeingRemoved.php',
46
+                    __DIR__.'/../../../../asset/api/old/ClassWithMethodsBeingRemoved.php',
47 47
                     $locator
48 48
                 )))->reflect('RoaveTestAsset\\ClassWithMethodsBeingRemoved'),
49 49
                 (new ClassReflector(new SingleFileSourceLocator(
50
-                    __DIR__ . '/../../../../asset/api/new/ClassWithMethodsBeingRemoved.php',
50
+                    __DIR__.'/../../../../asset/api/new/ClassWithMethodsBeingRemoved.php',
51 51
                     $locator
52 52
                 )))->reflect('RoaveTestAsset\\ClassWithMethodsBeingRemoved'),
53 53
                 [
Please login to merge, or discard this patch.
BackwardsCompatibility/ClassBased/PropertyVisibilityReducedTest.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\\ClassWithPropertyVisibilitiesBeingChanged' => [
45 45
                 (new ClassReflector(new SingleFileSourceLocator(
46
-                    __DIR__ . '/../../../../asset/api/old/ClassWithPropertyVisibilitiesBeingChanged.php',
46
+                    __DIR__.'/../../../../asset/api/old/ClassWithPropertyVisibilitiesBeingChanged.php',
47 47
                     $locator
48 48
                 )))->reflect('RoaveTestAsset\\ClassWithPropertyVisibilitiesBeingChanged'),
49 49
                 (new ClassReflector(new SingleFileSourceLocator(
50
-                    __DIR__ . '/../../../../asset/api/new/ClassWithPropertyVisibilitiesBeingChanged.php',
50
+                    __DIR__.'/../../../../asset/api/new/ClassWithPropertyVisibilitiesBeingChanged.php',
51 51
                     $locator
52 52
                 )))->reflect('RoaveTestAsset\\ClassWithPropertyVisibilitiesBeingChanged'),
53 53
                 [
Please login to merge, or discard this patch.
unit/Comparator/BackwardsCompatibility/ClassBased/MultiClassBasedTest.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -1,6 +1,6 @@
 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
 
Please login to merge, or discard this patch.
BackwardsCompatibility/ClassBased/MethodVisibilityReducedTest.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
 
@@ -30,7 +30,7 @@  discard block
 block discarded – undo
30 30
 
31 31
         self::assertSame(
32 32
             $expectedMessages,
33
-            array_map(function (Change $change) : string {
33
+            array_map(function(Change $change) : string {
34 34
                 return $change->__toString();
35 35
             }, iterator_to_array($changes))
36 36
         );
@@ -44,11 +44,11 @@  discard block
 block discarded – undo
44 44
         return [
45 45
             'RoaveTestAsset\\ClassWithPropertyVisibilitiesBeingChanged' => [
46 46
                 (new ClassReflector(new SingleFileSourceLocator(
47
-                    __DIR__ . '/../../../../asset/api/old/ClassWithMethodVisibilitiesBeingChanged.php',
47
+                    __DIR__.'/../../../../asset/api/old/ClassWithMethodVisibilitiesBeingChanged.php',
48 48
                     $locator
49 49
                 )))->reflect('RoaveTestAsset\\ClassWithMethodVisibilitiesBeingChanged'),
50 50
                 (new ClassReflector(new SingleFileSourceLocator(
51
-                    __DIR__ . '/../../../../asset/api/new/ClassWithMethodVisibilitiesBeingChanged.php',
51
+                    __DIR__.'/../../../../asset/api/new/ClassWithMethodVisibilitiesBeingChanged.php',
52 52
                     $locator
53 53
                 )))->reflect('RoaveTestAsset\\ClassWithMethodVisibilitiesBeingChanged'),
54 54
                 [
Please login to merge, or discard this patch.
src/Command/ApiCompare.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 Roave\ApiCompare\Command;
5 5
 
@@ -113,14 +113,14 @@  discard block
 block discarded – undo
113 113
         $toRevision = $this->parseRevision->fromStringForRepository($input->getOption('to'), $sourceRepo);
114 114
         $sourcesPath = $input->getArgument('sources-path');
115 115
 
116
-        $output->writeln(sprintf('Comparing from %s to %s...', (string)$fromRevision, (string)$toRevision));
116
+        $output->writeln(sprintf('Comparing from %s to %s...', (string) $fromRevision, (string) $toRevision));
117 117
 
118 118
         $fromPath = $this->git->checkout($sourceRepo, $fromRevision);
119 119
         $toPath = $this->git->checkout($sourceRepo, $toRevision);
120 120
 
121 121
         try {
122
-            $fromSources = $fromPath . '/' . $sourcesPath;
123
-            $toSources   = $toPath . '/' . $sourcesPath;
122
+            $fromSources = $fromPath.'/'.$sourcesPath;
123
+            $toSources   = $toPath.'/'.$sourcesPath;
124 124
 
125 125
             Assert::that($fromSources)->directory();
126 126
             Assert::that($toSources)->directory();
@@ -134,8 +134,8 @@  discard block
 block discarded – undo
134 134
                         new MethodVisibilityReduced()
135 135
                     )
136 136
                 ))->compare(
137
-                    $this->reflectorFactory->__invoke((string)$fromPath . '/' . $sourcesPath),
138
-                    $this->reflectorFactory->__invoke((string)$toPath . '/' . $sourcesPath)
137
+                    $this->reflectorFactory->__invoke((string) $fromPath.'/'.$sourcesPath),
138
+                    $this->reflectorFactory->__invoke((string) $toPath.'/'.$sourcesPath)
139 139
                 )
140 140
             );
141 141
         } finally {
@@ -153,7 +153,7 @@  discard block
 block discarded – undo
153 153
     private function parseRevisionFromInput(InputInterface $input, CheckedOutRepository $repository) : Revision
154 154
     {
155 155
         return $this->parseRevision->fromStringForRepository(
156
-            (string)$input->getOption('from'),
156
+            (string) $input->getOption('from'),
157 157
             $repository
158 158
         );
159 159
     }
Please login to merge, or discard this patch.
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.