Fix unwanted whitespace between super class constructor and its argument list - Githubissues

pinterest / ktlint

An anti-bikeshedding Kotlin linter with built-in formatter

https://pinterest.github.io/ktlint/

MIT License

6.06k stars 504 forks source link

Fix unwanted whitespace between super class constructor and its argument list #2630

Closed tKe closed 2 months ago

tKe commented 2 months ago

Description

Adds support for detecting whitespace before a value argument list when it follows the type argument list of a constructor call.

This is needed to correctly detect and remove the excess whitespace in the following sample:

class MyClass : MySuper<String> ("hello")

Checklist

Before submitting the PR, please check following (checks which are not relevant may be ignored):

[x] Commit message are well written. In addition to a short title, the commit message also explain why a change is made.
[ ] ~~At least one commit message contains a reference Closes #<xxx> or Fixes #<xxx> (replace<xxx> with issue number)~~
[x] Tests are added
[x] KtLint format has been applied on source code itself and violations are fixed
[x] PR title is short and clear (it is used as description in the release changelog)
[x] PR description added (background information)

Documentation is updated. See difference between snapshot and release documentation

[ ] ~~Snapshot documentation in case documentation is to be released together with a code change~~
[ ] ~~Release documentation in case documentation is related to a released version of ktlint and has to be published as soon as the change is merged to master~~

paul-dingemans commented 2 months ago

Tnx for your contribution. I have added similar logic to the class-signature rule which in the long run might replace (parts of) the paren-spacing rule.