Skip to content

[config] Add FromServicePublicToDefaultsPublicRector#857

Merged
TomasVotruba merged 1 commit intomainfrom
tv-public-once
Oct 26, 2025
Merged

[config] Add FromServicePublicToDefaultsPublicRector#857
TomasVotruba merged 1 commit intomainfrom
tv-public-once

Conversation

@TomasVotruba
Copy link
Copy Markdown
Member

@TomasVotruba TomasVotruba commented Oct 26, 2025

One more manual step automated 💪


use Symfony\Component\DependencyInjection\Loader\Configurator\ContainerConfigurator;

return static function (ContainerConfigurator $containerConfigurator): void {
    $services = $containerConfigurator->services();

+   $services->defaults()->public();

-    $services->set(SomeCommand::class)
+    $services->set(SomeCommand::class);
-        ->public();

-    $services->set(AnotherCommand::class)
+    $services->set(AnotherCommand::class);
-        ->public();

-    $services->set(NextCommand::class)
+    $services->set(NextCommand::class);
-        ->public();
};

@TomasVotruba TomasVotruba merged commit 7ef1d4b into main Oct 26, 2025
5 checks passed
@TomasVotruba TomasVotruba deleted the tv-public-once branch October 26, 2025 16:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant