rubocop-govuk 3.3.1 → 3.7.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.
metadata CHANGED
@@ -1,14 +1,14 @@
1
1
  --- !ruby/object:Gem::Specification
2
2
  name: rubocop-govuk
3
3
  version: !ruby/object:Gem::Version
4
- version: 3.3.1
4
+ version: 3.7.0
5
5
  platform: ruby
6
6
  authors:
7
7
  - Government Digital Service
8
8
  autorequire:
9
9
  bindir: bin
10
10
  cert_chain: []
11
- date: 2020-03-18 00:00:00.000000000 Z
11
+ date: 2020-04-29 00:00:00.000000000 Z
12
12
  dependencies:
13
13
  - !ruby/object:Gem::Dependency
14
14
  name: rake
@@ -30,14 +30,14 @@ dependencies:
30
30
  requirements:
31
31
  - - '='
32
32
  - !ruby/object:Gem::Version
33
- version: 0.80.1
33
+ version: 0.82.0
34
34
  type: :runtime
35
35
  prerelease: false
36
36
  version_requirements: !ruby/object:Gem::Requirement
37
37
  requirements:
38
38
  - - '='
39
39
  - !ruby/object:Gem::Version
40
- version: 0.80.1
40
+ version: 0.82.0
41
41
  - !ruby/object:Gem::Dependency
42
42
  name: rubocop-rails
43
43
  requirement: !ruby/object:Gem::Requirement
@@ -52,6 +52,20 @@ dependencies:
52
52
  - - "~>"
53
53
  - !ruby/object:Gem::Version
54
54
  version: '2'
55
+ - !ruby/object:Gem::Dependency
56
+ name: rubocop-rake
57
+ requirement: !ruby/object:Gem::Requirement
58
+ requirements:
59
+ - - "~>"
60
+ - !ruby/object:Gem::Version
61
+ version: 0.5.1
62
+ type: :runtime
63
+ prerelease: false
64
+ version_requirements: !ruby/object:Gem::Requirement
65
+ requirements:
66
+ - - "~>"
67
+ - !ruby/object:Gem::Version
68
+ version: 0.5.1
55
69
  - !ruby/object:Gem::Dependency
56
70
  name: rubocop-rspec
57
71
  requirement: !ruby/object:Gem::Requirement
@@ -76,13 +90,15 @@ files:
76
90
  - CHANGELOG.md
77
91
  - LICENSE.md
78
92
  - README.md
93
+ - config/bundler.yml
79
94
  - config/default.yml
80
- - config/gds-ruby-styleguide.yml
81
- - config/other-excludes.yml
82
- - config/other-lint.yml
83
- - config/other-metrics.yml
84
- - config/other-style.yml
95
+ - config/layout.yml
96
+ - config/lint.yml
97
+ - config/metrics.yml
98
+ - config/naming.yml
85
99
  - config/rails.yml
100
+ - config/rake.yml
101
+ - config/style.yml
86
102
  homepage: https://github.com/alphagov/rubocop-govuk
87
103
  licenses:
88
104
  - MIT
@@ -102,7 +118,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
102
118
  - !ruby/object:Gem::Version
103
119
  version: '0'
104
120
  requirements: []
105
- rubygems_version: 3.0.3
121
+ rubygems_version: 3.1.2
106
122
  signing_key:
107
123
  specification_version: 4
108
124
  summary: RuboCop GOV.UK
@@ -1,337 +0,0 @@
1
- ############
2
- ## General
3
- ############
4
-
5
- Layout/BlockAlignment:
6
- Description: 'Align block ends correctly.'
7
- Enabled: true
8
-
9
- Layout/CaseIndentation:
10
- Description: Indentation of when in a case/when/[else/]end.
11
- Enabled: true
12
- EnforcedStyle: case
13
- SupportedStyles:
14
- - case
15
- - end
16
- IndentOneStep: false
17
-
18
- Layout/ClosingParenthesisIndentation:
19
- Description: 'Checks the indentation of hanging closing parentheses.'
20
- Enabled: false
21
-
22
- Style/MutableConstant:
23
- Description: 'Freeze mutable objects assigned to constants.'
24
- Enabled: true
25
-
26
- Lint/ElseLayout:
27
- Description: 'Check for odd code arrangement in an else block.'
28
- Enabled: true
29
-
30
- # Supports --auto-correct
31
- Layout/EmptyLineBetweenDefs:
32
- Description: Use empty lines between defs.
33
- Enabled: true
34
- AllowAdjacentOneLineDefs: false
35
-
36
- Layout/EmptyLines:
37
- Description: "Don't use several empty lines in a row."
38
- Enabled: true
39
-
40
- Layout/EndAlignment:
41
- Description: 'Align ends correctly.'
42
- Enabled: true
43
-
44
- Layout/EndOfLine:
45
- Description: 'Use Unix-style line endings.'
46
- Enabled: true
47
-
48
- # Supports --auto-correct
49
- Layout/IndentationWidth:
50
- Description: Use 2 spaces for indentation.
51
- Enabled: true
52
-
53
- # Supports --auto-correct
54
- Layout/IndentationConsistency:
55
- Description: Keep indentation straight.
56
- Enabled: true
57
-
58
- Layout/LineLength:
59
- Description: Limit lines to 80 characters.
60
- Enabled: false
61
- Max: 80
62
-
63
- # Supports --auto-correct
64
- Layout/SpaceAroundOperators:
65
- Description: Use spaces around operators.
66
- Enabled: true
67
-
68
- # Supports --auto-correct
69
- Layout/SpaceBeforeBlockBraces:
70
- Description: Checks that the left block brace has or doesn't have space before it.
71
- Enabled: true
72
- EnforcedStyle: space
73
- SupportedStyles:
74
- - space
75
- - no_space
76
-
77
- # Supports --auto-correct
78
- Layout/SpaceAfterSemicolon:
79
- Description: Use spaces after semicolons.
80
- Enabled: true
81
-
82
- # Supports --auto-correct
83
- Layout/SpaceAfterColon:
84
- Description: Use spaces after colons.
85
- Enabled: true
86
-
87
- # Supports --auto-correct
88
- Layout/SpaceAfterComma:
89
- Description: Use spaces after commas.
90
- Enabled: true
91
-
92
- # Supports --auto-correct
93
- Layout/SpaceInsideReferenceBrackets:
94
- Description: No spaces after array[ or before ].
95
- Enabled: true
96
-
97
- # Supports --auto-correct
98
- Layout/SpaceInsideArrayLiteralBrackets:
99
- Description: No spaces after array = [ or before ].
100
- Enabled: true
101
-
102
- # Supports --auto-correct
103
- Layout/SpaceInsideParens:
104
- Description: No spaces after ( or before ).
105
- Enabled: true
106
-
107
- Layout/Tab:
108
- Description: No hard tabs.
109
- Enabled: true
110
-
111
- # Supports --auto-correct
112
- Layout/TrailingWhitespace:
113
- Description: Avoid trailing whitespace.
114
- Enabled: true
115
-
116
- # Supports --auto-correct
117
- Layout/TrailingEmptyLines:
118
- Description: Checks trailing blank lines and final newline.
119
- Enabled: true
120
- EnforcedStyle: final_newline
121
- SupportedStyles:
122
- - final_newline
123
- - final_blank_line
124
-
125
- ## Syntax
126
-
127
- # Supports --auto-correct
128
- Style/AndOr:
129
- Description: Use &&/|| instead of and/or.
130
- Enabled: true
131
-
132
- # Supports --auto-correct
133
- Style/DefWithParentheses:
134
- Description: Use def with parentheses when there are arguments.
135
- Enabled: true
136
-
137
- Style/For:
138
- Description: Checks use of for or each in multiline loops.
139
- Enabled: true
140
- EnforcedStyle: each
141
- SupportedStyles:
142
- - for
143
- - each
144
-
145
- Style/IfUnlessModifier:
146
- Description: Favor modifier if/unless usage when you have a single-line body.
147
- Enabled: false
148
-
149
- Style/MethodCalledOnDoEndBlock:
150
- Description: Avoid chaining a method call on a do...end block.
151
- Enabled: true
152
-
153
- Style/MethodCallWithoutArgsParentheses:
154
- Description: 'Do not use parentheses for method calls with no arguments.'
155
- Enabled: true
156
-
157
- Style/MultilineBlockChain:
158
- Description: Avoid multi-line chains of blocks.
159
- Enabled: true
160
-
161
- Style/MultilineIfThen:
162
- Description: Never use then for multi-line if/unless.
163
- Enabled: true
164
-
165
- Style/MultilineTernaryOperator:
166
- Description: 'Avoid multi-line ?: (the ternary operator); use if/unless instead.'
167
- Enabled: true
168
-
169
- Style/NestedTernaryOperator:
170
- Description: Use one expression per branch in a ternary operator.
171
- Enabled: true
172
-
173
- Style/OneLineConditional:
174
- Description: Favor the ternary operator(?:) over if/then/else/end constructs.
175
- Enabled: true
176
-
177
- # Supports --auto-correct
178
- Style/ParenthesesAroundCondition:
179
- Description: Don't use parentheses around the condition of an if/unless/while.
180
- Enabled: true
181
- AllowSafeAssignment: true
182
-
183
- # Supports --auto-correct
184
- Style/RedundantReturn:
185
- Description: Don't use return where it's not required.
186
- Enabled: true
187
- AllowMultipleReturnValues: false
188
-
189
- # Supports --auto-correct
190
- Layout/SpaceAfterMethodName:
191
- Description: Never put a space between a method name and the opening parenthesis in
192
- a method definition.
193
- Enabled: true
194
-
195
- # Supports --auto-correct
196
- Layout/SpaceAroundEqualsInParameterDefault:
197
- Description: Checks that the equals signs in parameter default assignments have or
198
- don't have surrounding space depending on configuration.
199
- Enabled: true
200
- EnforcedStyle: space
201
- SupportedStyles:
202
- - space
203
- - no_space
204
-
205
- Style/UnlessElse:
206
- Description: Never use unless with else. Rewrite these with the positive case first.
207
- Enabled: true
208
-
209
- # Supports --auto-correct
210
- Lint/UnusedBlockArgument:
211
- Description: Checks for unused block arguments.
212
- Enabled: true
213
-
214
- ## Naming
215
-
216
- Naming/ClassAndModuleCamelCase:
217
- Description: Use CamelCase for classes and modules.
218
- Enabled: true
219
-
220
- # Supports --auto-correct
221
- Style/ClassMethods:
222
- Description: Use self when defining module/class methods.
223
- Enabled: true
224
-
225
- Style/ClassVars:
226
- Description: Avoid the use of class variables.
227
- Enabled: true
228
-
229
- Naming/ConstantName:
230
- Description: Constants should use SCREAMING_SNAKE_CASE.
231
- Enabled: true
232
-
233
- Naming/MethodName:
234
- Description: Use the configured style when naming methods.
235
- Enabled: true
236
- EnforcedStyle: snake_case
237
- SupportedStyles:
238
- - snake_case
239
- - camelCase
240
-
241
- # Supports --auto-correct
242
- Style/TrivialAccessors:
243
- Description: Prefer attr_* methods to trivial readers/writers.
244
- Enabled: true
245
- ExactNameMatch: true
246
- AllowPredicates: true
247
- AllowDSLWriters: false
248
- AllowedMethods:
249
- - to_ary
250
- - to_a
251
- - to_c
252
- - to_enum
253
- - to_h
254
- - to_hash
255
- - to_i
256
- - to_int
257
- - to_io
258
- - to_open
259
- - to_path
260
- - to_proc
261
- - to_r
262
- - to_regexp
263
- - to_str
264
- - to_s
265
- - to_sym
266
-
267
- Naming/VariableName:
268
- Description: Use the configured style when naming variables.
269
- Enabled: true
270
- EnforcedStyle: snake_case
271
- SupportedStyles:
272
- - snake_case
273
- - camelCase
274
-
275
- ## Exceptions
276
-
277
- # Supports --auto-correct
278
- Lint/RescueException:
279
- Description: Avoid rescuing the Exception class.
280
- Enabled: true
281
-
282
- ## Collections
283
-
284
- # Supports --auto-correct
285
- Layout/ArrayAlignment:
286
- Description: Align the elements of an array literal if they span more than one line.
287
- Enabled: true
288
-
289
- # Supports --auto-correct
290
- Style/HashSyntax:
291
- Description: 'Prefer Ruby 1.9 hash syntax { a: 1, b: 2 } over 1.8 syntax { :a => 1,
292
- :b => 2 }.'
293
- Enabled: true
294
- Exclude:
295
- - 'db/schema.rb'
296
-
297
- EnforcedStyle: ruby19
298
- SupportedStyles:
299
- - ruby19
300
- - hash_rockets
301
-
302
- Style/TrailingCommaInArrayLiteral:
303
- Enabled: true
304
- EnforcedStyleForMultiline: comma
305
-
306
- Style/TrailingCommaInHashLiteral:
307
- Enabled: true
308
- EnforcedStyleForMultiline: comma
309
-
310
- Style/TrailingCommaInArguments:
311
- Enabled: true
312
- EnforcedStyleForMultiline: comma
313
-
314
- # Supports --auto-correct
315
- Style/WordArray:
316
- Description: Use %w or %W for arrays of words.
317
- Enabled: true
318
- MinSize: 0
319
-
320
- Layout/MultilineMethodCallIndentation:
321
- Enabled: false
322
-
323
- ## Strings
324
-
325
- # Supports --auto-correct
326
- Lint/RedundantStringCoercion:
327
- Description: Checks for Object#to_s usage in string interpolation.
328
- Enabled: true
329
-
330
- # Supports --auto-correct
331
- Style/StringLiterals:
332
- Description: Checks if uses of quotes match the configured preference.
333
- Enabled: true
334
- EnforcedStyle: double_quotes
335
- SupportedStyles:
336
- - single_quotes
337
- - double_quotes
@@ -1,23 +0,0 @@
1
- AllCops:
2
- Exclude:
3
- - db/schema.rb
4
-
5
- Metrics/BlockLength:
6
- Enabled: true
7
- Exclude:
8
- - test/**/*
9
- - "**/spec/**/*"
10
- ExcludedMethods: ["namespace"]
11
-
12
- Bundler/DuplicatedGem:
13
- Enabled: false
14
-
15
- Style/FormatStringToken:
16
- Enabled: false
17
-
18
- # These can result in false positives
19
- Style/HashTransformKeys:
20
- Enabled: false
21
-
22
- Style/HashTransformValues:
23
- Enabled: false
@@ -1,129 +0,0 @@
1
- Lint/AmbiguousOperator:
2
- Description: >-
3
- Checks for ambiguous operators in the first argument of a
4
- method invocation without parentheses.
5
- Enabled: true
6
-
7
- Lint/AmbiguousRegexpLiteral:
8
- Description: >-
9
- Checks for ambiguous regexp literals in the first argument of
10
- a method invocation without parenthesis.
11
- Enabled: true
12
-
13
- Lint/AssignmentInCondition:
14
- Description: "Don't use assignment in conditions."
15
- Enabled: true
16
-
17
- Layout/ConditionPosition:
18
- Description: 'Checks for condition placed in a confusing position relative to the keyword.'
19
- Enabled: true
20
-
21
- Lint/Debugger:
22
- Description: 'Check for debugger calls.'
23
- Enabled: true
24
-
25
- Lint/DeprecatedClassMethods:
26
- Description: 'Check for deprecated class method calls.'
27
- Enabled: true
28
-
29
- Lint/EmptyEnsure:
30
- Description: 'Checks for empty ensure block.'
31
- Enabled: true
32
-
33
- Lint/EmptyInterpolation:
34
- Description: 'Checks for empty string interpolation.'
35
- Enabled: true
36
-
37
- Lint/EndInMethod:
38
- Description: 'END blocks should not be placed inside method definitions.'
39
- Enabled: true
40
-
41
- Lint/EnsureReturn:
42
- Description: 'Never use return in an ensure block.'
43
- Enabled: true
44
-
45
- Security/Eval:
46
- Description: 'The use of eval represents a serious security risk.'
47
- Enabled: true
48
-
49
- Lint/SuppressedException:
50
- Description: "Don't suppress exception."
51
- Enabled: true
52
-
53
- Lint/LiteralAsCondition:
54
- Description: 'Checks of literals used in conditions.'
55
- Enabled: true
56
-
57
- Lint/LiteralInInterpolation:
58
- Description: 'Checks for literals used in interpolation.'
59
- Enabled: true
60
-
61
- Lint/Loop:
62
- Description: >-
63
- Use Kernel#loop with break rather than begin/end/until or
64
- begin/end/while for post-loop tests.
65
- Enabled: true
66
-
67
- Lint/ParenthesesAsGroupedExpression:
68
- Description: >-
69
- Checks for method calls with a space before the opening
70
- parenthesis.
71
- Enabled: true
72
-
73
- Lint/PercentStringArray:
74
- Description: Checks for unwanted commas and quotes in %w/%W literals eg %w('foo', “bar”).
75
- Enabled: true
76
-
77
- Lint/RequireParentheses:
78
- Description: >-
79
- Use parentheses in the method call to avoid confusion
80
- about precedence.
81
- Enabled: true
82
-
83
- Lint/ShadowingOuterLocalVariable:
84
- Description: >-
85
- Do not use the same name as outer local variable
86
- for block arguments or block local variables.
87
- Enabled: true
88
-
89
- Layout/SpaceBeforeFirstArg:
90
- Description: >-
91
- Put a space between a method name and the first argument
92
- in a method call without parentheses.
93
- Enabled: true
94
-
95
- Lint/UnderscorePrefixedVariableName:
96
- Description: 'Do not use prefix `_` for a variable that is used.'
97
- Enabled: true
98
-
99
- Lint/UnusedMethodArgument:
100
- Description: 'Checks for unused method arguments.'
101
- Enabled: true
102
-
103
- Lint/UnreachableCode:
104
- Description: 'Unreachable code.'
105
- Enabled: true
106
-
107
- Lint/UselessAccessModifier:
108
- Description: 'Checks for useless access modifiers.'
109
- Enabled: true
110
-
111
- Lint/UselessAssignment:
112
- Description: 'Checks for useless assignment to a local variable.'
113
- Enabled: true
114
-
115
- Lint/UselessComparison:
116
- Description: 'Checks for comparison of something with itself.'
117
- Enabled: true
118
-
119
- Lint/UselessElseWithoutRescue:
120
- Description: 'Checks for useless `else` in `begin..end` without `rescue`.'
121
- Enabled: true
122
-
123
- Lint/UselessSetterCall:
124
- Description: 'Checks for useless setter call to a local variable.'
125
- Enabled: true
126
-
127
- Lint/Void:
128
- Description: 'Possible use of operator/literal/variable in void context.'
129
- Enabled: true