-
Notifications
You must be signed in to change notification settings - Fork 50
/
detekt.yml
315 lines (306 loc) · 7.58 KB
/
detekt.yml
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
197
198
199
200
201
202
203
204
205
206
207
208
209
210
211
212
213
214
215
216
217
218
219
220
221
222
223
224
225
226
227
228
229
230
231
232
233
234
235
236
237
238
239
240
241
242
243
244
245
246
247
248
249
250
251
252
253
254
255
256
257
258
259
260
261
262
263
264
265
266
267
268
269
270
271
272
273
274
275
276
277
278
279
280
281
282
283
284
285
286
287
288
289
290
291
292
293
294
295
296
297
298
299
300
301
302
303
304
305
306
307
308
309
310
311
312
313
314
315
build:
maxIssues: 0
processors:
active: false
potential-bugs:
active: true
DuplicateCaseInWhenExpression:
active: true
EqualsAlwaysReturnsTrueOrFalse:
active: false
EqualsWithHashCodeExist:
active: true
WrongEqualsTypeParameter:
active: false
IteratorHasNextCallsNextMethod:
active: false
ExplicitGarbageCollectionCall:
active: true
UnconditionalJumpStatementInLoop:
active: false
IteratorNotThrowingNoSuchElementException:
active: false
UnreachableCode:
active: true
LateinitUsage:
active: false
excludeAnnotatedProperties: ''
ignoreOnClassesPattern: ''
UnsafeCallOnNullableType:
active: false
UnsafeCast:
active: false
UselessPostfixExpression:
active: false
performance:
active: true
ForEachOnRange:
active: true
SpreadOperator:
active: false
UnnecessaryTemporaryInstantiation:
active: true
exceptions:
active: true
ExceptionRaisedInUnexpectedLocation:
active: false
methodNames: 'toString,hashCode,equals,finalize'
SwallowedException:
active: false
TooGenericExceptionCaught:
active: true
excludes: "**/test/**,**/androidTest/**,**/*Test.kt,**/*.Test.kt,**/*.Spec.kt,**/*.Spek.kt"
exceptionNames:
- ArrayIndexOutOfBoundsException
- Error
- Exception
- IllegalMonitorStateException
- IndexOutOfBoundsException
- InterruptedException
- NullPointerException
- RuntimeException
- Throwable
TooGenericExceptionThrown:
active: true
excludes: "**/test/**,**/androidTest/**,**/*Test.kt,**/*.Test.kt,**/*.Spec.kt,**/*.Spek.kt"
exceptionNames:
- Error
- Exception
- NullPointerException
- RuntimeException
- Throwable
InstanceOfCheckForException:
active: false
NotImplementedDeclaration:
active: true
ThrowingExceptionsWithoutMessageOrCause:
active: true
exceptions: 'IllegalArgumentException,IllegalStateException,IOException'
PrintStackTrace:
active: false
RethrowCaughtException:
active: false
ReturnFromFinally:
active: true
ThrowingExceptionFromFinally:
active: true
ThrowingExceptionInMain:
active: false
ThrowingNewInstanceOfSameException:
active: false
empty-blocks:
active: true
EmptyCatchBlock:
active: true
EmptyClassBlock:
active: false
EmptyDefaultConstructor:
active: true
EmptyDoWhileBlock:
active: true
EmptyElseBlock:
active: true
EmptyFinallyBlock:
active: true
EmptyForBlock:
active: true
EmptyFunctionBlock:
active: true
EmptyIfBlock:
active: true
EmptyInitBlock:
active: true
EmptyKtFile:
active: true
EmptySecondaryConstructor:
active: true
EmptyWhenBlock:
active: true
EmptyWhileBlock:
active: true
complexity:
active: true
LongMethod:
active: true
threshold: 60
NestedBlockDepth:
active: true
threshold: 4 # TODO: create custom rule that doesn't count run/let/apply/etc.
LongParameterList:
excludes: "**/test/**,**/androidTest/**,**/*Test.kt,**/*.Test.kt,**/*.Spec.kt,**/*.Spek.kt"
active: true
functionThreshold: 5
constructorThreshold: 5
LargeClass:
active: true
threshold: 600
ComplexInterface:
active: false
threshold: 10
includeStaticDeclarations: false
ComplexMethod:
active: false
threshold: 10
MethodOverloading:
active: false
threshold: 5
TooManyFunctions:
active: false
ComplexCondition:
active: true
threshold: 5
LabeledExpression:
active: false
StringLiteralDuplication:
active: false
threshold: 2
ignoreAnnotation: true
excludeStringsWithLessThan5Characters: true
ignoreStringsRegex: '$^'
naming:
active: true
PackageNaming:
active: true
packagePattern: '^[a-z]+(\.[a-z][a-z0-9_]*)*$'
ClassNaming:
active: true
classPattern: '[A-Z$][a-zA-Z$0-9]*'
EnumNaming:
active: true
enumEntryPattern: '^[A-Z$][a-zA-Z_$0-9]*$'
FunctionNaming:
active: true
functionPattern: '^([a-z$][a-zA-Z$0-9]*)|(`.*`)$'
ignoreAnnotated: [ 'Composable' ]
VariableNaming:
active: true
variablePattern: '^(_)?[a-z$][a-zA-Z$0-9]*$'
TopLevelPropertyNaming:
active: true
constantPattern: '[A-Z][_A-Z0-9]*'
propertyPattern: '[A-Za-z][_A-Za-z0-9]*'
privatePropertyPattern: '_?[A-Za-z][_A-Za-z0-9]*'
VariableMaxLength:
active: false
maximumVariableNameLength: 30
VariableMinLength:
active: false
minimumVariableNameLength: 3
ForbiddenClassName:
active: false
forbiddenName: ''
InvalidPackageDeclaration:
active: true
FunctionMaxLength:
active: false
maximumFunctionNameLength: 30
FunctionMinLength:
active: false
minimumFunctionNameLength: 3
MemberNameEqualsClassName:
active: false
ignoreOverridden: true
style:
active: true
ReturnCount:
active: true
max: 3
ThrowsCount:
active: true
max: 2
NewLineAtEndOfFile:
active: true
OptionalAbstractKeyword:
active: true
OptionalWhenBraces:
active: false
CollapsibleIfStatements:
active: false
EqualsNullCall:
active: false
EqualsOnSignatureLine:
active: true
ForbiddenComment:
active: true
values: 'STOPSHIP'
ForbiddenImport:
active: false
imports: ''
LoopWithTooManyJumpStatements:
active: true
maxJumpCount: 1
ModifierOrder:
active: true
MagicNumber:
active: true
excludes: "**/test/**,**/androidTest/**,**/*Test.kt,**/*.Test.kt,**/*.Spec.kt,**/*.Spek.kt"
ignoreNumbers: '-1,0,1,2'
ignoreHashCodeFunction: false
ignorePropertyDeclaration: false
ignoreConstantDeclaration: true
ignoreCompanionObjectPropertyDeclaration: true
ignoreAnnotation: false
ignoreNamedArgument: true
ignoreEnums: false
WildcardImport:
excludes: "**/test/**,**/androidTest/**,**/*Test.kt,**/*.Test.kt,**/*.Spec.kt,**/*.Spek.kt"
active: true
SafeCast:
active: true
MaxLineLength:
active: true
excludes: "**/test/**,**/androidTest/**,**/*Test.kt,**/*.Test.kt,**/*.Spec.kt,**/*.Spek.kt"
maxLineLength: 160
excludePackageStatements: false
excludeImportStatements: false
ProtectedMemberInFinalClass:
active: false
SerialVersionUIDInSerializableClass:
active: false
UnnecessaryParentheses:
active: false
UnnecessaryInheritance:
active: false
UtilityClassWithPublicConstructor:
active: false
DataClassContainsFunctions:
active: false
conversionFunctionPrefix: 'to'
UseDataClass:
active: false
UnnecessaryAbstractClass:
active: false
OptionalUnit:
active: true
ExpressionBodySyntax:
active: true
UnusedImports:
active: true
NestedClassesVisibility:
active: false # TODO: Activate when https://github.com/arturbosch/detekt/issues/1075 is solved or don't activate at all
RedundantVisibilityModifierRule:
active: true
FunctionOnlyReturningConstant:
active: true
ignoreOverridableFunction: true
excludedFunctions: 'describeContents'
LibraryCodeMustSpecifyReturnType:
active: true
excludes: "**/test/**,**/androidTest/**,**/*Test.kt,**/*.Test.kt,**/*.Spec.kt,**/*.Spek.kt,**/*InstrumentationTestUtils/**"
MandatoryBracesIfStatements:
active: false
comments:
active: true
excludes: "**/test/**,**/androidTest/**,**/*Test.kt,**/*.Test.kt,**/*.Spec.kt,**/*.Spek.kt,**/*InstrumentationTestUtils/**"
CommentOverPrivateFunction:
active: false
CommentOverPrivateProperty:
active: true
UndocumentedPublicClass:
active: false
searchInNestedClass: true
searchInInnerClass: true
searchInInnerObject: true
searchInInnerInterface: true
UndocumentedPublicFunction:
active: false