What do folks do with LongParameterList warnings? ...

# detekt

[kenkyee]

What do folks do with LongParameterList warnings? I used to recommend that people use a data class to pass paramaters into a constructor or function to group the parameters logically.
The current versions of Detekt also flag too many parameters in data class constructors... 😔

[Jukka Siivonen]

There's a parameter for that which should be true by default ignoreDataClasses https://arturbosch.github.io/detekt/complexity.html

[John Towell]

we ignored it and wrote our own to warn only for long unnamed parameters, similar to this suggestion https://github.com/detekt/detekt/issues/1007

[John Towell]

long named parameters is more readable than long unnamed parameters to a data object imo