Skip to content

fix: LineLengthFixer shortens constructors with promoted properties#81

Closed
TomasVotruba wants to merge 1 commit into
mainfrom
fix/issue-74-linelengthfixer-doesn-t-shorten-constructor-with-p
Closed

fix: LineLengthFixer shortens constructors with promoted properties#81
TomasVotruba wants to merge 1 commit into
mainfrom
fix/issue-74-linelengthfixer-doesn-t-shorten-constructor-with-p

Conversation

@TomasVotruba
Copy link
Copy Markdown
Member

Fixes #74

Move the promoted-property skip from LineLengthTransformer (unconditional) into LineLengthFixer where it is only applied when StandaloneLinePromotedPropertyFixer is registered. Standalone use of LineLengthFixer can now inline short constructors with promoted properties, while the combined-fixer behavior remains unchanged.

…ixes #74)

Move the promoted-property skip from LineLengthTransformer (unconditional)
into LineLengthFixer where it is only applied when
StandaloneLinePromotedPropertyFixer is registered. Standalone use of
LineLengthFixer can now inline short constructors with promoted properties,
while the combined-fixer behavior remains unchanged.
@TomasVotruba
Copy link
Copy Markdown
Member Author

Closing, as conflicts with StandaloneLinePromotedPropertyFixer

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.

LineLengthFixer doesn't shorten constructor with property promotion

1 participant