'If' should force curly braces

[jasondhl]

As stated in the Android Kotlin Style Guide under Formatting, even if an If statement have only one line, it should be surrounded by a curly brace.

In my project, my code:

if (foo)
    bar = "hello world"

was deemed acceptable by ktlint. Is there anything I can do short of writing my own rule? Thanks in advance.

[shyiko]

Hi @jasondhl.
Unfortunately this is not enforced at the moment.
PR is welcome, though.

[jasondhl]

Thanks for the prompt reply @shyiko ! Not very sure how to go about adding this Rule, but I'll check it out.

[shyiko]

@jasondhl take a look inside https://github.com/shyiko/ktlint/tree/master/ktlint-ruleset-standard and https://github.com/shyiko/ktlint#ast (that's more or less all there is to it).

[jkaan]

@jasondhl @shyiko I could have a swing at this, I've implemented this exact rule for Detekt so I have some experience with this :). What do you guys think?

[shyiko]

@jkaan that would be great!

[shyiko]

This rule was contributed by @yokotaso (thank you, Tomoya!). It will be included in the next release.

[shyiko]

I'm reopening the issue as auto-correction isn't quite there yet.

[AleksandrSl]

I can try to add auto-correction

[shyiko]

@AleksandrSl that would be awesome! (here is a little bit of context)

[AleksandrSl]

Can I rely on IndentationRule to be run after other rules, or this rule should generate properly indented code?