Skip to content

Conversation

@TomasVotruba
Copy link
Member

@TomasVotruba TomasVotruba commented Dec 10, 2025

The SortNamedParamRector was doing too much. Now the attributes are handled by their own rule - SortAttributeNamedArgsRector

@TomasVotruba TomasVotruba force-pushed the tv-split-attribute-named-params branch from 89b9dfc to 953c043 Compare December 10, 2025 18:35
@TomasVotruba TomasVotruba changed the title [code-quality] Extract standalone SortAttributeNamedArgsRector from SortNamedParamRector, to slightly different areas [code-quality] Extract standalone SortAttributeNamedArgsRector from SortNamedParamRector, to allow standalone use Dec 10, 2025
@TomasVotruba TomasVotruba force-pushed the tv-split-attribute-named-params branch from a4f257c to d3f6d55 Compare December 10, 2025 18:37
…ortNamedParamRector, to slightly different areas
@TomasVotruba TomasVotruba force-pushed the tv-split-attribute-named-params branch from d3f6d55 to 80eaa10 Compare December 10, 2025 18:37
@TomasVotruba TomasVotruba merged commit 4ffd470 into main Dec 10, 2025
58 checks passed
@TomasVotruba TomasVotruba deleted the tv-split-attribute-named-params branch December 10, 2025 18:40
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.

2 participants