cookstyle 0.0.1 → 1.0.0
This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
- checksums.yaml +4 -4
- data/.travis.yml +9 -2
- data/README.md +29 -17
- data/Rakefile +2 -0
- data/config/cookstyle.yml +80 -4
- data/config/disable_all.yml +75 -1
- data/config/disabled.yml +25 -35
- data/config/enabled.yml +373 -137
- data/config/upstream.yml +231 -11
- data/cookstyle.gemspec +1 -1
- data/lib/cookstyle/version.rb +2 -2
- metadata +7 -7
data/config/enabled.yml
CHANGED
@@ -2,23 +2,24 @@
|
|
2
2
|
|
3
3
|
Style/AccessModifierIndentation:
|
4
4
|
Description: Check indentation of private/protected visibility modifiers.
|
5
|
-
StyleGuide: '
|
5
|
+
StyleGuide: '#indent-public-private-protected'
|
6
6
|
Enabled: true
|
7
7
|
|
8
8
|
Style/AccessorMethodName:
|
9
9
|
Description: Check the naming of accessor methods for get_/set_.
|
10
|
+
StyleGuide: '#accessor_mutator_method_names'
|
10
11
|
Enabled: true
|
11
12
|
|
12
13
|
Style/Alias:
|
13
14
|
Description: 'Use alias instead of alias_method.'
|
14
|
-
StyleGuide: '
|
15
|
+
StyleGuide: '#alias-method'
|
15
16
|
Enabled: true
|
16
17
|
|
17
18
|
Style/AlignArray:
|
18
19
|
Description: >-
|
19
20
|
Align the elements of an array literal if they span more than
|
20
21
|
one line.
|
21
|
-
StyleGuide: '
|
22
|
+
StyleGuide: '#align-multiline-arrays'
|
22
23
|
Enabled: true
|
23
24
|
|
24
25
|
Style/AlignHash:
|
@@ -31,47 +32,47 @@ Style/AlignParameters:
|
|
31
32
|
Description: >-
|
32
33
|
Align the parameters of a method call if they span more
|
33
34
|
than one line.
|
34
|
-
StyleGuide: '
|
35
|
+
StyleGuide: '#no-double-indent'
|
35
36
|
Enabled: true
|
36
37
|
|
37
38
|
Style/AndOr:
|
38
39
|
Description: 'Use &&/|| instead of and/or.'
|
39
|
-
StyleGuide: '
|
40
|
+
StyleGuide: '#no-and-or-or'
|
40
41
|
Enabled: true
|
41
42
|
|
42
43
|
Style/ArrayJoin:
|
43
44
|
Description: 'Use Array#join instead of Array#*.'
|
44
|
-
StyleGuide: '
|
45
|
+
StyleGuide: '#array-join'
|
45
46
|
Enabled: true
|
46
47
|
|
47
48
|
Style/AsciiComments:
|
48
49
|
Description: 'Use only ascii symbols in comments.'
|
49
|
-
StyleGuide: '
|
50
|
+
StyleGuide: '#english-comments'
|
50
51
|
Enabled: true
|
51
52
|
|
52
53
|
Style/AsciiIdentifiers:
|
53
54
|
Description: 'Use only ascii symbols in identifiers.'
|
54
|
-
StyleGuide: '
|
55
|
+
StyleGuide: '#english-identifiers'
|
55
56
|
Enabled: true
|
56
57
|
|
57
58
|
Style/Attr:
|
58
59
|
Description: 'Checks for uses of Module#attr.'
|
59
|
-
StyleGuide: '
|
60
|
+
StyleGuide: '#attr'
|
60
61
|
Enabled: true
|
61
62
|
|
62
63
|
Style/BeginBlock:
|
63
64
|
Description: 'Avoid the use of BEGIN blocks.'
|
64
|
-
StyleGuide: '
|
65
|
+
StyleGuide: '#no-BEGIN-blocks'
|
65
66
|
Enabled: true
|
66
67
|
|
67
68
|
Style/BarePercentLiterals:
|
68
69
|
Description: 'Checks if usage of %() or %Q() matches configuration.'
|
69
|
-
StyleGuide: '
|
70
|
+
StyleGuide: '#percent-q-shorthand'
|
70
71
|
Enabled: true
|
71
72
|
|
72
73
|
Style/BlockComments:
|
73
74
|
Description: 'Do not use block comments.'
|
74
|
-
StyleGuide: '
|
75
|
+
StyleGuide: '#no-block-comments'
|
75
76
|
Enabled: true
|
76
77
|
|
77
78
|
Style/BlockEndNewline:
|
@@ -83,7 +84,7 @@ Style/BlockDelimiters:
|
|
83
84
|
Avoid using {...} for multi-line blocks (multiline chaining is
|
84
85
|
always ugly).
|
85
86
|
Prefer {...} over do...end for single-line blocks.
|
86
|
-
StyleGuide: '
|
87
|
+
StyleGuide: '#single-line-blocks'
|
87
88
|
Enabled: true
|
88
89
|
|
89
90
|
Style/BracesAroundHashParameters:
|
@@ -92,22 +93,22 @@ Style/BracesAroundHashParameters:
|
|
92
93
|
|
93
94
|
Style/CaseEquality:
|
94
95
|
Description: 'Avoid explicit use of the case equality operator(===).'
|
95
|
-
StyleGuide: '
|
96
|
+
StyleGuide: '#no-case-equality'
|
96
97
|
Enabled: true
|
97
98
|
|
98
99
|
Style/CaseIndentation:
|
99
100
|
Description: 'Indentation of when in a case/when/[else/]end.'
|
100
|
-
StyleGuide: '
|
101
|
+
StyleGuide: '#indent-when-to-case'
|
101
102
|
Enabled: true
|
102
103
|
|
103
104
|
Style/CharacterLiteral:
|
104
105
|
Description: 'Checks for uses of character literals.'
|
105
|
-
StyleGuide: '
|
106
|
+
StyleGuide: '#no-character-literals'
|
106
107
|
Enabled: true
|
107
108
|
|
108
109
|
Style/ClassAndModuleCamelCase:
|
109
110
|
Description: 'Use CamelCase for classes and modules.'
|
110
|
-
StyleGuide: '
|
111
|
+
StyleGuide: '#camelcase-classes'
|
111
112
|
Enabled: true
|
112
113
|
|
113
114
|
Style/ClassAndModuleChildren:
|
@@ -120,12 +121,12 @@ Style/ClassCheck:
|
|
120
121
|
|
121
122
|
Style/ClassMethods:
|
122
123
|
Description: 'Use self when defining module/class methods.'
|
123
|
-
StyleGuide: '
|
124
|
+
StyleGuide: '#def-self-class-methods'
|
124
125
|
Enabled: true
|
125
126
|
|
126
127
|
Style/ClassVars:
|
127
128
|
Description: 'Avoid the use of class variables.'
|
128
|
-
StyleGuide: '
|
129
|
+
StyleGuide: '#no-class-vars'
|
129
130
|
Enabled: true
|
130
131
|
|
131
132
|
Style/ClosingParenthesisIndentation:
|
@@ -134,19 +135,19 @@ Style/ClosingParenthesisIndentation:
|
|
134
135
|
|
135
136
|
Style/ColonMethodCall:
|
136
137
|
Description: 'Do not use :: for method call.'
|
137
|
-
StyleGuide: '
|
138
|
+
StyleGuide: '#double-colons'
|
138
139
|
Enabled: true
|
139
140
|
|
140
141
|
Style/CommandLiteral:
|
141
142
|
Description: 'Use `` or %x around command literals.'
|
142
|
-
StyleGuide: '
|
143
|
+
StyleGuide: '#percent-x'
|
143
144
|
Enabled: true
|
144
145
|
|
145
146
|
Style/CommentAnnotation:
|
146
147
|
Description: >-
|
147
148
|
Checks formatting of special comments
|
148
149
|
(TODO, FIXME, OPTIMIZE, HACK, REVIEW).
|
149
|
-
StyleGuide: '
|
150
|
+
StyleGuide: '#annotate-keywords'
|
150
151
|
Enabled: true
|
151
152
|
|
152
153
|
Style/CommentIndentation:
|
@@ -162,17 +163,12 @@ Style/ConditionalAssignment:
|
|
162
163
|
|
163
164
|
Style/ConstantName:
|
164
165
|
Description: 'Constants should use SCREAMING_SNAKE_CASE.'
|
165
|
-
StyleGuide: '
|
166
|
+
StyleGuide: '#screaming-snake-case'
|
166
167
|
Enabled: true
|
167
168
|
|
168
169
|
Style/DefWithParentheses:
|
169
170
|
Description: 'Use def with parentheses when there are arguments.'
|
170
|
-
StyleGuide: '
|
171
|
-
Enabled: true
|
172
|
-
|
173
|
-
Style/DeprecatedHashMethods:
|
174
|
-
Description: 'Checks for use of deprecated Hash methods.'
|
175
|
-
StyleGuide: 'https://github.com/bbatsov/ruby-style-guide#hash-key'
|
171
|
+
StyleGuide: '#method-parens'
|
176
172
|
Enabled: true
|
177
173
|
|
178
174
|
Style/Documentation:
|
@@ -184,12 +180,18 @@ Style/Documentation:
|
|
184
180
|
|
185
181
|
Style/DotPosition:
|
186
182
|
Description: 'Checks the position of the dot in multi-line method calls.'
|
187
|
-
StyleGuide: '
|
183
|
+
StyleGuide: '#consistent-multi-line-chains'
|
188
184
|
Enabled: true
|
189
185
|
|
190
186
|
Style/DoubleNegation:
|
191
187
|
Description: 'Checks for uses of double negation (!!).'
|
192
|
-
StyleGuide: '
|
188
|
+
StyleGuide: '#no-bang-bang'
|
189
|
+
Enabled: true
|
190
|
+
|
191
|
+
Style/EachForSimpleLoop:
|
192
|
+
Description: >-
|
193
|
+
Use `Integer#times` for a simple loop which iterates a fixed
|
194
|
+
number of times.
|
193
195
|
Enabled: true
|
194
196
|
|
195
197
|
Style/EachWithObject:
|
@@ -204,9 +206,13 @@ Style/EmptyElse:
|
|
204
206
|
Description: 'Avoid empty else-clauses.'
|
205
207
|
Enabled: true
|
206
208
|
|
209
|
+
Style/EmptyCaseCondition:
|
210
|
+
Description: 'Avoid empty condition in case statements.'
|
211
|
+
Enabled: true
|
212
|
+
|
207
213
|
Style/EmptyLineBetweenDefs:
|
208
214
|
Description: 'Use empty lines between defs.'
|
209
|
-
StyleGuide: '
|
215
|
+
StyleGuide: '#empty-lines-between-methods'
|
210
216
|
Enabled: true
|
211
217
|
|
212
218
|
Style/EmptyLines:
|
@@ -235,22 +241,27 @@ Style/EmptyLinesAroundMethodBody:
|
|
235
241
|
|
236
242
|
Style/EmptyLiteral:
|
237
243
|
Description: 'Prefer literals to Array.new/Hash.new/String.new.'
|
238
|
-
StyleGuide: '
|
244
|
+
StyleGuide: '#literal-array-hash'
|
245
|
+
Enabled: true
|
246
|
+
|
247
|
+
Style/EmptyMethod:
|
248
|
+
Description: 'Checks the formatting of empty method definitions.'
|
249
|
+
StyleGuide: '#no-single-line-methods'
|
239
250
|
Enabled: true
|
240
251
|
|
241
252
|
Style/EndBlock:
|
242
253
|
Description: 'Avoid the use of END blocks.'
|
243
|
-
StyleGuide: '
|
254
|
+
StyleGuide: '#no-END-blocks'
|
244
255
|
Enabled: true
|
245
256
|
|
246
257
|
Style/EndOfLine:
|
247
258
|
Description: 'Use Unix-style line endings.'
|
248
|
-
StyleGuide: '
|
259
|
+
StyleGuide: '#crlf'
|
249
260
|
Enabled: true
|
250
261
|
|
251
262
|
Style/EvenOdd:
|
252
|
-
Description: 'Favor the use of
|
253
|
-
StyleGuide: '
|
263
|
+
Description: 'Favor the use of Integer#even? && Integer#odd?'
|
264
|
+
StyleGuide: '#predicate-methods'
|
254
265
|
Enabled: true
|
255
266
|
|
256
267
|
Style/ExtraSpacing:
|
@@ -259,7 +270,7 @@ Style/ExtraSpacing:
|
|
259
270
|
|
260
271
|
Style/FileName:
|
261
272
|
Description: 'Use snake_case for source file names.'
|
262
|
-
StyleGuide: '
|
273
|
+
StyleGuide: '#snake-case-files'
|
263
274
|
Enabled: true
|
264
275
|
|
265
276
|
Style/FrozenStringLiteralComment:
|
@@ -279,35 +290,35 @@ Style/FirstParameterIndentation:
|
|
279
290
|
|
280
291
|
Style/FlipFlop:
|
281
292
|
Description: 'Checks for flip flops'
|
282
|
-
StyleGuide: '
|
293
|
+
StyleGuide: '#no-flip-flops'
|
283
294
|
Enabled: true
|
284
295
|
|
285
296
|
Style/For:
|
286
297
|
Description: 'Checks use of for or each in multiline loops.'
|
287
|
-
StyleGuide: '
|
298
|
+
StyleGuide: '#no-for-loops'
|
288
299
|
Enabled: true
|
289
300
|
|
290
301
|
Style/FormatString:
|
291
302
|
Description: 'Enforce the use of Kernel#sprintf, Kernel#format or String#%.'
|
292
|
-
StyleGuide: '
|
303
|
+
StyleGuide: '#sprintf'
|
293
304
|
Enabled: true
|
294
305
|
|
295
306
|
Style/GlobalVars:
|
296
307
|
Description: 'Do not introduce global variables.'
|
297
|
-
StyleGuide: '
|
308
|
+
StyleGuide: '#instance-vars'
|
298
309
|
Reference: 'http://www.zenspider.com/Languages/Ruby/QuickRef.html'
|
299
310
|
Enabled: true
|
300
311
|
|
301
312
|
Style/GuardClause:
|
302
313
|
Description: 'Check for conditionals that can be replaced with guard clauses'
|
303
|
-
StyleGuide: '
|
314
|
+
StyleGuide: '#no-nested-conditionals'
|
304
315
|
Enabled: true
|
305
316
|
|
306
317
|
Style/HashSyntax:
|
307
318
|
Description: >-
|
308
319
|
Prefer Ruby 1.9 hash syntax { a: 1, b: 2 } over 1.8 syntax
|
309
320
|
{ :a => 1, :b => 2 }.
|
310
|
-
StyleGuide: '
|
321
|
+
StyleGuide: '#hash-literals'
|
311
322
|
Enabled: true
|
312
323
|
|
313
324
|
Style/IfInsideElse:
|
@@ -318,7 +329,7 @@ Style/IfUnlessModifier:
|
|
318
329
|
Description: >-
|
319
330
|
Favor modifier if/unless usage when you have a
|
320
331
|
single-line body.
|
321
|
-
StyleGuide: '
|
332
|
+
StyleGuide: '#if-as-a-modifier'
|
322
333
|
Enabled: true
|
323
334
|
|
324
335
|
Style/IfUnlessModifierOfIfUnless:
|
@@ -328,7 +339,7 @@ Style/IfUnlessModifierOfIfUnless:
|
|
328
339
|
|
329
340
|
Style/IfWithSemicolon:
|
330
341
|
Description: 'Do not use if x; .... Use the ternary operator instead.'
|
331
|
-
StyleGuide: '
|
342
|
+
StyleGuide: '#no-semicolon-ifs'
|
332
343
|
Enabled: true
|
333
344
|
|
334
345
|
Style/IndentationConsistency:
|
@@ -337,7 +348,7 @@ Style/IndentationConsistency:
|
|
337
348
|
|
338
349
|
Style/IndentationWidth:
|
339
350
|
Description: 'Use 2 spaces for indentation.'
|
340
|
-
StyleGuide: '
|
351
|
+
StyleGuide: '#spaces-indentation'
|
341
352
|
Enabled: true
|
342
353
|
|
343
354
|
Style/IdenticalConditionalBranches:
|
@@ -365,22 +376,26 @@ Style/IndentHash:
|
|
365
376
|
|
366
377
|
Style/InfiniteLoop:
|
367
378
|
Description: 'Use Kernel#loop for infinite loops.'
|
368
|
-
StyleGuide: '
|
379
|
+
StyleGuide: '#infinite-loop'
|
369
380
|
Enabled: true
|
370
381
|
|
371
382
|
Style/Lambda:
|
372
383
|
Description: 'Use the new lambda literal syntax for single-line blocks.'
|
373
|
-
StyleGuide: '
|
384
|
+
StyleGuide: '#lambda-multi-line'
|
385
|
+
Enabled: true
|
386
|
+
|
387
|
+
Style/SpaceInLambdaLiteral:
|
388
|
+
Description: 'Checks for spaces in lambda literals.'
|
374
389
|
Enabled: true
|
375
390
|
|
376
391
|
Style/LambdaCall:
|
377
392
|
Description: 'Use lambda.call(...) instead of lambda.(...).'
|
378
|
-
StyleGuide: '
|
393
|
+
StyleGuide: '#proc-call'
|
379
394
|
Enabled: true
|
380
395
|
|
381
396
|
Style/LeadingCommentSpace:
|
382
397
|
Description: 'Comments should start with a space.'
|
383
|
-
StyleGuide: '
|
398
|
+
StyleGuide: '#hash-space'
|
384
399
|
Enabled: true
|
385
400
|
|
386
401
|
Style/LineEndConcatenation:
|
@@ -391,38 +406,73 @@ Style/LineEndConcatenation:
|
|
391
406
|
|
392
407
|
Style/MethodCallParentheses:
|
393
408
|
Description: 'Do not use parentheses for method calls with no arguments.'
|
394
|
-
StyleGuide: '
|
409
|
+
StyleGuide: '#method-invocation-parens'
|
395
410
|
Enabled: true
|
396
411
|
|
397
412
|
Style/MethodDefParentheses:
|
398
413
|
Description: >-
|
399
414
|
Checks if the method definitions have or don't have
|
400
415
|
parentheses.
|
401
|
-
StyleGuide: '
|
416
|
+
StyleGuide: '#method-parens'
|
402
417
|
Enabled: true
|
403
418
|
|
404
419
|
Style/MethodName:
|
405
420
|
Description: 'Use the configured style when naming methods.'
|
406
|
-
StyleGuide: '
|
421
|
+
StyleGuide: '#snake-case-symbols-methods-vars'
|
422
|
+
Enabled: true
|
423
|
+
|
424
|
+
Style/MethodMissing:
|
425
|
+
Description: 'Avoid using `method_missing`.'
|
426
|
+
StyleGuide: '#no-method-missing'
|
407
427
|
Enabled: true
|
408
428
|
|
409
429
|
Style/ModuleFunction:
|
410
430
|
Description: 'Checks for usage of `extend self` in modules.'
|
411
|
-
StyleGuide: '
|
431
|
+
StyleGuide: '#module-function'
|
432
|
+
Enabled: true
|
433
|
+
|
434
|
+
Style/MultilineArrayBraceLayout:
|
435
|
+
Description: >-
|
436
|
+
Checks that the closing brace in an array literal is
|
437
|
+
either on the same line as the last array element, or
|
438
|
+
a new line.
|
412
439
|
Enabled: true
|
413
440
|
|
414
441
|
Style/MultilineBlockChain:
|
415
442
|
Description: 'Avoid multi-line chains of blocks.'
|
416
|
-
StyleGuide: '
|
443
|
+
StyleGuide: '#single-line-blocks'
|
417
444
|
Enabled: true
|
418
445
|
|
419
446
|
Style/MultilineBlockLayout:
|
420
447
|
Description: 'Ensures newlines after multiline block do statements.'
|
421
448
|
Enabled: true
|
422
449
|
|
450
|
+
Style/MultilineHashBraceLayout:
|
451
|
+
Description: >-
|
452
|
+
Checks that the closing brace in a hash literal is
|
453
|
+
either on the same line as the last hash element, or
|
454
|
+
a new line.
|
455
|
+
Enabled: true
|
456
|
+
|
423
457
|
Style/MultilineIfThen:
|
424
458
|
Description: 'Do not use then for multi-line if/unless.'
|
425
|
-
StyleGuide: '
|
459
|
+
StyleGuide: '#no-then'
|
460
|
+
Enabled: true
|
461
|
+
|
462
|
+
Style/MultilineIfModifier:
|
463
|
+
Description: 'Only use if/unless modifiers on single line statements.'
|
464
|
+
StyleGuide: '#no-multiline-if-modifiers'
|
465
|
+
Enabled: true
|
466
|
+
|
467
|
+
Style/MultilineMemoization:
|
468
|
+
Description: 'Wrap multiline memoizations in a `begin` and `end` block.'
|
469
|
+
Enabled: true
|
470
|
+
|
471
|
+
Style/MultilineMethodCallBraceLayout:
|
472
|
+
Description: >-
|
473
|
+
Checks that the closing brace in a method call is
|
474
|
+
either on the same line as the last method argument, or
|
475
|
+
a new line.
|
426
476
|
Enabled: true
|
427
477
|
|
428
478
|
Style/MultilineMethodCallIndentation:
|
@@ -431,6 +481,13 @@ Style/MultilineMethodCallIndentation:
|
|
431
481
|
that span more than one line.
|
432
482
|
Enabled: true
|
433
483
|
|
484
|
+
Style/MultilineMethodDefinitionBraceLayout:
|
485
|
+
Description: >-
|
486
|
+
Checks that the closing brace in a method definition is
|
487
|
+
either on the same line as the last method parameter, or
|
488
|
+
a new line.
|
489
|
+
Enabled: true
|
490
|
+
|
434
491
|
Style/MultilineOperationIndentation:
|
435
492
|
Description: >-
|
436
493
|
Checks indentation of binary operations that span more than
|
@@ -441,7 +498,7 @@ Style/MultilineTernaryOperator:
|
|
441
498
|
Description: >-
|
442
499
|
Avoid multi-line ?: (the ternary operator);
|
443
500
|
use if/unless instead.
|
444
|
-
StyleGuide: '
|
501
|
+
StyleGuide: '#no-multiline-ternary'
|
445
502
|
Enabled: true
|
446
503
|
|
447
504
|
Style/MutableConstant:
|
@@ -452,17 +509,17 @@ Style/NegatedIf:
|
|
452
509
|
Description: >-
|
453
510
|
Favor unless over if for negative conditions
|
454
511
|
(or control flow or).
|
455
|
-
StyleGuide: '
|
512
|
+
StyleGuide: '#unless-for-negatives'
|
456
513
|
Enabled: true
|
457
514
|
|
458
515
|
Style/NegatedWhile:
|
459
516
|
Description: 'Favor until over while for negative conditions.'
|
460
|
-
StyleGuide: '
|
517
|
+
StyleGuide: '#until-for-negatives'
|
461
518
|
Enabled: true
|
462
519
|
|
463
520
|
Style/NestedModifier:
|
464
521
|
Description: 'Avoid using nested modifiers.'
|
465
|
-
StyleGuide: '
|
522
|
+
StyleGuide: '#no-nested-modifiers'
|
466
523
|
Enabled: true
|
467
524
|
|
468
525
|
Style/NestedParenthesizedCalls:
|
@@ -473,53 +530,69 @@ Style/NestedParenthesizedCalls:
|
|
473
530
|
|
474
531
|
Style/NestedTernaryOperator:
|
475
532
|
Description: 'Use one expression per branch in a ternary operator.'
|
476
|
-
StyleGuide: '
|
533
|
+
StyleGuide: '#no-nested-ternary'
|
477
534
|
Enabled: true
|
478
535
|
|
479
536
|
Style/Next:
|
480
537
|
Description: 'Use `next` to skip iteration instead of a condition at the end.'
|
481
|
-
StyleGuide: '
|
538
|
+
StyleGuide: '#no-nested-conditionals'
|
482
539
|
Enabled: true
|
483
540
|
|
484
541
|
Style/NilComparison:
|
485
542
|
Description: 'Prefer x.nil? to x == nil.'
|
486
|
-
StyleGuide: '
|
543
|
+
StyleGuide: '#predicate-methods'
|
487
544
|
Enabled: true
|
488
545
|
|
489
546
|
Style/NonNilCheck:
|
490
547
|
Description: 'Checks for redundant nil checks.'
|
491
|
-
StyleGuide: '
|
548
|
+
StyleGuide: '#no-non-nil-checks'
|
492
549
|
Enabled: true
|
493
550
|
|
494
551
|
Style/Not:
|
495
552
|
Description: 'Use ! instead of not.'
|
496
|
-
StyleGuide: '
|
553
|
+
StyleGuide: '#bang-not-not'
|
497
554
|
Enabled: true
|
498
555
|
|
499
556
|
Style/NumericLiterals:
|
500
557
|
Description: >-
|
501
558
|
Add underscores to large numeric literals to improve their
|
502
559
|
readability.
|
503
|
-
StyleGuide: '
|
560
|
+
StyleGuide: '#underscores-in-numerics'
|
561
|
+
Enabled: true
|
562
|
+
|
563
|
+
Style/NumericLiteralPrefix:
|
564
|
+
Description: 'Use smallcase prefixes for numeric literals.'
|
565
|
+
StyleGuide: '#numeric-literal-prefixes'
|
566
|
+
Enabled: true
|
567
|
+
|
568
|
+
Style/NumericPredicate:
|
569
|
+
Description: >-
|
570
|
+
Checks for the use of predicate- or comparison methods for
|
571
|
+
numeric comparisons.
|
572
|
+
StyleGuide: '#predicate-methods'
|
573
|
+
# This will change to a new method call which isn't guaranteed to be on the
|
574
|
+
# object. Switching these methods has to be done with knowledge of the types
|
575
|
+
# of the variables which rubocop doesn't have.
|
576
|
+
AutoCorrect: false
|
504
577
|
Enabled: true
|
505
578
|
|
506
579
|
Style/OneLineConditional:
|
507
580
|
Description: >-
|
508
581
|
Favor the ternary operator(?:) over
|
509
582
|
if/then/else/end constructs.
|
510
|
-
StyleGuide: '
|
583
|
+
StyleGuide: '#ternary-operator'
|
511
584
|
Enabled: true
|
512
585
|
|
513
586
|
Style/OpMethod:
|
514
587
|
Description: 'When defining binary operators, name the argument other.'
|
515
|
-
StyleGuide: '
|
588
|
+
StyleGuide: '#other-arg'
|
516
589
|
Enabled: true
|
517
590
|
|
518
591
|
Style/OptionalArguments:
|
519
592
|
Description: >-
|
520
593
|
Checks for optional arguments that do not appear at the end
|
521
594
|
of the argument list
|
522
|
-
StyleGuide: '
|
595
|
+
StyleGuide: '#optional-arguments'
|
523
596
|
Enabled: true
|
524
597
|
|
525
598
|
Style/ParallelAssignment:
|
@@ -527,19 +600,19 @@ Style/ParallelAssignment:
|
|
527
600
|
Check for simple usages of parallel assignment.
|
528
601
|
It will only warn when the number of variables
|
529
602
|
matches on both sides of the assignment.
|
530
|
-
StyleGuide: '
|
603
|
+
StyleGuide: '#parallel-assignment'
|
531
604
|
Enabled: true
|
532
605
|
|
533
606
|
Style/ParenthesesAroundCondition:
|
534
607
|
Description: >-
|
535
608
|
Don't use parentheses around the condition of an
|
536
609
|
if/unless/while.
|
537
|
-
StyleGuide: '
|
610
|
+
StyleGuide: '#no-parens-around-condition'
|
538
611
|
Enabled: true
|
539
612
|
|
540
613
|
Style/PercentLiteralDelimiters:
|
541
614
|
Description: 'Use `%`-literal delimiters consistently'
|
542
|
-
StyleGuide: '
|
615
|
+
StyleGuide: '#percent-literal-braces'
|
543
616
|
Enabled: true
|
544
617
|
|
545
618
|
Style/PercentQLiterals:
|
@@ -548,32 +621,37 @@ Style/PercentQLiterals:
|
|
548
621
|
|
549
622
|
Style/PerlBackrefs:
|
550
623
|
Description: 'Avoid Perl-style regex back references.'
|
551
|
-
StyleGuide: '
|
624
|
+
StyleGuide: '#no-perl-regexp-last-matchers'
|
552
625
|
Enabled: true
|
553
626
|
|
554
627
|
Style/PredicateName:
|
555
628
|
Description: 'Check the names of predicate methods.'
|
556
|
-
StyleGuide: '
|
629
|
+
StyleGuide: '#bool-methods-qmark'
|
630
|
+
Enabled: true
|
631
|
+
|
632
|
+
Style/PreferredHashMethods:
|
633
|
+
Description: 'Checks use of `has_key?` and `has_value?` Hash methods.'
|
634
|
+
StyleGuide: '#hash-key'
|
557
635
|
Enabled: true
|
558
636
|
|
559
637
|
Style/Proc:
|
560
638
|
Description: 'Use proc instead of Proc.new.'
|
561
|
-
StyleGuide: '
|
639
|
+
StyleGuide: '#proc'
|
562
640
|
Enabled: true
|
563
641
|
|
564
642
|
Style/RaiseArgs:
|
565
643
|
Description: 'Checks the arguments passed to raise/fail.'
|
566
|
-
StyleGuide: '
|
644
|
+
StyleGuide: '#exception-class-messages'
|
567
645
|
Enabled: true
|
568
646
|
|
569
647
|
Style/RedundantBegin:
|
570
648
|
Description: "Don't use begin blocks when they are not needed."
|
571
|
-
StyleGuide: '
|
649
|
+
StyleGuide: '#begin-implicit'
|
572
650
|
Enabled: true
|
573
651
|
|
574
652
|
Style/RedundantException:
|
575
653
|
Description: "Checks for an obsolete RuntimeException argument in raise/fail."
|
576
|
-
StyleGuide: '
|
654
|
+
StyleGuide: '#no-explicit-runtimeerror'
|
577
655
|
Enabled: true
|
578
656
|
|
579
657
|
Style/RedundantFreeze:
|
@@ -586,17 +664,17 @@ Style/RedundantParentheses:
|
|
586
664
|
|
587
665
|
Style/RedundantReturn:
|
588
666
|
Description: "Don't use return where it's not required."
|
589
|
-
StyleGuide: '
|
667
|
+
StyleGuide: '#no-explicit-return'
|
590
668
|
Enabled: true
|
591
669
|
|
592
670
|
Style/RedundantSelf:
|
593
671
|
Description: "Don't use self where it's not needed."
|
594
|
-
StyleGuide: '
|
672
|
+
StyleGuide: '#no-self-unless-required'
|
595
673
|
Enabled: true
|
596
674
|
|
597
675
|
Style/RegexpLiteral:
|
598
676
|
Description: 'Use / or %r around regular expressions.'
|
599
|
-
StyleGuide: '
|
677
|
+
StyleGuide: '#percent-r'
|
600
678
|
Enabled: true
|
601
679
|
|
602
680
|
Style/RescueEnsureAlignment:
|
@@ -605,34 +683,40 @@ Style/RescueEnsureAlignment:
|
|
605
683
|
|
606
684
|
Style/RescueModifier:
|
607
685
|
Description: 'Avoid using rescue in its modifier form.'
|
608
|
-
StyleGuide: '
|
686
|
+
StyleGuide: '#no-rescue-modifiers'
|
687
|
+
Enabled: true
|
688
|
+
|
689
|
+
Style/SafeNavigation:
|
690
|
+
Description: >-
|
691
|
+
This cop transforms usages of a method call safeguarded by
|
692
|
+
a check for the existance of the object to
|
693
|
+
safe navigation (`&.`).
|
609
694
|
Enabled: true
|
610
695
|
|
611
696
|
Style/SelfAssignment:
|
612
697
|
Description: >-
|
613
698
|
Checks for places where self-assignment shorthand should have
|
614
699
|
been used.
|
615
|
-
StyleGuide: '
|
700
|
+
StyleGuide: '#self-assignment'
|
616
701
|
Enabled: true
|
617
702
|
|
618
703
|
Style/Semicolon:
|
619
704
|
Description: "Don't use semicolons to terminate expressions."
|
620
|
-
StyleGuide: '
|
705
|
+
StyleGuide: '#no-semicolon'
|
621
706
|
Enabled: true
|
622
707
|
|
623
708
|
Style/SignalException:
|
624
709
|
Description: 'Checks for proper usage of fail and raise.'
|
625
|
-
StyleGuide: '
|
710
|
+
StyleGuide: '#prefer-raise-over-fail'
|
626
711
|
Enabled: true
|
627
712
|
|
628
713
|
Style/SingleLineBlockParams:
|
629
714
|
Description: 'Enforces the names of some block params.'
|
630
|
-
StyleGuide: 'https://github.com/bbatsov/ruby-style-guide#reduce-blocks'
|
631
715
|
Enabled: true
|
632
716
|
|
633
717
|
Style/SingleLineMethods:
|
634
718
|
Description: 'Avoid single-line methods.'
|
635
|
-
StyleGuide: '
|
719
|
+
StyleGuide: '#no-single-line-methods'
|
636
720
|
Enabled: true
|
637
721
|
|
638
722
|
Style/SpaceBeforeFirstArg:
|
@@ -643,29 +727,29 @@ Style/SpaceBeforeFirstArg:
|
|
643
727
|
|
644
728
|
Style/SpaceAfterColon:
|
645
729
|
Description: 'Use spaces after colons.'
|
646
|
-
StyleGuide: '
|
730
|
+
StyleGuide: '#spaces-operators'
|
647
731
|
Enabled: true
|
648
732
|
|
649
733
|
Style/SpaceAfterComma:
|
650
734
|
Description: 'Use spaces after commas.'
|
651
|
-
StyleGuide: '
|
735
|
+
StyleGuide: '#spaces-operators'
|
652
736
|
Enabled: true
|
653
737
|
|
654
738
|
Style/SpaceAfterMethodName:
|
655
739
|
Description: >-
|
656
740
|
Do not put a space between a method name and the opening
|
657
741
|
parenthesis in a method definition.
|
658
|
-
StyleGuide: '
|
742
|
+
StyleGuide: '#parens-no-spaces'
|
659
743
|
Enabled: true
|
660
744
|
|
661
745
|
Style/SpaceAfterNot:
|
662
746
|
Description: Tracks redundant space after the ! operator.
|
663
|
-
StyleGuide: '
|
747
|
+
StyleGuide: '#no-space-bang'
|
664
748
|
Enabled: true
|
665
749
|
|
666
750
|
Style/SpaceAfterSemicolon:
|
667
751
|
Description: 'Use spaces after semicolons.'
|
668
|
-
StyleGuide: '
|
752
|
+
StyleGuide: '#spaces-operators'
|
669
753
|
Enabled: true
|
670
754
|
|
671
755
|
Style/SpaceBeforeBlockBraces:
|
@@ -704,7 +788,7 @@ Style/SpaceAroundEqualsInParameterDefault:
|
|
704
788
|
Checks that the equals signs in parameter default assignments
|
705
789
|
have or don't have surrounding space depending on
|
706
790
|
configuration.
|
707
|
-
StyleGuide: '
|
791
|
+
StyleGuide: '#spaces-around-equals'
|
708
792
|
Enabled: true
|
709
793
|
|
710
794
|
Style/SpaceAroundKeyword:
|
@@ -713,47 +797,55 @@ Style/SpaceAroundKeyword:
|
|
713
797
|
|
714
798
|
Style/SpaceAroundOperators:
|
715
799
|
Description: 'Use a single space around operators.'
|
716
|
-
StyleGuide: '
|
800
|
+
StyleGuide: '#spaces-operators'
|
801
|
+
Enabled: true
|
802
|
+
|
803
|
+
Style/SpaceInsideArrayPercentLiteral:
|
804
|
+
Description: 'No unnecessary additional spaces between elements in %i/%w literals.'
|
805
|
+
Enabled: true
|
806
|
+
|
807
|
+
Style/SpaceInsidePercentLiteralDelimiters:
|
808
|
+
Description: 'No unnecessary spaces inside delimiters of %i/%w/%x literals.'
|
717
809
|
Enabled: true
|
718
810
|
|
719
811
|
Style/SpaceInsideBrackets:
|
720
812
|
Description: 'No spaces after [ or before ].'
|
721
|
-
StyleGuide: '
|
813
|
+
StyleGuide: '#no-spaces-braces'
|
722
814
|
Enabled: true
|
723
815
|
|
724
816
|
Style/SpaceInsideHashLiteralBraces:
|
725
817
|
Description: "Use spaces inside hash literal braces - or don't."
|
726
|
-
StyleGuide: '
|
818
|
+
StyleGuide: '#spaces-operators'
|
727
819
|
Enabled: true
|
728
820
|
|
729
821
|
Style/SpaceInsideParens:
|
730
822
|
Description: 'No spaces after ( or before ).'
|
731
|
-
StyleGuide: '
|
823
|
+
StyleGuide: '#no-spaces-braces'
|
732
824
|
Enabled: true
|
733
825
|
|
734
826
|
Style/SpaceInsideRangeLiteral:
|
735
827
|
Description: 'No spaces inside range literals.'
|
736
|
-
StyleGuide: '
|
828
|
+
StyleGuide: '#no-space-inside-range-literals'
|
737
829
|
Enabled: true
|
738
830
|
|
739
831
|
Style/SpaceInsideStringInterpolation:
|
740
832
|
Description: 'Checks for padding/surrounding spaces inside string interpolation.'
|
741
|
-
StyleGuide: '
|
833
|
+
StyleGuide: '#string-interpolation'
|
742
834
|
Enabled: true
|
743
835
|
|
744
836
|
Style/SpecialGlobalVars:
|
745
837
|
Description: 'Avoid Perl-style global variables.'
|
746
|
-
StyleGuide: '
|
838
|
+
StyleGuide: '#no-cryptic-perlisms'
|
747
839
|
Enabled: true
|
748
840
|
|
749
841
|
Style/StabbyLambdaParentheses:
|
750
842
|
Description: 'Check for the usage of parentheses around stabby lambda arguments.'
|
751
|
-
StyleGuide: '
|
843
|
+
StyleGuide: '#stabby-lambda-with-args'
|
752
844
|
Enabled: true
|
753
845
|
|
754
846
|
Style/StringLiterals:
|
755
847
|
Description: 'Checks if uses of quotes match the configured preference.'
|
756
|
-
StyleGuide: '
|
848
|
+
StyleGuide: '#consistent-string-literals'
|
757
849
|
Enabled: true
|
758
850
|
|
759
851
|
Style/StringLiteralsInInterpolation:
|
@@ -764,7 +856,7 @@ Style/StringLiteralsInInterpolation:
|
|
764
856
|
|
765
857
|
Style/StructInheritance:
|
766
858
|
Description: 'Checks for inheritance from Struct.new.'
|
767
|
-
StyleGuide: '
|
859
|
+
StyleGuide: '#no-extend-struct-new'
|
768
860
|
Enabled: true
|
769
861
|
|
770
862
|
Style/SymbolLiteral:
|
@@ -777,39 +869,43 @@ Style/SymbolProc:
|
|
777
869
|
|
778
870
|
Style/Tab:
|
779
871
|
Description: 'No hard tabs.'
|
780
|
-
StyleGuide: '
|
872
|
+
StyleGuide: '#spaces-indentation'
|
873
|
+
Enabled: true
|
874
|
+
|
875
|
+
Style/TernaryParentheses:
|
876
|
+
Description: 'Checks for use of parentheses around ternary conditions.'
|
781
877
|
Enabled: true
|
782
878
|
|
783
879
|
Style/TrailingBlankLines:
|
784
880
|
Description: 'Checks trailing blank lines and final newline.'
|
785
|
-
StyleGuide: '
|
881
|
+
StyleGuide: '#newline-eof'
|
786
882
|
Enabled: true
|
787
883
|
|
788
884
|
Style/TrailingCommaInArguments:
|
789
885
|
Description: 'Checks for trailing comma in argument lists.'
|
790
|
-
StyleGuide: '
|
886
|
+
StyleGuide: '#no-trailing-params-comma'
|
791
887
|
Enabled: true
|
792
888
|
|
793
889
|
Style/TrailingCommaInLiteral:
|
794
890
|
Description: 'Checks for trailing comma in array and hash literals.'
|
795
|
-
StyleGuide: '
|
891
|
+
StyleGuide: '#no-trailing-array-commas'
|
796
892
|
Enabled: true
|
797
893
|
|
798
894
|
Style/TrailingWhitespace:
|
799
895
|
Description: 'Avoid trailing whitespace.'
|
800
|
-
StyleGuide: '
|
896
|
+
StyleGuide: '#no-trailing-whitespace'
|
801
897
|
Enabled: true
|
802
898
|
|
803
899
|
Style/TrivialAccessors:
|
804
900
|
Description: 'Prefer attr_* methods to trivial readers/writers.'
|
805
|
-
StyleGuide: '
|
901
|
+
StyleGuide: '#attr_family'
|
806
902
|
Enabled: true
|
807
903
|
|
808
904
|
Style/UnlessElse:
|
809
905
|
Description: >-
|
810
906
|
Do not use unless with else. Rewrite these with the positive
|
811
907
|
case first.
|
812
|
-
StyleGuide: '
|
908
|
+
StyleGuide: '#no-else-with-unless'
|
813
909
|
Enabled: true
|
814
910
|
|
815
911
|
Style/UnneededCapitalW:
|
@@ -822,7 +918,7 @@ Style/UnneededInterpolation:
|
|
822
918
|
|
823
919
|
Style/UnneededPercentQ:
|
824
920
|
Description: 'Checks for %q/%Q when single quotes or double quotes would do.'
|
825
|
-
StyleGuide: '
|
921
|
+
StyleGuide: '#percent-q'
|
826
922
|
Enabled: true
|
827
923
|
|
828
924
|
Style/TrailingUnderscoreVariable:
|
@@ -836,34 +932,38 @@ Style/VariableInterpolation:
|
|
836
932
|
Description: >-
|
837
933
|
Don't interpolate global, instance and class variables
|
838
934
|
directly in strings.
|
839
|
-
StyleGuide: '
|
935
|
+
StyleGuide: '#curlies-interpolate'
|
840
936
|
Enabled: true
|
841
937
|
|
842
938
|
Style/VariableName:
|
843
939
|
Description: 'Use the configured style when naming variables.'
|
844
|
-
StyleGuide: '
|
940
|
+
StyleGuide: '#snake-case-symbols-methods-vars'
|
941
|
+
Enabled: true
|
942
|
+
|
943
|
+
Style/VariableNumber:
|
944
|
+
Description: 'Use the configured style when numbering variables.'
|
845
945
|
Enabled: true
|
846
946
|
|
847
947
|
Style/WhenThen:
|
848
948
|
Description: 'Use when x then ... for one-line cases.'
|
849
|
-
StyleGuide: '
|
949
|
+
StyleGuide: '#one-line-cases'
|
850
950
|
Enabled: true
|
851
951
|
|
852
952
|
Style/WhileUntilDo:
|
853
953
|
Description: 'Checks for redundant do after while or until.'
|
854
|
-
StyleGuide: '
|
954
|
+
StyleGuide: '#no-multiline-while-do'
|
855
955
|
Enabled: true
|
856
956
|
|
857
957
|
Style/WhileUntilModifier:
|
858
958
|
Description: >-
|
859
959
|
Favor modifier while/until usage when you have a
|
860
960
|
single-line body.
|
861
|
-
StyleGuide: '
|
961
|
+
StyleGuide: '#while-as-a-modifier'
|
862
962
|
Enabled: true
|
863
963
|
|
864
964
|
Style/WordArray:
|
865
965
|
Description: 'Use %w or %W for arrays of words.'
|
866
|
-
StyleGuide: '
|
966
|
+
StyleGuide: '#percent-w'
|
867
967
|
Enabled: true
|
868
968
|
|
869
969
|
Style/ZeroLengthPredicate:
|
@@ -881,7 +981,7 @@ Metrics/AbcSize:
|
|
881
981
|
|
882
982
|
Metrics/BlockNesting:
|
883
983
|
Description: 'Avoid excessive block nesting'
|
884
|
-
StyleGuide: '
|
984
|
+
StyleGuide: '#three-is-the-number-thou-shalt-count'
|
885
985
|
Enabled: true
|
886
986
|
|
887
987
|
Metrics/ClassLength:
|
@@ -900,17 +1000,21 @@ Metrics/CyclomaticComplexity:
|
|
900
1000
|
|
901
1001
|
Metrics/LineLength:
|
902
1002
|
Description: 'Limit lines to 80 characters.'
|
903
|
-
StyleGuide: '
|
1003
|
+
StyleGuide: '#80-character-limits'
|
904
1004
|
Enabled: true
|
905
1005
|
|
906
1006
|
Metrics/MethodLength:
|
907
1007
|
Description: 'Avoid methods longer than 10 lines of code.'
|
908
|
-
StyleGuide: '
|
1008
|
+
StyleGuide: '#short-methods'
|
1009
|
+
Enabled: true
|
1010
|
+
|
1011
|
+
Metrics/BlockLength:
|
1012
|
+
Description: 'Avoid long blocks with many lines.'
|
909
1013
|
Enabled: true
|
910
1014
|
|
911
1015
|
Metrics/ParameterLists:
|
912
1016
|
Description: 'Avoid parameter lists longer than three or four parameters.'
|
913
|
-
StyleGuide: '
|
1017
|
+
StyleGuide: '#too-many-params'
|
914
1018
|
Enabled: true
|
915
1019
|
|
916
1020
|
Metrics/PerceivedComplexity:
|
@@ -926,7 +1030,7 @@ Lint/AmbiguousOperator:
|
|
926
1030
|
Description: >-
|
927
1031
|
Checks for ambiguous operators in the first argument of a
|
928
1032
|
method invocation without parentheses.
|
929
|
-
StyleGuide: '
|
1033
|
+
StyleGuide: '#method-invocation-parens'
|
930
1034
|
Enabled: true
|
931
1035
|
|
932
1036
|
Lint/AmbiguousRegexpLiteral:
|
@@ -937,7 +1041,7 @@ Lint/AmbiguousRegexpLiteral:
|
|
937
1041
|
|
938
1042
|
Lint/AssignmentInCondition:
|
939
1043
|
Description: "Don't use assignment in conditions."
|
940
|
-
StyleGuide: '
|
1044
|
+
StyleGuide: '#safe-assignment-in-condition'
|
941
1045
|
Enabled: true
|
942
1046
|
|
943
1047
|
Lint/BlockAlignment:
|
@@ -952,7 +1056,7 @@ Lint/ConditionPosition:
|
|
952
1056
|
Description: >-
|
953
1057
|
Checks for condition placed in a confusing position relative to
|
954
1058
|
the keyword.
|
955
|
-
StyleGuide: '
|
1059
|
+
StyleGuide: '#same-line-condition'
|
956
1060
|
Enabled: true
|
957
1061
|
|
958
1062
|
Lint/Debugger:
|
@@ -967,6 +1071,10 @@ Lint/DeprecatedClassMethods:
|
|
967
1071
|
Description: 'Check for deprecated class method calls.'
|
968
1072
|
Enabled: true
|
969
1073
|
|
1074
|
+
Lint/DuplicateCaseCondition:
|
1075
|
+
Description: 'Do not repeat values in case conditionals.'
|
1076
|
+
Enabled: true
|
1077
|
+
|
970
1078
|
Lint/DuplicateMethods:
|
971
1079
|
Description: 'Check for duplicate method definitions.'
|
972
1080
|
Enabled: true
|
@@ -987,10 +1095,18 @@ Lint/EmptyEnsure:
|
|
987
1095
|
Description: 'Checks for empty ensure block.'
|
988
1096
|
Enabled: true
|
989
1097
|
|
1098
|
+
Lint/EmptyExpression:
|
1099
|
+
Description: 'Checks for empty expressions.'
|
1100
|
+
Enabled: true
|
1101
|
+
|
990
1102
|
Lint/EmptyInterpolation:
|
991
1103
|
Description: 'Checks for empty string interpolation.'
|
992
1104
|
Enabled: true
|
993
1105
|
|
1106
|
+
Lint/EmptyWhen:
|
1107
|
+
Description: 'Checks for `when` branches with empty bodies.'
|
1108
|
+
Enabled: true
|
1109
|
+
|
994
1110
|
Lint/EndAlignment:
|
995
1111
|
Description: 'Align ends correctly.'
|
996
1112
|
Enabled: true
|
@@ -1001,7 +1117,7 @@ Lint/EndInMethod:
|
|
1001
1117
|
|
1002
1118
|
Lint/EnsureReturn:
|
1003
1119
|
Description: 'Do not use return in an ensure block.'
|
1004
|
-
StyleGuide: '
|
1120
|
+
StyleGuide: '#no-return-ensure'
|
1005
1121
|
Enabled: true
|
1006
1122
|
|
1007
1123
|
Lint/Eval:
|
@@ -1020,7 +1136,7 @@ Lint/FormatParameterMismatch:
|
|
1020
1136
|
|
1021
1137
|
Lint/HandleExceptions:
|
1022
1138
|
Description: "Don't suppress exception."
|
1023
|
-
StyleGuide: '
|
1139
|
+
StyleGuide: '#dont-hide-exceptions'
|
1024
1140
|
Enabled: true
|
1025
1141
|
|
1026
1142
|
Lint/ImplicitStringConcatenation:
|
@@ -1035,6 +1151,10 @@ Lint/IneffectiveAccessModifier:
|
|
1035
1151
|
the visibility of a class method, which does not work.
|
1036
1152
|
Enabled: true
|
1037
1153
|
|
1154
|
+
Lint/InheritException:
|
1155
|
+
Description: 'Avoid inheriting from the `Exception` class.'
|
1156
|
+
Enabled: true
|
1157
|
+
|
1038
1158
|
Lint/InvalidCharacterLiteral:
|
1039
1159
|
Description: >-
|
1040
1160
|
Checks for invalid character literals with a non-escaped
|
@@ -1053,12 +1173,12 @@ Lint/Loop:
|
|
1053
1173
|
Description: >-
|
1054
1174
|
Use Kernel#loop with break rather than begin/end/until or
|
1055
1175
|
begin/end/while for post-loop tests.
|
1056
|
-
StyleGuide: '
|
1176
|
+
StyleGuide: '#loop-with-break'
|
1057
1177
|
Enabled: true
|
1058
1178
|
|
1059
1179
|
Lint/NestedMethodDefinition:
|
1060
1180
|
Description: 'Do not use nested method definitions.'
|
1061
|
-
StyleGuide: '
|
1181
|
+
StyleGuide: '#no-nested-methods'
|
1062
1182
|
Enabled: true
|
1063
1183
|
|
1064
1184
|
Lint/NextWithoutAccumulator:
|
@@ -1075,7 +1195,17 @@ Lint/ParenthesesAsGroupedExpression:
|
|
1075
1195
|
Description: >-
|
1076
1196
|
Checks for method calls with a space before the opening
|
1077
1197
|
parenthesis.
|
1078
|
-
StyleGuide: '
|
1198
|
+
StyleGuide: '#parens-no-spaces'
|
1199
|
+
Enabled: true
|
1200
|
+
|
1201
|
+
Lint/PercentStringArray:
|
1202
|
+
Description: >-
|
1203
|
+
Checks for unwanted commas and quotes in %w/%W literals.
|
1204
|
+
Enabled: true
|
1205
|
+
|
1206
|
+
Lint/PercentSymbolArray:
|
1207
|
+
Description: >-
|
1208
|
+
Checks for unwanted commas and colons in %i/%I literals.
|
1079
1209
|
Enabled: true
|
1080
1210
|
|
1081
1211
|
Lint/RandOne:
|
@@ -1092,7 +1222,13 @@ Lint/RequireParentheses:
|
|
1092
1222
|
|
1093
1223
|
Lint/RescueException:
|
1094
1224
|
Description: 'Avoid rescuing the Exception class.'
|
1095
|
-
StyleGuide: '
|
1225
|
+
StyleGuide: '#no-blind-rescues'
|
1226
|
+
Enabled: true
|
1227
|
+
|
1228
|
+
Lint/ShadowedException:
|
1229
|
+
Description: >-
|
1230
|
+
Avoid rescuing a higher level exception
|
1231
|
+
before a lower level exception.
|
1096
1232
|
Enabled: true
|
1097
1233
|
|
1098
1234
|
Lint/ShadowingOuterLocalVariable:
|
@@ -1103,13 +1239,17 @@ Lint/ShadowingOuterLocalVariable:
|
|
1103
1239
|
|
1104
1240
|
Lint/StringConversionInInterpolation:
|
1105
1241
|
Description: 'Checks for Object#to_s usage in string interpolation.'
|
1106
|
-
StyleGuide: '
|
1242
|
+
StyleGuide: '#no-to-s'
|
1107
1243
|
Enabled: true
|
1108
1244
|
|
1109
1245
|
Lint/UnderscorePrefixedVariableName:
|
1110
1246
|
Description: 'Do not use prefix `_` for a variable that is used.'
|
1111
1247
|
Enabled: true
|
1112
1248
|
|
1249
|
+
Lint/UnifiedInteger:
|
1250
|
+
Description: 'Use Integer instead of Fixnum or Bignum'
|
1251
|
+
Enabled: true
|
1252
|
+
|
1113
1253
|
Lint/UnneededDisable:
|
1114
1254
|
Description: >-
|
1115
1255
|
Checks for rubocop:disable comments that can be removed.
|
@@ -1117,14 +1257,18 @@ Lint/UnneededDisable:
|
|
1117
1257
|
It must be explicitly disabled.
|
1118
1258
|
Enabled: true
|
1119
1259
|
|
1260
|
+
Lint/UnneededSplatExpansion:
|
1261
|
+
Description: 'Checks for splat unnecessarily being called on literals'
|
1262
|
+
Enabled: true
|
1263
|
+
|
1120
1264
|
Lint/UnusedBlockArgument:
|
1121
1265
|
Description: 'Checks for unused block arguments.'
|
1122
|
-
StyleGuide: '
|
1266
|
+
StyleGuide: '#underscore-unused-vars'
|
1123
1267
|
Enabled: true
|
1124
1268
|
|
1125
1269
|
Lint/UnusedMethodArgument:
|
1126
1270
|
Description: 'Checks for unused method arguments.'
|
1127
|
-
StyleGuide: '
|
1271
|
+
StyleGuide: '#underscore-unused-vars'
|
1128
1272
|
Enabled: true
|
1129
1273
|
|
1130
1274
|
Lint/UnreachableCode:
|
@@ -1134,10 +1278,11 @@ Lint/UnreachableCode:
|
|
1134
1278
|
Lint/UselessAccessModifier:
|
1135
1279
|
Description: 'Checks for useless access modifiers.'
|
1136
1280
|
Enabled: true
|
1281
|
+
ContextCreatingMethods: []
|
1137
1282
|
|
1138
1283
|
Lint/UselessAssignment:
|
1139
1284
|
Description: 'Checks for useless assignment to a local variable.'
|
1140
|
-
StyleGuide: '
|
1285
|
+
StyleGuide: '#underscore-unused-vars'
|
1141
1286
|
Enabled: true
|
1142
1287
|
|
1143
1288
|
Lint/UselessComparison:
|
@@ -1203,6 +1348,10 @@ Performance/DoubleStartEndWith:
|
|
1203
1348
|
Performance/EndWith:
|
1204
1349
|
Description: 'Use `end_with?` instead of a regex match anchored to the end of a string.'
|
1205
1350
|
Reference: 'https://github.com/JuanitoFatas/fast-ruby#stringmatch-vs-stringstart_withstringend_with-code-start-code-end'
|
1351
|
+
# This will change to a new method call which isn't guaranteed to be on the
|
1352
|
+
# object. Switching these methods has to be done with knowledge of the types
|
1353
|
+
# of the variables which rubocop doesn't have.
|
1354
|
+
AutoCorrect: false
|
1206
1355
|
Enabled: true
|
1207
1356
|
|
1208
1357
|
Performance/FixedSize:
|
@@ -1226,7 +1375,7 @@ Performance/HashEachMethods:
|
|
1226
1375
|
Description: >-
|
1227
1376
|
Use `Hash#each_key` and `Hash#each_value` instead of
|
1228
1377
|
`Hash#keys.each` and `Hash#values.each`.
|
1229
|
-
StyleGuide: '
|
1378
|
+
StyleGuide: '#hash-each'
|
1230
1379
|
Enabled: true
|
1231
1380
|
AutoCorrect: false
|
1232
1381
|
|
@@ -1267,7 +1416,7 @@ Performance/ReverseEach:
|
|
1267
1416
|
Performance/Sample:
|
1268
1417
|
Description: >-
|
1269
1418
|
Use `sample` instead of `shuffle.first`,
|
1270
|
-
`shuffle.last`, and `shuffle[
|
1419
|
+
`shuffle.last`, and `shuffle[Integer]`.
|
1271
1420
|
Reference: 'https://github.com/JuanitoFatas/fast-ruby#arrayshufflefirst-vs-arraysample-code'
|
1272
1421
|
Enabled: true
|
1273
1422
|
|
@@ -1278,9 +1427,17 @@ Performance/Size:
|
|
1278
1427
|
Reference: 'https://github.com/JuanitoFatas/fast-ruby#arraycount-vs-arraysize-code'
|
1279
1428
|
Enabled: true
|
1280
1429
|
|
1430
|
+
Performance/CompareWithBlock:
|
1431
|
+
Description: 'Use `sort_by(&:foo)` instead of `sort_by { |a, b| a.foo <=> b.foo }`.'
|
1432
|
+
Enabled: true
|
1433
|
+
|
1281
1434
|
Performance/StartWith:
|
1282
1435
|
Description: 'Use `start_with?` instead of a regex match anchored to the beginning of a string.'
|
1283
1436
|
Reference: 'https://github.com/JuanitoFatas/fast-ruby#stringmatch-vs-stringstart_withstringend_with-code-start-code-end'
|
1437
|
+
# This will change to a new method call which isn't guaranteed to be on the
|
1438
|
+
# object. Switching these methods has to be done with knowledge of the types
|
1439
|
+
# of the variables which rubocop doesn't have.
|
1440
|
+
AutoCorrect: false
|
1284
1441
|
Enabled: true
|
1285
1442
|
|
1286
1443
|
Performance/StringReplacement:
|
@@ -1311,22 +1468,60 @@ Rails/Delegate:
|
|
1311
1468
|
Description: 'Prefer delegate method for delegations.'
|
1312
1469
|
Enabled: true
|
1313
1470
|
|
1471
|
+
Rails/DelegateAllowBlank:
|
1472
|
+
Description: 'Do not use allow_blank as an option to delegate.'
|
1473
|
+
Enabled: true
|
1474
|
+
|
1475
|
+
Rails/DynamicFindBy:
|
1476
|
+
Description: 'Use `find_by` instead of dynamic `find_by_*`.'
|
1477
|
+
StyleGuide: 'https://github.com/bbatsov/rails-style-guide#find_by'
|
1478
|
+
Enabled: true
|
1479
|
+
|
1480
|
+
Rails/EnumUniqueness:
|
1481
|
+
Description: 'Avoid duplicate integers in hash-syntax `enum` declaration.'
|
1482
|
+
Enabled: true
|
1483
|
+
|
1484
|
+
Rails/Exit:
|
1485
|
+
Description: >-
|
1486
|
+
Favor `fail`, `break`, `return`, etc. over `exit` in
|
1487
|
+
application or library code outside of Rake files to avoid
|
1488
|
+
exits during unit testing or running in production.
|
1489
|
+
Enabled: true
|
1490
|
+
|
1314
1491
|
Rails/FindBy:
|
1315
1492
|
Description: 'Prefer find_by over where.first.'
|
1493
|
+
StyleGuide: 'https://github.com/bbatsov/rails-style-guide#find_by'
|
1316
1494
|
Enabled: true
|
1317
1495
|
|
1318
1496
|
Rails/FindEach:
|
1319
1497
|
Description: 'Prefer all.find_each over all.find.'
|
1498
|
+
StyleGuide: 'https://github.com/bbatsov/rails-style-guide#find-each'
|
1320
1499
|
Enabled: true
|
1321
1500
|
|
1322
1501
|
Rails/HasAndBelongsToMany:
|
1323
1502
|
Description: 'Prefer has_many :through to has_and_belongs_to_many.'
|
1503
|
+
StyleGuide: 'https://github.com/bbatsov/rails-style-guide#has-many-through'
|
1504
|
+
Enabled: true
|
1505
|
+
|
1506
|
+
Rails/HttpPositionalArguments:
|
1507
|
+
Description: 'Use keyword arguments instead of positional arguments in http method calls.'
|
1508
|
+
Enabled: true
|
1509
|
+
Include:
|
1510
|
+
- 'spec/**/*'
|
1511
|
+
- 'test/**/*'
|
1512
|
+
|
1513
|
+
Rails/NotNullColumn:
|
1514
|
+
Description: 'Do not add a NOT NULL column without a default value'
|
1324
1515
|
Enabled: true
|
1325
1516
|
|
1326
1517
|
Rails/Output:
|
1327
1518
|
Description: 'Checks for calls to puts, print, etc.'
|
1328
1519
|
Enabled: true
|
1329
1520
|
|
1521
|
+
Rails/OutputSafety:
|
1522
|
+
Description: 'The use of `html_safe` or `raw` may be a security risk.'
|
1523
|
+
Enabled: true
|
1524
|
+
|
1330
1525
|
Rails/PluralizationGrammar:
|
1331
1526
|
Description: 'Checks for incorrect grammar when using methods like `3.day.ago`.'
|
1332
1527
|
Enabled: true
|
@@ -1335,6 +1530,15 @@ Rails/ReadWriteAttribute:
|
|
1335
1530
|
Description: >-
|
1336
1531
|
Checks for read_attribute(:attr) and
|
1337
1532
|
write_attribute(:attr, val).
|
1533
|
+
StyleGuide: 'https://github.com/bbatsov/rails-style-guide#read-attribute'
|
1534
|
+
Enabled: true
|
1535
|
+
|
1536
|
+
Rails/RequestReferer:
|
1537
|
+
Description: 'Use consistent syntax for request.referer.'
|
1538
|
+
Enabled: true
|
1539
|
+
|
1540
|
+
Rails/SafeNavigation:
|
1541
|
+
Description: "Use Ruby's safe navigation operator (`&.`) instead of `try!`"
|
1338
1542
|
Enabled: true
|
1339
1543
|
|
1340
1544
|
Rails/ScopeArgs:
|
@@ -1347,6 +1551,38 @@ Rails/TimeZone:
|
|
1347
1551
|
Reference: 'http://danilenko.org/2012/7/6/rails_timezones'
|
1348
1552
|
Enabled: true
|
1349
1553
|
|
1554
|
+
Rails/UniqBeforePluck:
|
1555
|
+
Description: 'Prefer the use of uniq or distinct before pluck.'
|
1556
|
+
Enabled: true
|
1557
|
+
|
1350
1558
|
Rails/Validation:
|
1351
1559
|
Description: 'Use validates :attribute, hash of validations.'
|
1352
1560
|
Enabled: true
|
1561
|
+
|
1562
|
+
Security/JSONLoad:
|
1563
|
+
Description: >-
|
1564
|
+
Prefer usage of `JSON.parse` over `JSON.load` due to potential
|
1565
|
+
security issues. See reference for more information.
|
1566
|
+
Reference: 'http://ruby-doc.org/stdlib-2.3.0/libdoc/json/rdoc/JSON.html#method-i-load'
|
1567
|
+
Enabled: true
|
1568
|
+
# Autocorrect here will change to a method that may cause crashes depending
|
1569
|
+
# on the value of the argument.
|
1570
|
+
AutoCorrect: false
|
1571
|
+
|
1572
|
+
##################### Bundler #############################
|
1573
|
+
|
1574
|
+
Bundler/DuplicatedGem:
|
1575
|
+
Description: 'Checks for duplicate gem entries in Gemfile.'
|
1576
|
+
Enabled: true
|
1577
|
+
Include:
|
1578
|
+
- '**/Gemfile'
|
1579
|
+
- '**/gems.rb'
|
1580
|
+
|
1581
|
+
Bundler/OrderedGems:
|
1582
|
+
Description: >-
|
1583
|
+
Sort alphabetically gems appearing within a contiguous set
|
1584
|
+
of lines in the Gemfile
|
1585
|
+
Enabled: true
|
1586
|
+
Include:
|
1587
|
+
- '**/Gemfile'
|
1588
|
+
- '**/gems.rb'
|