So after updating IDEA plugin I get LongParameterL...
# detektj
Jukka Siivonen
05/13/2020, 7:49 AMSo after updating IDEA plugin I get LongParameterList error for many of my non data type constructors, is this intended behavior? I would expect to get this error only for function parameters
Jukka Siivonen
05/13/2020, 7:51 AMaccording to error constructor is function
Jukka Siivonen
05/13/2020, 7:52 AMLongParameterList: The function Foo has too many parameters. The current threshold is set to 6.
b
Brais Gabin
05/13/2020, 7:52 AMYes, it is an intended behavior. You can configure both thresholds if you want.
j
Jukka Siivonen
05/13/2020, 7:53 AMbut changing `constructorThreshold`would not help?
Jukka Siivonen
05/13/2020, 7:53 AMit's using
functionThresholdb
Brais Gabin
05/13/2020, 7:53 AMOh, that's a bug in the message. Could you open an issue to track it?
j
Jukka Siivonen
05/13/2020, 7:54 AMyou sure? because `functionThreshold`default is 7 not 6 which is in error message
b
Brais Gabin
05/13/2020, 7:54 AMIt should use constructorTheshold, yes.
j
Jukka Siivonen
05/13/2020, 7:57 AMyou are right it's using
Copy code
constructorThresholdb
Brais Gabin
05/13/2020, 7:58 AMYeah, it seems an issue in the message.
s
schalkms
05/13/2020, 6:55 PMimprovement: https://github.com/detekt/detekt/pull/2684
schalkms
05/13/2020, 6:55 PMThanks for reporting this @Jukka Siivonen
92 Views