-
-
Notifications
You must be signed in to change notification settings - Fork 18
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
allow for multiple comma-separated inputs in check-with-type-resolution #214
allow for multiple comma-separated inputs in check-with-type-resolution #214
Conversation
WalkthroughThe recent changes to the Changes
TipsChat with CodeRabbit Bot (
|
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.
…576) for now, we can use the entire base directory as the input source for detekt type resolution. [i've submitted a change](Ozsie/detekt-maven-plugin#214) against the `detekt-maven-plugin` we're using that will allow us to specify multiple input sources. once this goes through we can be more granular to explicitly use the source directory plus generated sources with the following config: `<input>${project.basedir}/src/main/kotlin,${project.build.directory}/generated-sources/ftl</input>`
Codecov Report
❗ Your organization needs to install the Codecov GitHub app to enable full functionality. @@ Coverage Diff @@
## main #214 +/- ##
============================================
+ Coverage 74.11% 74.13% +0.02%
- Complexity 38 39 +1
============================================
Files 9 9
Lines 170 174 +4
Branches 40 41 +1
============================================
+ Hits 126 129 +3
Misses 17 17
- Partials 27 28 +1
|
@Ozsie Thank you for merging this change. Would it be possible to request a new release that incorporates this? Thank you! |
I'll prepare a release now. |
@worstell Version 1.23.3.1 has been released. |
Summary by CodeRabbit