Skip to content

Commit

Permalink
fix typo in option name that lead to missed config loader
Browse files Browse the repository at this point in the history
  • Loading branch information
TomasVotruba committed May 15, 2021
1 parent 83e02e8 commit 037cfd4
Show file tree
Hide file tree
Showing 5 changed files with 4 additions and 7 deletions.
5 changes: 0 additions & 5 deletions config/config.php
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,6 @@
declare(strict_types=1);

use Rector\Core\Bootstrap\ExtensionConfigResolver;
use Rector\Core\Configuration\Option;
use Symfony\Component\DependencyInjection\Loader\Configurator\ContainerConfigurator;

return static function (ContainerConfigurator $containerConfigurator): void {
Expand All @@ -20,8 +19,4 @@

// require only in dev
$containerConfigurator->import(__DIR__ . '/../utils/compiler/config/config.php', null, 'not_found');

// to override extension-loaded config
$parameters = $containerConfigurator->parameters();
$parameters->set(Option::PHPSTAN_FOR_RECTOR_PATH, getcwd() . '/phpstan-for-rector.neon');

This comment has been minimized.

Copy link
@TomasVotruba

TomasVotruba May 15, 2021

Author Member

Package configs should never override this parameter.

Fix in Nette here: efabrica-team/rector-nette@69ea2b0
Config was mooved to tests only

};
2 changes: 2 additions & 0 deletions config/parameters.php
Original file line number Diff line number Diff line change
Expand Up @@ -27,6 +27,8 @@
$parameters->set(Option::NESTED_CHAIN_METHOD_CALL_LIMIT, 30);
$parameters->set(Option::SKIP, []);

$parameters->set(Option::PHPSTAN_FOR_RECTOR_PATH, null);

// cache
$parameters->set(Option::ENABLE_CACHE, false);
$parameters->set(Option::CACHE_DIR, sys_get_temp_dir() . '/rector_cached_files');
Expand Down
1 change: 1 addition & 0 deletions rector.php
Original file line number Diff line number Diff line change
Expand Up @@ -104,6 +104,7 @@
'*/Expected*/*',
]);

$parameters->set(Option::PHPSTAN_FOR_RECTOR_PATH, __DIR__ . '/phpstan-for-rector.neon');
$parameters->set(Option::PHP_VERSION_FEATURES, PhpVersion::PHP_74);
$parameters->set(Option::ENABLE_CACHE, true);
};
2 changes: 1 addition & 1 deletion src/Bootstrap/RectorConfigsResolver.php
Original file line number Diff line number Diff line change
Expand Up @@ -43,7 +43,7 @@ private function resolveRectorRecipeConfig(ArgvInput $argvInput): ?SmartFileInfo

private function resolveFromInput(ArgvInput $argvInput): ?SmartFileInfo
{
$configValue = $this->getOptionValue($argvInput, ['config', 'c']);
$configValue = $this->getOptionValue($argvInput, ['--config', '-c']);
if ($configValue === null) {
return null;
}
Expand Down
1 change: 0 additions & 1 deletion src/HttpKernel/RectorKernel.php
Original file line number Diff line number Diff line change
Expand Up @@ -69,7 +69,6 @@ public function getLogDir(): string
public function registerContainerConfiguration(LoaderInterface $loader): void
{
$loader->load(__DIR__ . '/../../config/config.php');

foreach ($this->configFileInfos as $configFileInfo) {
$loader->load($configFileInfo->getRealPath());
}
Expand Down

0 comments on commit 037cfd4

Please sign in to comment.