Hello there, we use the limit of 60 loc per method as a `Detekt` rule ```LongMethod: active: tru...

# detekt

[rudolf.hladik]

Hello there, we use the limit of 60 loc per method as a Detekt rule

LongMethod:
    active: true
    threshold: 60

but we find it too small for @Composable functions. Is there a way to exclude composables from this limit, something like ignoreAnnotated in LongParameterList ?

[Brais Gabin]

If you updates to the last RC (the stable version is near to be released) you can use ignoreAnnotated in that rule (or any other rule)

[rudolf.hladik]

thx, I see its already in v1.18.0 🙂 I will upgrade

[gammax]

Just got released btw 🙂

[LeoColman]

I don't usually face this issue. My @Composable methods are not longer than my non-composable ones

[diego-gomez-olvera]

which threshold do you use? Maybe the configuration that I use is too strict

[LeoColman]

I use the default one

[diego-gomez-olvera]

ah, 60

[Brais Gabin]

Same for me. 60 lines (default value). My composables shouldn't be bigger than other functions. 60 give me a lot of room.

[Brais Gabin]

Ideally I would like to have another threshold for for LongMethod but I do not know if that's possible

This is a know limition of the current detekt. There is no way to have the same rule with two different confirgurations. We want to fix that for 2.0.

As a workaround you can create a new rule copy&pasting the code in LongMethod and configure it as you wish

[diego-gomez-olvera]

cool, I will review the current threshold and se how many @Composable exceed 60.
If it's common to have legitimate occurrences, I will consider as you suggest and have a custom LongMethod for @Composable method. Thanks!

[Jan Skrasek]

We face similar compose problem but rather with number of parameters. As it is more common to pass them destructed and not wrap to additional objects causing allocations and remember usage. So definitely split setup would be nice 👍

[Sam Pengilly]

Method length is probably a good sign to refactor such a composable into one that accepts “slot” parameters. Separating the concern of layout from the concern of contents, lifting the concern of state up a level, and hopefully reducing the size of the composable functions as well as making them more general.