| @@ -33,7 +33,6 @@ discard block | ||
| 33 | 33 | /** | 
| 34 | 34 | * Constructor. This should not be documented | 
| 35 | 35 | * | 
| 36 | - * @param array|Iterator $files | |
| 37 | 36 | */ | 
| 38 | 37 | public function __construct() | 
| 39 | 38 |      { | 
| @@ -44,7 +43,7 @@ discard block | ||
| 44 | 43 | * | 
| 45 | 44 | * @param string $dst | 
| 46 | 45 | * | 
| 47 | - * @return Concat The current instance | |
| 46 | + * @return TestedRoboTask The current instance | |
| 48 | 47 | */ | 
| 49 | 48 | public function to($dst) | 
| 50 | 49 |      { | 
| @@ -132,6 +132,9 @@ | ||
| 132 | 132 |          $this->assertNotNull($this->app->find('generate:user-avatar')); | 
| 133 | 133 | } | 
| 134 | 134 | |
| 135 | + /** | |
| 136 | + * @param string $name | |
| 137 | + */ | |
| 135 | 138 | protected function createCommand($name) | 
| 136 | 139 |      { | 
| 137 | 140 | $commandInfo = new CommandInfo($this->roboCommandFileInstance, $name); | 
| @@ -20,7 +20,7 @@ | ||
| 20 | 20 |  { | 
| 21 | 21 | use _generated\CodeGuyActions; | 
| 22 | 22 | |
| 23 | - /** | |
| 24 | - * Define custom actions here | |
| 25 | - */ | |
| 23 | + /** | |
| 24 | + * Define custom actions here | |
| 25 | + */ | |
| 26 | 26 | } | 
| @@ -11,10 +11,10 @@ discard block | ||
| 11 | 11 |      { | 
| 12 | 12 |          $I->wantTo('write lines with WriteToFile task'); | 
| 13 | 13 |          $I->taskWriteToFile('blogpost.md') | 
| 14 | -           ->line('****') | |
| 15 | -           ->line('hello world') | |
| 16 | -           ->line('****') | |
| 17 | - ->run(); | |
| 14 | +            ->line('****') | |
| 15 | +            ->line('hello world') | |
| 16 | +            ->line('****') | |
| 17 | + ->run(); | |
| 18 | 18 |          $I->seeFileFound('blogpost.md'); | 
| 19 | 19 | $I->seeFileContentsEqual(<<<HERE | 
| 20 | 20 | **** | 
| @@ -28,9 +28,9 @@ discard block | ||
| 28 | 28 | public function appendToFile(CliGuy $I) | 
| 29 | 29 |      { | 
| 30 | 30 |          $I->taskWriteToFile('a.txt') | 
| 31 | - ->append() | |
| 32 | -           ->line('hello world') | |
| 33 | - ->run(); | |
| 31 | + ->append() | |
| 32 | +            ->line('hello world') | |
| 33 | + ->run(); | |
| 34 | 34 |          $I->seeFileFound('a.txt'); | 
| 35 | 35 | $I->seeFileContentsEqual(<<<HERE | 
| 36 | 36 | Ahello world | 
| @@ -42,7 +42,7 @@ discard block | ||
| 42 | 42 | public function testWouldChange(CliGuy $I) | 
| 43 | 43 |      { | 
| 44 | 44 |          $writeTask = $I->taskWriteToFile('a.txt') | 
| 45 | - ->append(); | |
| 45 | + ->append(); | |
| 46 | 46 | $I->assertEquals(false, $writeTask->wouldChange(), "No changes to test file."); | 
| 47 | 47 |          $writeTask->line('hello world'); | 
| 48 | 48 | $I->assertEquals(true, $writeTask->wouldChange(), "Test file would change."); | 
| @@ -68,15 +68,15 @@ discard block | ||
| 68 | 68 |      { | 
| 69 | 69 |          $I->wantTo('append lines with WriteToFile task, but only if pattern does not match'); | 
| 70 | 70 |          $I->taskWriteToFile('blogpost.md') | 
| 71 | -           ->line('****') | |
| 72 | -           ->line('hello world') | |
| 73 | -           ->line('****') | |
| 74 | -           ->appendUnlessMatches('/hello/', 'Should not add this') | |
| 75 | -           ->appendUnlessMatches('/goodbye/', 'Should add this') | |
| 76 | -           ->appendIfMatches('/hello/', ' and should also add this') | |
| 77 | -           ->appendIfMatches('/goodbye/', ' but should not add this') | |
| 78 | -           ->appendIfMatches('/should/', '!') | |
| 79 | - ->run(); | |
| 71 | +            ->line('****') | |
| 72 | +            ->line('hello world') | |
| 73 | +            ->line('****') | |
| 74 | +            ->appendUnlessMatches('/hello/', 'Should not add this') | |
| 75 | +            ->appendUnlessMatches('/goodbye/', 'Should add this') | |
| 76 | +            ->appendIfMatches('/hello/', ' and should also add this') | |
| 77 | +            ->appendIfMatches('/goodbye/', ' but should not add this') | |
| 78 | +            ->appendIfMatches('/should/', '!') | |
| 79 | + ->run(); | |
| 80 | 80 |          $I->seeFileFound('blogpost.md'); | 
| 81 | 81 | $I->seeFileContentsEqual(<<<HERE | 
| 82 | 82 | **** | 
| @@ -11,8 +11,8 @@ | ||
| 11 | 11 | $initialFileSize = filesize($sampleCss); | 
| 12 | 12 | |
| 13 | 13 | $I->taskMinify($sampleCss) | 
| 14 | -  ->to('minifiedSample.css') | |
| 15 | - ->run(); | |
| 14 | +    ->to('minifiedSample.css') | |
| 15 | + ->run(); | |
| 16 | 16 | |
| 17 | 17 | $I->seeFileFound($outputCss); | 
| 18 | 18 | $minifiedFileSize = filesize($outputCss); | 
| @@ -44,7 +44,7 @@ | ||
| 44 | 44 | * @param string $resource File or folder. | 
| 45 | 45 | * @param string $type "file", "dir", "fileAndDir" | 
| 46 | 46 | * | 
| 47 | - * @return bool | |
| 47 | + * @return boolean|null | |
| 48 | 48 | */ | 
| 49 | 49 | protected function checkResource($resource, $type) | 
| 50 | 50 |      { | 
| @@ -90,25 +90,25 @@ | ||
| 90 | 90 | $this->checkResources($targets); | 
| 91 | 91 | } | 
| 92 | 92 | |
| 93 | - /** | |
| 94 | - * Wrapper method around phps is_dir() | |
| 95 | - * | |
| 96 | - * @param string $directory | |
| 97 | - * | |
| 98 | - * @return bool | |
| 99 | - */ | |
| 93 | + /** | |
| 94 | + * Wrapper method around phps is_dir() | |
| 95 | + * | |
| 96 | + * @param string $directory | |
| 97 | + * | |
| 98 | + * @return bool | |
| 99 | + */ | |
| 100 | 100 | protected function isDir($directory) | 
| 101 | 101 |      { | 
| 102 | 102 | return is_dir($directory); | 
| 103 | 103 | } | 
| 104 | 104 | |
| 105 | - /** | |
| 106 | - * Wrapper method around phps file_exists() | |
| 107 | - * | |
| 108 | - * @param string $file | |
| 109 | - * | |
| 110 | - * @return bool | |
| 111 | - */ | |
| 105 | + /** | |
| 106 | + * Wrapper method around phps file_exists() | |
| 107 | + * | |
| 108 | + * @param string $file | |
| 109 | + * | |
| 110 | + * @return bool | |
| 111 | + */ | |
| 112 | 112 | protected function isFile($file) | 
| 113 | 113 |      { | 
| 114 | 114 | return file_exists($file); | 
| @@ -167,7 +167,7 @@ | ||
| 167 | 167 | } | 
| 168 | 168 | |
| 169 | 169 | /** | 
| 170 | - * @param $inProgress | |
| 170 | + * @param boolean $inProgress | |
| 171 | 171 | */ | 
| 172 | 172 | protected function showTaskProgress($inProgress) | 
| 173 | 173 |      { | 
| @@ -119,7 +119,7 @@ discard block | ||
| 119 | 119 | } | 
| 120 | 120 | |
| 121 | 121 | /** | 
| 122 | - * @param array|string|Traversable $src one or more source values | |
| 122 | + * @param string $src one or more source values | |
| 123 | 123 | * | 
| 124 | 124 | * @return $this | 
| 125 | 125 | */ | 
| @@ -143,7 +143,7 @@ discard block | ||
| 143 | 143 | } | 
| 144 | 144 | |
| 145 | 145 | /** | 
| 146 | - * @param array|string $exts one or more extensions | |
| 146 | + * @param string $exts one or more extensions | |
| 147 | 147 | * | 
| 148 | 148 | * @return $this | 
| 149 | 149 | */ | 
| @@ -154,7 +154,7 @@ discard block | ||
| 154 | 154 | } | 
| 155 | 155 | |
| 156 | 156 | /** | 
| 157 | - * @param array|string $exclude one or more exclusions | |
| 157 | + * @param string[] $exclude one or more exclusions | |
| 158 | 158 | * | 
| 159 | 159 | * @return $this | 
| 160 | 160 | */ | 
| @@ -167,7 +167,7 @@ discard block | ||
| 167 | 167 | } | 
| 168 | 168 | |
| 169 | 169 | /** | 
| 170 | - * @param array|string|Traversable $path one or more skip-doc-path values | |
| 170 | + * @param \SplDoublyLinkedList $path one or more skip-doc-path values | |
| 171 | 171 | * | 
| 172 | 172 | * @return $this | 
| 173 | 173 | */ | 
| @@ -193,7 +193,7 @@ discard block | ||
| 193 | 193 | } | 
| 194 | 194 | |
| 195 | 195 | /** | 
| 196 | - * @param array|string $charset one or more charsets | |
| 196 | + * @param string[] $charset one or more charsets | |
| 197 | 197 | * | 
| 198 | 198 | * @return $this | 
| 199 | 199 | */ | 
| @@ -206,7 +206,7 @@ | ||
| 206 | 206 | /** | 
| 207 | 207 | * @param string $filename | 
| 208 | 208 | * | 
| 209 | - * @return bool|string | |
| 209 | + * @return string|false | |
| 210 | 210 | */ | 
| 211 | 211 | protected static function archiveType($filename) | 
| 212 | 212 |      { |