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

?

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)

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

Just got released btw 🙂

I don't usually face this issue. My

@Composable

methods are not longer than my non-composable ones

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

I use the default one

ah, 60

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

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!

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 👍

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.