[IgnoredReturnValue] Add returnType?.isNothing() check

[atulgpt]

Fixes #7765

[codecov]

Codecov Report

Attention: Patch coverage is 0% with 1 line in your changes missing coverage. Please review.

Project coverage is 85.43%. Comparing base (754b4b6) to head (4c9189c).

Files with missing lines	Patch %	Lines
...arturbosch/detekt/rules/bugs/IgnoredReturnValue.kt	0.00%	0 Missing and 1 partial ⚠️

Additional details and impacted files

@@             Coverage Diff              @@
##               main    #7772      +/-   ##
============================================
- Coverage     85.44%   85.43%   -0.01%     
- Complexity     4175     4177       +2     
============================================
  Files           571      571              
  Lines         13136    13137       +1     
  Branches       2489     2490       +1     
============================================
  Hits          11224    11224              
  Misses          664      664              
- Partials       1248     1249       +1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[BraisGabin]

x"D I was just reviewing the issue and saw "oh there is a PR already". I enter and it was just 1 minute old :P

Thanks for all the contributions (I know that you have more things that need my review but I have not the time :'( )

[BraisGabin]

Didn't we have a test for this case? Good catch then!

[atulgpt]

I could only find Unit involving with chain

[schalkms]

Thanks for fixing this issue!
I’d also like to have an explicit test case for Nothing, e.g.

fun fail(message: String): Nothing {
    //…
}