Skip to content

Commit a10bb09

Browse files
committed
minor #41009 [Config ]Remove log message when no configuration is found in ConfigBuilder (alexander-schranz)
This PR was merged into the 5.3-dev branch. Discussion ---------- [Config ]Remove log message when no configuration is found in ConfigBuilder | Q | A | ------------- | --- | Branch? | 5.x for features / 4.4 or 5.2 for bug fixes <!-- see below --> | Bug fix? | - | New feature? | no <!-- please update src/**/CHANGELOG.md files --> | Deprecations? | no <!-- please update UPGRADE-*.md and src/**/CHANGELOG.md files --> | Tickets | Fix #... <!-- prefix each issue number with "Fix #", no need to create an issue if none exist, explain below instead --> | License | MIT | Doc PR | symfony/symfony-docs#... <!-- required for new features --> <!-- Replace this notice by a short README for your feature/bugfix. This will help people understand your PR and can be used as a start for the documentation. Additionally (see https://symfony.com/releases): - Always add tests and ensure they pass. - Never break backward compatibility (see https://symfony.com/bc). - Bug fixes must be submitted against the lowest maintained branch where they apply (lowest branches are regularly merged to upper ones so they get the fixes too.) - Features and deprecations must be submitted against branch 5.x. - Changelog entry should follow https://symfony.com/doc/current/contributing/code/conventions.html#writing-a-changelog-entry --> I think when a bundle has no configuration we should also not log an warning. Commits ------- 20c03b5c87 Remove log message when not configuration is found
2 parents 59e8646 + bb5f04a commit a10bb09

File tree

1 file changed

+0
-4
lines changed

1 file changed

+0
-4
lines changed

CacheWarmer/ConfigBuilderCacheWarmer.php

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -77,10 +77,6 @@ private function dumpExtension(ExtensionInterface $extension, ConfigBuilderGener
7777
}
7878

7979
if (!$configuration) {
80-
if ($this->logger) {
81-
$this->logger->info('No configuration found for extension {extensionClass}.', ['extensionClass' => \get_class($extension)]);
82-
}
83-
8480
return;
8581
}
8682

0 commit comments

Comments
 (0)