-
-
Notifications
You must be signed in to change notification settings - Fork 760
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 rule from NamingRules multi rule #5212
Conversation
detekt-rules-naming/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/naming/ClassNaming.kt
Fixed
Show fixed
Hide fixed
...s-naming/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/naming/FunctionParameterNaming.kt
Fixed
Show fixed
Hide fixed
...aming/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/naming/ConstructorParameterNaming.kt
Fixed
Show fixed
Hide fixed
...les-naming/src/test/kotlin/io/gitlab/arturbosch/detekt/rules/naming/VariableMinLengthSpec.kt
Fixed
Show fixed
Hide fixed
...les-naming/src/test/kotlin/io/gitlab/arturbosch/detekt/rules/naming/FunctionMinLengthSpec.kt
Fixed
Show fixed
Hide fixed
@marschwar The merge request ready for review. |
detekt-rules-naming/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/naming/NamingProvider.kt
Outdated
Show resolved
Hide resolved
...lin/io/gitlab/arturbosch/detekt/rules/coroutines/SuspendFunWithCoroutineScopeReceiverSpec.kt
Outdated
Show resolved
Hide resolved
Codecov Report
@@ Coverage Diff @@
## main #5212 +/- ##
============================================
- Coverage 84.93% 84.88% -0.06%
+ Complexity 3716 3705 -11
============================================
Files 514 513 -1
Lines 12068 12024 -44
Branches 2271 2270 -1
============================================
- Hits 10250 10206 -44
+ Misses 698 691 -7
- Partials 1120 1127 +7
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. |
5e74730
to
1871aac
Compare
...aming/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/naming/ConstructorParameterNaming.kt
Fixed
Show fixed
Hide fixed
@cortinico is there anything else that should be changed here? |
Nope we can merge it 👍 |
@VovaStelmashchuk Thanks for this PR :) |
This is part of #5192