-
Notifications
You must be signed in to change notification settings - Fork 503
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Wrong new line check #842
Comments
Thanks for the report @kozaxinan. To clarify: both errors are reporting on the |
Likely introduced by this commit a94b43f |
Yes that is a regression in alpha and yes it is on trimIndent line. |
@kozaxinan I could not reproduce it with following Kotlin file: val test =
"""
|longtext longtext longtext longtext longtext longtext
|longtext longtext longtext longtext longtext longtext
|longtext longtext longtext longtext longtext longtext
|longtext longtext longtext longtext longtext longtext
""".trimMargin("|") Maybe you could provide full Kotlin file or link to it? |
Running ktlint on this file gives two errors.
|
@kozaxinan sorry, but you example only produces indentation errors for me with latest build from
Could you try |
I tested that code example with 0.38 and command line ktlint. I will test issue in a fresh repo. |
Do you have |
I checked code in empty folder outside of project without editorconfig. Running 'ktlint --android' gives errors for me with 0.38 |
Yeah, with |
@kozaxinan with 0.39.0 release you could disable experimental 'argument-list-wrapping' rule to suppress this issue. |
The bug you're experiencing might have already be reported!
Please search in the issues before creating one.
Expected Behavior
Method after multiline text
Observed Behavior
Ktlint gives 2 error.
Argument should be on a separate line (unless all arguments can fit a single line)ktlintTestSourceSetCheck
Missing newline before ")"
Steps to Reproduce
0.38.0-alpha01
ktlintCheck
from https://github.com/JLLeitschuh/ktlint-gradleThe text was updated successfully, but these errors were encountered: