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
Add option to add a reason to ForbiddenMethodCall
#4910
Conversation
Codecov Report
@@ Coverage Diff @@
## main #4910 +/- ##
============================================
- Coverage 84.92% 84.91% -0.01%
- Complexity 3605 3614 +9
============================================
Files 502 502
Lines 11873 11887 +14
Branches 2227 2237 +10
============================================
+ Hits 10083 10094 +11
+ Misses 691 690 -1
- Partials 1099 1103 +4
Continue to review full report at Codecov.
|
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.
👍
...t-rules-style/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/style/ForbiddenMethodCall.kt
Outdated
Show resolved
Hide resolved
How Do we go about breaking changes in minor releases? Do we need an upgrade gradle task for users to call? |
...les-style/src/test/kotlin/io/gitlab/arturbosch/detekt/rules/style/ForbiddenMethodCallSpec.kt
Fixed
Show fixed
Hide fixed
...les-style/src/test/kotlin/io/gitlab/arturbosch/detekt/rules/style/ForbiddenMethodCallSpec.kt
Fixed
Show fixed
Hide fixed
…/rules/style/ForbiddenMethodCall.kt Co-authored-by: schalkms <30376729+schalkms@users.noreply.github.com>
* Improve tests * Improve issue message * Removes that test support with comma-separated strings * Add support for reasons in ForbiddenMethodCall * Update detekt-rules-style/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/style/ForbiddenMethodCall.kt Co-authored-by: schalkms <30376729+schalkms@users.noreply.github.com> Co-authored-by: schalkms <30376729+schalkms@users.noreply.github.com>
Same as #4909 but now adding it to
ForbiddenMethodCall
Important: This introduces a breaking change in
ForbiddenMethodCall
. It no longer allows comma-separated imports as a String. It should use yml lists. I'm targetting this to1.22.0
because on1.21.0
we added make that all those usages will be a warning. Tell me is we should be more conservative.The reason that this is a draft is just to avoid accidental merges. But the code should be ready to merge.
Related with #3501