minor #61332 Cleanup .php-cs-fixer.dist.php (nicolas-grekas)

nicolas-grekas · nicolas-grekas · commit 7b2d29fa6011 · 2025-08-04T18:59:13.000+02:00
This PR was merged into the 7.4 branch. Discussion ---------- Cleanup .php-cs-fixer.dist.php | Q | A | ------------- | --- | Branch? | 7.4 | Bug fix? | no | New feature? | no | Deprecations? | no | Issues | - | License | MIT Better have a few false-positives than missing an important change (esp a phpunit attribute) Commits ------- 5d4e211 Cleanup .php-cs-fixer.dist.php
diff --git a/.php-cs-fixer.dist.php b/.php-cs-fixer.dist.php
@@ -28,11 +28,8 @@
 ];
 
 return (new PhpCsFixer\Config())
-    // @see https://github.com/PHP-CS-Fixer/PHP-CS-Fixer/pull/7777
     ->setParallelConfig(PhpCsFixer\Runner\Parallel\ParallelConfigFactory::detect())
     ->setRules([
-        '@PHP71Migration' => true,
-        '@PHPUnit75Migration:risky' => true,
         '@Symfony' => true,
         '@Symfony:risky' => true,
         'phpdoc_var_annotation_correct_order' => true,
@@ -61,25 +58,13 @@
                 'Symfony/Component/Emoji/Resources/',
                 'Symfony/Component/Intl/Resources/data/',
             ])
-            // explicit tests for ommited @param type, against `no_superfluous_phpdoc_tags`
-            ->notPath('Symfony/Component/PropertyInfo/Tests/Extractor/PhpDocExtractorTest.php')
-            ->notPath('Symfony/Component/PropertyInfo/Tests/Extractor/PhpStanExtractorTest.php')
             // Support for older PHPunit version
-            ->notPath('Symfony/Bridge/PhpUnit/SymfonyTestsListener.php')
             ->notPath('#Symfony/Bridge/PhpUnit/.*Mock\.php#')
             ->notPath('#Symfony/Bridge/PhpUnit/.*Legacy#')
-            // explicit trigger_error tests
-            ->notPath('Symfony/Component/ErrorHandler/Tests/DebugClassLoaderTest.php')
-            // stop removing spaces on the end of the line in strings
-            ->notPath('Symfony/Component/Messenger/Tests/Command/FailedMessagesShowCommandTest.php')
             // disable to not apply `native_function_invocation` rule, as we explicitly break it for testability reason, ref https://github.com/symfony/symfony/pull/59195
             ->notPath('Symfony/Component/Mailer/Transport/NativeTransportFactory.php')
             // auto-generated proxies
-            ->notPath('Symfony/Component/Cache/Traits/RelayProxy.php')
-            ->notPath('Symfony/Component/Cache/Traits/Redis5Proxy.php')
-            ->notPath('Symfony/Component/Cache/Traits/Redis6Proxy.php')
-            ->notPath('Symfony/Component/Cache/Traits/RedisCluster5Proxy.php')
-            ->notPath('Symfony/Component/Cache/Traits/RedisCluster6Proxy.php')
+            ->notPath('#Symfony/Component/Cache/Traits/Re.*Proxy\.php#')
             // svg
             ->notPath('Symfony/Component/ErrorHandler/Resources/assets/images/symfony-ghost.svg.php')
             // HTML templates
