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

Space after constructor #1207

Closed
BraisGabin opened this issue Aug 9, 2021 · 0 comments · Fixed by #1211
Closed

Space after constructor #1207

BraisGabin opened this issue Aug 9, 2021 · 0 comments · Fixed by #1211

Comments

@BraisGabin
Copy link

Expected Behavior

class Klass constructor ()

should be flagged and fixed to:

class Klass constructor()

Observed Behavior

It doesn't

Steps to Reproduce

echo 'class Klass constructor ()'> Klass.kt && ktlint Klass.kt

Your Environment

  • Version of ktlint used: 0.42.1 (same for 0.41.0)
paul-dingemans pushed a commit to paul-dingemans/ktlint that referenced this issue Aug 9, 2021
Remove spaces before primary constructor
ccb2047 
Closes pinterest#1207
@paul-dingemans paul-dingemans mentioned this issue Aug 11, 2021
Merged
2 tasks
romtsn added a commit that referenced this issue Aug 20, 2021
@paul-dingemans @romtsn
Remove spaces before primary constructor (#1211)
7ae692a 
* Remove spaces before primary constructor

Closes #1207

* Update changelog

Co-authored-by: Paul Dingemans <pdingemans@bol.com>
Co-authored-by: Roman Zavarnitsyn <rom4ek93@gmail.com>
# 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
No milestone
Development

Successfully merging a pull request may close this issue.

Remove spaces before primary constructor paul-dingemans/ktlint
1 participant
@BraisGabin