Skip to content

Commit 50b55dc

Browse files
authored
bug #40 Remove duplicated ClassAttributesSeparationFixer and use default config (LucaGallinari)
This PR was merged into the 4.0-dev branch. Discussion ---------- Sorry but there was an error in the previous PR. The ClassAttributesSeparationFixer is specified twice and the [upgrade.md ](https://github.com/FriendsOfPHP/PHP-CS-Fixer/blob/3.0/UPGRADE-v3.md)file was misleading: `['elements' => ['method']]` is not a valid config for this fixer. Btw the default config is ok https://cs.symfony.com/doc/rules/class_notation/class_attributes_separation.html so we can avoid to specify a custom configuration for this fixer. Commits ------- ed9634a Remove duplicated ClassAttributesSeparationFixer and use default config
2 parents 8c52560 + ed9634a commit 50b55dc

File tree

1 file changed

+0
-3
lines changed

1 file changed

+0
-3
lines changed

ecs.php

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -189,9 +189,6 @@
189189
$services->set(ClassDefinitionFixer::class)
190190
->call('configure', [['single_item_single_line' => true, 'multi_line_extends_each_single_line' => true]]);
191191

192-
$services->set(ClassAttributesSeparationFixer::class)
193-
->call('configure', [['elements' => ['method']]]);
194-
195192
$services->set(NoBlankLinesAfterClassOpeningFixer::class);
196193

197194
$services->set(NoNullPropertyInitializationFixer::class);

0 commit comments

Comments
 (0)