Hello everyone, I switched to `buildUponDefaultCon...

# detekt

[sanogueralorenzo]

Hello everyone, I switched to buildUponDefaultConfig = true recently and realised that overriden functions also show as EmptyFunctionBlock (I know there is a flag to ignoreOverriden) I'm more curious to know what was the reasoning behind leaving it to false and what is the best approach to solve situations where you are implementing an interface from the framework or third party provider that you have no control over.

[Brais Gabin]

If you add a comment in the body of an overriden method the rule doesn’t flag it. I think that the reason for this is that if you didn’t implement a method you should say why. This way, if someone else read your code knows why that’s empty. If you work with Android this is a mess because there are far too much things that you don’t want to implement because anyone know that it’s pointless but 🤷 for that reason we have the switch.

[sanogueralorenzo]

Makes sense, thank you!