Completed
Pull Request — master (#823)
by Greg
02:16
created
src/Task/Development/PackPhar.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -129,7 +129,7 @@
 block discarded – undo
129 129
     {
130 130
         // run() will call advanceProgressIndicator() once for each
131 131
         // file, one after calling stopBuffering, and again after compression.
132
-        return count($this->files)+2;
132
+        return count($this->files) + 2;
133 133
     }
134 134
 
135 135
     /**
Please login to merge, or discard this patch.
src/Task/Development/OpenBrowser.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -35,7 +35,7 @@
 block discarded – undo
35 35
      */
36 36
     public function __construct($url)
37 37
     {
38
-        $this->urls = (array) $url;
38
+        $this->urls = (array)$url;
39 39
     }
40 40
 
41 41
     /**
Please login to merge, or discard this patch.
src/Result.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -189,7 +189,7 @@
 block discarded – undo
189 189
      */
190 190
     public function cloneTask()
191 191
     {
192
-        $reflect  = new \ReflectionClass(get_class($this->task));
192
+        $reflect = new \ReflectionClass(get_class($this->task));
193 193
         return $reflect->newInstanceArgs(func_get_args());
194 194
     }
195 195
 
Please login to merge, or discard this patch.
src/Common/ExecTrait.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -247,7 +247,7 @@
 block discarded – undo
247 247
         $this->process = $process;
248 248
 
249 249
         if (!$output_callback) {
250
-            $output_callback = function ($type, $buffer) {
250
+            $output_callback = function($type, $buffer) {
251 251
                 $progressWasVisible = $this->hideTaskProgress();
252 252
                 $this->writeMessage($buffer);
253 253
                 $this->showTaskProgress($progressWasVisible);
Please login to merge, or discard this patch.
src/Task/ApiGen/ApiGen.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -71,7 +71,7 @@
 block discarded – undo
71 71
         if (!is_array($args)) {
72 72
             $args = func_get_args();
73 73
         }
74
-        $args = array_map(function ($arg) {
74
+        $args = array_map(function($arg) {
75 75
             if (preg_match('/^\w+$/', trim($arg)) === 1) {
76 76
                 $this->operation = $arg;
77 77
                 return null;
Please login to merge, or discard this patch.
tests/unit/Common/ResultDataTest.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -33,8 +33,8 @@
 block discarded – undo
33 33
         $a = new ResultData(ResultData::EXITCODE_OK, '', ['one' => 'first', 'two' => 'second']);
34 34
 
35 35
         $to_be_merged = [
36
-            ['one' => 'ignored',],
37
-            ['three' => 'new',],
36
+            ['one' => 'ignored', ],
37
+            ['three' => 'new', ],
38 38
         ];
39 39
 
40 40
         foreach ($to_be_merged as $mergeThis) {
Please login to merge, or discard this patch.
src/Collection/Collection.php 1 patch
Spacing   +5 added lines, -5 removed lines patch added patch discarded remove patch
@@ -135,7 +135,7 @@  discard block
 block discarded – undo
135 135
     {
136 136
         $collection = $this;
137 137
         $completionRegistrationTask = new CallableTask(
138
-            function () use ($collection, $completionTask) {
138
+            function() use ($collection, $completionTask) {
139 139
 
140 140
                 $collection->registerCompletion($completionTask);
141 141
             },
@@ -178,7 +178,7 @@  discard block
 block discarded – undo
178 178
         $context += ['name' => 'Progress'];
179 179
         $context += TaskInfo::getTaskContext($this);
180 180
         return $this->addCode(
181
-            function () use ($level, $text, $context) {
181
+            function() use ($level, $text, $context) {
182 182
                 $context += $this->getState()->getData();
183 183
                 $this->printTaskOutput($level, $text, $context);
184 184
             }
@@ -194,7 +194,7 @@  discard block
 block discarded – undo
194 194
     {
195 195
         $collection = $this;
196 196
         $rollbackRegistrationTask = new CallableTask(
197
-            function () use ($collection, $rollbackTask) {
197
+            function() use ($collection, $rollbackTask) {
198 198
                 $collection->registerRollback($rollbackTask);
199 199
             },
200 200
             $this
@@ -247,7 +247,7 @@  discard block
 block discarded – undo
247 247
         if ($task instanceof StackBasedTask) {
248 248
             $task->stopOnFail(false);
249 249
         }
250
-        $ignoreErrorsInTask = function () use ($task) {
250
+        $ignoreErrorsInTask = function() use ($task) {
251 251
             $data = [];
252 252
             try {
253 253
                 $result = $this->runSubtask($task);
@@ -679,7 +679,7 @@  discard block
 block discarded – undo
679 679
     {
680 680
         return $this->defer(
681 681
             $task,
682
-            function ($task, $state) use ($functionName, $stateKey) {
682
+            function($task, $state) use ($functionName, $stateKey) {
683 683
                 $fn = [$task, $functionName];
684 684
                 $value = $state[$stateKey];
685 685
                 $fn($value);
Please login to merge, or discard this patch.
tests/unit/Task/CollectionTest.php 1 patch
Spacing   +8 added lines, -8 removed lines patch added patch discarded remove patch
@@ -110,11 +110,11 @@  discard block
 block discarded – undo
110 110
             ->completion($completion1)
111 111
             ->rollbackCode(function() use($rollback1) { $rollback1->run(); } )
112 112
             ->completionCode(function() use($completion1) { $completion1->run(); } )
113
-            ->addCode(function () { return 42; })
113
+            ->addCode(function() { return 42; })
114 114
             ->progressMessage("not reached")
115 115
             ->rollback($rollback2)
116 116
             ->completion($completion2)
117
-            ->addCode(function () { return 13; });
117
+            ->addCode(function() { return 13; });
118 118
 
119 119
         $collection->setLogger($this->guy->logger());
120 120
 
@@ -137,15 +137,15 @@  discard block
 block discarded – undo
137 137
 
138 138
         $result = $collection
139 139
             ->addCode(
140
-                function (Data $state) {
140
+                function(Data $state) {
141 141
                     $state['one'] = 'first';
142 142
                 })
143 143
             ->addCode(
144
-                function (Data $state) {
144
+                function(Data $state) {
145 145
                     $state['two'] = 'second';
146 146
                 })
147 147
             ->addCode(
148
-                function (Data $state) {
148
+                function(Data $state) {
149 149
                     $state['three'] = "{$state['one']} and {$state['two']}";
150 150
                 })
151 151
             ->run();
@@ -169,7 +169,7 @@  discard block
 block discarded – undo
169 169
             ->add($first)
170 170
             ->add($second)
171 171
             ->addCode(
172
-                function (Data $state) {
172
+                function(Data $state) {
173 173
                     $state['three'] = "{$state['one']} and {$state['two']}";
174 174
                 })
175 175
             ->run();
@@ -198,7 +198,7 @@  discard block
 block discarded – undo
198 198
             ->add($third)
199 199
                 ->defer(
200 200
                     $third,
201
-                    function ($task, $state) {
201
+                    function($task, $state) {
202 202
                         $task->provideData('three', "{$state['one']} and {$state['two']}");
203 203
                     }
204 204
                 )
@@ -230,7 +230,7 @@  discard block
 block discarded – undo
230 230
             ->add($third)
231 231
                 ->defer(
232 232
                     $third,
233
-                    function ($task, $state) {
233
+                    function($task, $state) {
234 234
                         $task->provideData('three', "{$state['one']} and {$state['two']}");
235 235
                     }
236 236
                 )
Please login to merge, or discard this patch.
tests/unit/Task/GulpTest.php 1 patch
Spacing   +9 added lines, -9 removed lines patch added patch discarded remove patch
@@ -23,27 +23,27 @@  discard block
 block discarded – undo
23 23
     public function testGulpGetCommand()
24 24
     {
25 25
         verify(
26
-            (new \Robo\Task\Gulp\Run('default','gulp'))->getCommand()
26
+            (new \Robo\Task\Gulp\Run('default', 'gulp'))->getCommand()
27 27
         )->equals($this->adjustQuotes("gulp 'default'"));
28 28
 
29 29
         verify(
30
-            (new \Robo\Task\Gulp\Run('another','gulp'))->getCommand()
30
+            (new \Robo\Task\Gulp\Run('another', 'gulp'))->getCommand()
31 31
         )->equals($this->adjustQuotes("gulp 'another'"));
32 32
 
33 33
         verify(
34
-            (new \Robo\Task\Gulp\Run('default','gulp'))->silent()->getCommand()
34
+            (new \Robo\Task\Gulp\Run('default', 'gulp'))->silent()->getCommand()
35 35
         )->equals($this->adjustQuotes("gulp 'default' --silent"));
36 36
 
37 37
         verify(
38
-            (new \Robo\Task\Gulp\Run('default','gulp'))->noColor()->getCommand()
38
+            (new \Robo\Task\Gulp\Run('default', 'gulp'))->noColor()->getCommand()
39 39
         )->equals($this->adjustQuotes("gulp 'default' --no-color"));
40 40
 
41 41
         verify(
42
-            (new \Robo\Task\Gulp\Run('default','gulp'))->color()->getCommand()
42
+            (new \Robo\Task\Gulp\Run('default', 'gulp'))->color()->getCommand()
43 43
         )->equals($this->adjustQuotes("gulp 'default' --color"));
44 44
 
45 45
         verify(
46
-            (new \Robo\Task\Gulp\Run('default','gulp'))->simple()->getCommand()
46
+            (new \Robo\Task\Gulp\Run('default', 'gulp'))->simple()->getCommand()
47 47
         )->equals($this->adjustQuotes("gulp 'default' --tasks-simple"));
48 48
     }
49 49
 
@@ -51,7 +51,7 @@  discard block
 block discarded – undo
51 51
     {
52 52
         $gulp = test::double('Robo\Task\Gulp\Run', ['executeCommand' => null, 'getConfig' => new \Robo\Config(), 'logger' => new \Psr\Log\NullLogger()]);
53 53
 
54
-        $task = (new \Robo\Task\Gulp\Run('default','gulp'))->simple();
54
+        $task = (new \Robo\Task\Gulp\Run('default', 'gulp'))->simple();
55 55
         verify($task->getCommand())->equals($this->adjustQuotes("gulp 'default' --tasks-simple"));
56 56
         $task->run();
57 57
         $gulp->verifyInvoked('executeCommand', [$this->adjustQuotes("gulp 'default' --tasks-simple")]);
@@ -64,13 +64,13 @@  discard block
 block discarded – undo
64 64
         if ($isWindows) {
65 65
 
66 66
             verify(
67
-                (new \Robo\Task\Gulp\Run('anotherWith weired!("\') Chars','gulp'))->getCommand()
67
+                (new \Robo\Task\Gulp\Run('anotherWith weired!("\') Chars', 'gulp'))->getCommand()
68 68
             )->equals('gulp "anotherWith weired!(\"\') Chars"');
69 69
 
70 70
         } else {
71 71
 
72 72
             verify(
73
-                (new \Robo\Task\Gulp\Run('anotherWith weired!("\') Chars','gulp'))->getCommand()
73
+                (new \Robo\Task\Gulp\Run('anotherWith weired!("\') Chars', 'gulp'))->getCommand()
74 74
             )->equals("gulp 'anotherWith weired!(\"'\\'') Chars'");
75 75
 
76 76
         }
Please login to merge, or discard this patch.