rudolf.hladik
11/29/2021, 10:34 AMDetekt
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
11/29/2021, 10:39 AMignoreAnnotated
in that rule (or any other rule)rudolf.hladik
11/29/2021, 10:42 AMgammax
11/29/2021, 10:52 AMLeoColman
12/29/2022, 1:43 PM@Composable
methods are not longer than my non-composable onesdiego-gomez-olvera
12/29/2022, 1:44 PMLeoColman
12/29/2022, 1:44 PMdiego-gomez-olvera
12/29/2022, 1:45 PMBrais Gabin
12/29/2022, 1:45 PMIdeally I would like to have another threshold for forThis 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 inbut I do not know if that's possibleLongMethod
LongMethod
and configure it as you wishdiego-gomez-olvera
12/29/2022, 1:51 PM@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
12/31/2022, 7:29 AMSam Pengilly
01/04/2023, 5:48 AM