Skip to content

[Sets] Drop DoctrineSetProvider and TwigSetProvider from the collector - #8298

Merged
TomasVotruba merged 4 commits into
mainfrom
drop-doctrine-set-provider
Aug 5, 2026
Merged

[Sets] Drop DoctrineSetProvider and TwigSetProvider from the collector#8298
TomasVotruba merged 4 commits into
mainfrom
drop-doctrine-set-provider

Conversation

@TomasVotruba

@TomasVotruba TomasVotruba commented Aug 5, 2026

Copy link
Copy Markdown
Member

Follows up on #8295 (PHPUnitSetProvider) and #8292 (Symfony per-version set providers) - removes the DoctrineSetProvider and TwigSetProvider registrations from SetProviderCollector, leaving only the core PHP/Core providers.

 use Rector\Set\Contract\SetInterface;
 use Rector\Set\Contract\SetProviderInterface;
 use Rector\Set\SetProvider\CoreSetProvider;
 use Rector\Set\SetProvider\PHPSetProvider;
 use Rector\Set\ValueObject\ComposerTriggeredSet;
-use Rector\Doctrine\Set\SetProvider\DoctrineSetProvider;
-use Rector\Symfony\Set\SetProvider\TwigSetProvider;

 ...

         $setProviders = [
             // register all known set providers here
             new PHPSetProvider(),
             new CoreSetProvider(),
-            new DoctrineSetProvider(),
-            new TwigSetProvider(),
         ];

Providers can still be passed in via the $extraSetProviders constructor argument - that is how RectorConfigBuilder::__invoke() already wires user-registered providers, and how SetManagerTest now supplies TwigSetProvider.

@TomasVotruba
TomasVotruba enabled auto-merge (squash) August 5, 2026 19:28
@TomasVotruba TomasVotruba changed the title [Sets] Drop DoctrineSetProvider from the collector [Sets] Drop DoctrineSetProvider and TwigSetProvider from the collector Aug 5, 2026
@TomasVotruba
TomasVotruba merged commit 1949c3d into main Aug 5, 2026
64 checks passed
@TomasVotruba
TomasVotruba deleted the drop-doctrine-set-provider branch August 5, 2026 20:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

1 participant