-
Notifications
You must be signed in to change notification settings - Fork 4
Meta: re-enable disabled detekt rules (tracking) #310
Copy link
Copy link
Open
Description
Summary
Tracker for re-enabling detekt rules that are currently turned off in backend/detekt.yml. Each rule has its own issue (labeled good first issue). Close this meta issue when every linked item is done, or keep it as historical reference—your call.
How to work an item
- Open the linked issue, enable the rule, run
./gradlew detektfrombackend/, fix violations, merge. - Check off the line below (or leave unchecked until the child issue is closed).
style
- Enable detekt rule: MagicNumber #279 Enable detekt rule: MagicNumber
- Enable detekt rule: WildcardImport #280 Enable detekt rule: WildcardImport
- Enable detekt rule: ReturnCount #281 Enable detekt rule: ReturnCount
- Enable detekt rule: ThrowsCount #282 Enable detekt rule: ThrowsCount
- Enable detekt rule: MaxLineLength #283 Enable detekt rule: MaxLineLength
- Enable detekt rule: UseRequire #284 Enable detekt rule: UseRequire
- Enable detekt rule: UseCheckOrError #285 Enable detekt rule: UseCheckOrError
- Enable detekt rule: ForbiddenComment #286 Enable detekt rule: ForbiddenComment
- Enable detekt rule: LoopWithTooManyJumpStatements #287 Enable detekt rule: LoopWithTooManyJumpStatements
- Enable detekt rule: DestructuringDeclarationWithTooManyEntries #288 Enable detekt rule: DestructuringDeclarationWithTooManyEntries
- Enable detekt rule: UnusedPrivateProperty #289 Enable detekt rule: UnusedPrivateProperty
- Enable detekt rule: UnusedPrivateMember #290 Enable detekt rule: UnusedPrivateMember
complexity
- Enable detekt rule: TooManyFunctions #291 Enable detekt rule: TooManyFunctions
- Enable detekt rule: LongMethod #292 Enable detekt rule: LongMethod
- Enable detekt rule: LongParameterList #293 Enable detekt rule: LongParameterList
- Enable detekt rule: CyclomaticComplexMethod #294 Enable detekt rule: CyclomaticComplexMethod
- Enable detekt rule: ComplexCondition #295 Enable detekt rule: ComplexCondition
- Enable detekt rule: LargeClass #296 Enable detekt rule: LargeClass
- Enable detekt rule: NestedBlockDepth #297 Enable detekt rule: NestedBlockDepth
- Enable detekt rule: NestedScopeFunctions #298 Enable detekt rule: NestedScopeFunctions
exceptions
- Enable detekt rule: TooGenericExceptionCaught #299 Enable detekt rule: TooGenericExceptionCaught
- Enable detekt rule: TooGenericExceptionThrown #300 Enable detekt rule: TooGenericExceptionThrown
- Enable detekt rule: SwallowedException #301 Enable detekt rule: SwallowedException
naming
- Enable detekt rule: MatchingDeclarationName #302 Enable detekt rule: MatchingDeclarationName
- Enable detekt rule: ConstructorParameterNaming #303 Enable detekt rule: ConstructorParameterNaming
- Enable detekt rule: FunctionNaming #304 Enable detekt rule: FunctionNaming
- Enable detekt rule: VariableNaming #305 Enable detekt rule: VariableNaming
performance
- Enable detekt rule: SpreadOperator #306 Enable detekt rule: SpreadOperator
empty-blocks
- Enable detekt rule: EmptyCatchBlock #307 Enable detekt rule: EmptyCatchBlock
potential-bugs
- Enable detekt rule: ImplicitDefaultLocale #308 Enable detekt rule: ImplicitDefaultLocale
formatting
- Enable detekt rule: ImportOrdering #309 Enable detekt rule: ImportOrdering
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
No labels