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
Remove ability to create AnalysisResult by passing source code and filename #7255
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #7255 +/- ##
============================================
+ Coverage 84.74% 84.79% +0.04%
- Complexity 3996 3998 +2
============================================
Files 577 577
Lines 12126 12119 -7
Branches 2484 2483 -1
============================================
Hits 10276 10276
+ Misses 623 616 -7
Partials 1227 1227 ☔ View full report in Codecov by Sentry. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ok, this have a lot of sense. While I was reviewing this PRs I was thinking "Why do we need this in our production code?". It was generating noise and it's great to move it so only the tests uses it.
…lename Calling detekt from production code should only ever pass a KtFile or a Path (from which a KtFile can be created). Creating KtFile from text directly is only required from tests.
Calling detekt from production code should only ever pass a
KtFile
or aPath
(from which a KtFile can be created).Creating
KtFile
from text directly is only required from tests.This is a breaking change for detekt-intellij-plugin, but a quick look at the source in that project suggests it will be a simple update.
Waiting on merge of: