- Issue created by @mstrelan
- πΊπΈUnited States smustgrave
So if I got this wrong I apologize, so I this reactor script
<?php declare(strict_types=1); use Rector\Config\RectorConfig; use Rector\TypeDeclaration\Rector\ClassMethod\ReturnTypeFromStrictNewArrayRector; return static function (RectorConfig $rectorConfig): void { $rectorConfig->rule(ReturnTypeFromStrictNewArrayRector::class); // Include only specific file extensions $rectorConfig->fileExtensions(['module', 'inc', 'engine', 'theme', 'profile', 'sh', 'php']); // Add paths where Rector should run $rectorConfig->paths([ __DIR__ . '/core' ]); };But I got 700+ additional changes.
- π¦πΊAustralia mstrelan
The rector script looks good except replace
'php'with'post_update.php'in the file extensions. This is so we only get procedural functions as per scope in the parent issue. - πΊπΈUnited States smustgrave
Applied the suggestion in #5 and now everything seems to line up.
Only other thing that popped up was in core/scripts/run-tests.sh but not sure we update that.
-
quietone β
committed 42b3b113 on 11.x
Issue #3514235 by mstrelan, smustgrave: Add array return to functions in...
-
quietone β
committed 42b3b113 on 11.x
Automatically closed - issue fixed for 2 weeks with no activity.