Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Change variable min & max length to match IntelliJ. #635

Merged
merged 3 commits into from Jan 1, 2018
Merged

Change variable min & max length to match IntelliJ. #635

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
728cc75
Change variable min & max length to match IntelliJ.
vanniktech Dec 21, 2017
e260c20
Update all the things.
vanniktech Dec 21, 2017
422f068
Add author.
vanniktech Dec 21, 2017
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
4 changes: 2 additions & 2 deletions ...rules/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/style/naming/VariableMaxLength.kt
View file
Open in desktop
Expand Up @@ -10,7 +10,7 @@ import io.gitlab.arturbosch.detekt.api.Severity
import org.jetbrains.kotlin.psi.KtProperty

/**
* @configuration maximumVariableNameLength - maximum name length (default: 30)
* @configuration maximumVariableNameLength - maximum name length (default: 64)
* @author Marvin Ramin
*/
class VariableMaxLength(config: Config = Config.empty) : Rule(config) {
Expand All @@ -33,6 +33,6 @@ class VariableMaxLength(config: Config = Config.empty) : Rule(config) {

companion object {
const val MAXIMUM_VARIABLE_NAME_LENGTH = "maximumVariableNameLength"
private const val DEFAULT_MAXIMUM_VARIABLE_NAME_LENGTH = 30
private const val DEFAULT_MAXIMUM_VARIABLE_NAME_LENGTH = 64
}
}
4 changes: 2 additions & 2 deletions ...rules/src/main/kotlin/io/gitlab/arturbosch/detekt/rules/style/naming/VariableMinLength.kt
View file
Open in desktop
Expand Up @@ -11,7 +11,7 @@ import org.jetbrains.kotlin.psi.KtProperty
import org.jetbrains.kotlin.resolve.calls.util.isSingleUnderscore

/**
* @configuration minimumVariableNameLength - maximum name length (default: 3)
* @configuration minimumVariableNameLength - maximum name length (default: 1)
* @author Marvin Ramin
*/
class VariableMinLength(config: Config = Config.empty) : Rule(config) {
Expand All @@ -38,6 +38,6 @@ class VariableMinLength(config: Config = Config.empty) : Rule(config) {

companion object {
const val MINIMUM_VARIABLE_NAME_LENGTH = "minimumVariableNameLength"
private const val DEFAULT_MINIMUM_VARIABLE_NAME_LENGTH = 3
private const val DEFAULT_MINIMUM_VARIABLE_NAME_LENGTH = 1
}
}