-
Notifications
You must be signed in to change notification settings - Fork 508
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
Line wrap for && #168
Comments
See #163 (comment). |
Fixed in 0.20.0. |
@shyiko No, it's still not working as expected. if (!isLocal
&& workspace.userId != account.userId
&& workspace.contactId != account.userId) {
}
Should I create a new issue or do you want to reopen this one? |
@vRallev It works in accordance with #163 (comment). |
I can't follow the comments in the link. They're mostly talking about +/- and not && or ||. I couldn't find anything about that. Note that the auto-formatter inside of IntelliJ formats my sample from above like this: if (!isLocal
&& workspace.userId != account.userId
&& workspace.contactId != account.userId
) {
// do something after 4 space indent
} But for the formatter the following is also fine if (!isLocal &&
workspace.userId != account.userId &&
workspace.contactId != account.userId
) {
// do something after 4 space indent
} So there's no evidence which one of them is correct. |
How can we get the intellij auto-formatter to always move those operators to the end of the line? I can't find any options for that setting 🤔 |
Did you find a solution for that? |
Have you found the settings for that? |
I've seen f13b93e and probably because of that my code is wrong. My snippet looks like this
Kotlin Lint complaints with
Line must not begin with "&&"
. I couldn't find anything regarding that in the Kotlin style guide and to me this rule seems wrong. Can you point me to the rule in the style guide? Is there a way to disable this specific check?On a side note, this is working fine and Kotlin Lint doesn't complain
The text was updated successfully, but these errors were encountered: