Generated by 🚫 dangerJS against 9dd130e

Who should add projects.detektPsiUtils in the classpath now? The clients? Or should we add it as a runtime only dependency on the core? Or :detekt-rules also as a runtime only?

It's already an implementation dependency of detekt-core, so it's on the runtime classpath:

Oh, I thought that ci was failing exactly for that issue 🤐

ci was failing exactly for that issue

If only it were that simple... it's failing because this function was catching an IndexOutOfBoundsException due to errors in the implementation of MaxLineLength:

The above code was catching the exception, and I didn't copy that exactly to the new location because it shouldn't be needed anymore.

The MaxLineLength implementation should be fixed, but it looks like it needs changes to Location and/or Entity... until that's been corrected I'd call this and #7261 blocked.

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 84.77%. Comparing base (2f5225d) to head (9dd130e).

@@            Coverage Diff            @@
##               main    #7260   +/-   ##
=========================================
  Coverage     84.77%   84.77%           
  Complexity     3990     3990           
=========================================
  Files           578      578           
  Lines         12023    12027    +4     
  Branches       2478     2478           
=========================================
+ Hits          10192    10196    +4     
  Misses          606      606           
  Partials       1225     1225           

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