-
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
"Missing newline after "("" triggered with parameter comments #433
Labels
Comments
I think I see the issue. I'll put up a PR soon. |
shashachu
added a commit
to shashachu/ktlint
that referenced
this issue
May 22, 2019
…if parameter comments were used Fixes pinterest#433
Thanks for the quick fix! |
orchestr7
pushed a commit
to saveourtool/diktat
that referenced
this issue
Jun 29, 2020
…if parameter comments were used (#434) * Fixing issue where ktlint would erroneously want to insert a newline if parameter comments were used Fixes pinterest/ktlint#433
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
I've enabled the --experimental flag on ktlint 0.32, and now we are seeing this error when there are comments on parameters, e.g.:
If foo() is defined in Kotlin, we can fix the error by changing the comments to parameter names. However, if foo() is defined in Java, there seems to be no way to name the parameters without triggering this erroneous message.
The text was updated successfully, but these errors were encountered: