Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

Multiline type parameter list is formatted incorrectly #1867

Closed
paul-dingemans opened this issue Mar 18, 2023 · 1 comment · Fixed by #1869
Closed

Multiline type parameter list is formatted incorrectly #1867

paul-dingemans opened this issue Mar 18, 2023 · 1 comment · Fixed by #1869
Milestone
0.49.0

Comments

@paul-dingemans
Copy link
Collaborator

Given code below:

fun <
    Foo,
    Bar,
    > foobar()

This is formatted by ktlint rule experimental:type-parameter-list-spacing as follows:

fun <Foo,
    Bar,> foobar()

while it should have been kept unchanged.
@paul-dingemans paul-dingemans added this to the 0.49.0 milestone Mar 18, 2023
@paul-dingemans
Copy link
Collaborator Author

Related to above, the code below:

fun <
    Foo, Bar,
    FooBar,
    > foobar()

should be formatted as:

fun <
    Foo,
    Bar,
    FooBar,
    > foobar()

paul-dingemans added a commit that referenced this issue Mar 18, 2023
@paul-dingemans
Do not remove newlines from multiline type parameter lists `type-para…
e05dac0 
…meter-list-spacing`

Closes #1867
paul-dingemans added a commit that referenced this issue Mar 18, 2023
@paul-dingemans
Wrap each type parameter in a multiline type parameter list
1ddfdb9 
Closes #1867
@paul-dingemans paul-dingemans mentioned this issue Mar 18, 2023
Merged
6 tasks
# for free to join this conversation on GitHub. Already have an account? # to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
0.49.0
Development

Successfully merging a pull request may close this issue.

Type parameter list spacing and wrapping pinterest/ktlint
1 participant
@paul-dingemans