Clean up code by using builtin associateBy function

[arturbosch]

No description provided.

[github-actions]

	Warnings
⚠️	detekt-cli/src/main/kotlin/io/gitlab/arturbosch/detekt/cli/OutputFacade.kt#L27 - Type checking is a sign of violating the Open-Closed Principle. Consider introducing an abstraction (interface) for OutputReport, ConsoleReport, with new implementations of the interface for every class.

Generated by 🚫 Danger Kotlin against 0d0e23e

[codecov]

Codecov Report

Merging #2715 into master will increase coverage by 0.04%.
The diff coverage is 100.00%.

@@             Coverage Diff              @@
##             master    #2715      +/-   ##
============================================
+ Coverage     80.52%   80.57%   +0.04%     
- Complexity     2322     2323       +1     
============================================
  Files           378      378              
  Lines          6944     6940       -4     
  Branches       1258     1257       -1     
============================================
  Hits           5592     5592              
+ Misses          725      721       -4     
  Partials        627      627              

Impacted Files	Coverage Δ	Complexity Δ
...ain/kotlin/io/gitlab/arturbosch/detekt/cli/Junk.kt	100.00% <ø> (+66.66%)	0.00 <0.00> (ø)
...in/io/gitlab/arturbosch/detekt/cli/OutputFacade.kt	94.44% <100.00%> (ø)	7.00 <3.00> (+1.00)

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 5926d84...0d0e23e. Read the comment docs.

[arturbosch]

Warnings

detekt-cli/src/main/kotlin/io/gitlab/arturbosch/detekt/cli/OutputFacade.kt#L27 - Type checking is a sign of violating the Open-Closed Principle. Consider introducing an abstraction (interface) for OutputReport, ConsoleReport, with new implementations of the interface for every class.

Generated by Danger Kotlin against 0d0e23e

Jap, its bad design here.