Drop FunctionMinLength & FunctionMaxLength

[vanniktech]

Looking at the inspections of IntelliJ it seems like it does not validate against min/max length. Glancing over the conventions / style guides I also could not find any restrictions.

[schalkms]

I don't think that this is good idea. I for one use this rule quite frequently.
In my opinion such a reasonable guideline for function naming makes sense.

As previously said, we should let the user decide. There's always the option in detekt to disable this rule, if you have no need for it.

[vanniktech]

FunctionMaxLength I might agree.

With FunctionMinLength it's hard. Often you have functions like id which are flagged. Same for logging verbosity such as d, i, e, w which also get flagged.

[vanniktech]

Another one is io

[arturbosch]

As @schalkms said its a good thing that detekt is highly configurable and the user is free to set his defaults as he wants. These rules are also off by default.

[lock]

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related topics.