-
-
Notifications
You must be signed in to change notification settings - Fork 779
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix false positives for MultilineLambdaItParameter.kt #3451
Conversation
Codecov Report
@@ Coverage Diff @@
## master #3451 +/- ##
============================================
- Coverage 80.28% 80.26% -0.02%
- Complexity 2785 2788 +3
============================================
Files 454 454
Lines 8415 8427 +12
Branches 1609 1612 +3
============================================
+ Hits 6756 6764 +8
- Misses 787 789 +2
- Partials 872 874 +2
Continue to review full report at Codecov.
|
Do we want more mixed rules? This one is kind of new. Shouldn't we just turn it in a pure type solving rule? The problem with mixed rules are the user expectations. Detekt seems buggy because of the mixed ones. |
I remember reading some discussions but couldn't remember the resolution. I can change it to a full type-resolving rule. |
+1. We discussed about it in #2994 |
This addresses #3449.
Notes:
it
is used as explicit parameters.