implement UnnecessaryLayoutWrapper rule

[sergey-shevtsov]

The rule reports if the use of Box, Column or Row is not required. For example, in code below, we can easily remove the parent column.

Column {
    Box(modifier = Modifier.size(200.dp)) {
        // Other content
    }
}

[dimsuz]

Please add examples of compliant and non compliant code (we'll use them later to generate docs).

See example here

[dimsuz]

Need a license header, here's an example

[dimsuz]

There's no need to accumulate expressions and report them in postVisit, you can report them right when you find them, i.e. instead of having

unnecessaryExpressions.add(expression)

just call

report(...)

Detekt will visit all call expressions anyway, so no need to complicate with preVisit, postVisit and state here...

[dimsuz]

nit: hasSingleLayoutCall?

[dimsuz]

This also needs to be added to the sample block in the README.md, some people will be copying rules from here (config.yml doesn't work for older detekt versions)

[dimsuz]

Secondly, this rule must be added to the rule set provider.

[dimsuz]

you should test all supported layout types here. data-driven testing feature can be used here:

https://kotest.io/docs/framework/datatesting/data-driven-testing.html