hm, would it make sense to have a rule for definin...

# detekt

[Peter Mandeljc]

hm, would it make sense to have a rule for defining exception as object? I think stacktrace would not be correct, if I do following:

object TitanicSinking : RuntimeException()

// somewhere else
throw TitanicSinking

[Brais Gabin]

https://detekt.dev/docs/rules/exceptions/#objectextendsthrowable

[Brais Gabin]

I think that's what you are looking for, right?

[Peter Mandeljc]

yep, didn't notice that, cool

[Peter Mandeljc]

hm, I actually have it enabled, but it didn't detect it. Will double check and submit bug report if needed

[Brais Gabin]

Do you use Type Solving?

[Peter Mandeljc]

not sure, where can I see that?

[Brais Gabin]

Do you run detekt or `detektMain`/`detektTest`?

[Peter Mandeljc]

just detekt

[Brais Gabin]

That's the reason. That rule only works with type solving enabled: https://detekt.dev/docs/gettingstarted/type-resolution/

[Brais Gabin]

You will get a lot more findings enabling it but it is slower.

[Brais Gabin]

We should remove the "experimental tag" on it. We aren't going to break it. And if we do that we will do that on 2.0

[Peter Mandeljc]

I see, thanks!