dvla-lint 1.7.1
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 +7 -0
- data/.github/workflows/gem-push.yml +48 -0
- data/.github/workflows/gem-test.yml +22 -0
- data/.gitignore +19 -0
- data/.rspec +3 -0
- data/.rubocop.yml +26 -0
- data/.ruby-version +1 -0
- data/Gemfile +6 -0
- data/LICENSE +21 -0
- data/README.md +32 -0
- data/Rakefile +8 -0
- data/bin/console +15 -0
- data/bin/setup +8 -0
- data/dvla-lint.gemspec +41 -0
- data/exe/dvla-lint +66 -0
- data/lib/dvla/lint/version.rb +7 -0
- data/lib/dvla/lint.rb +13 -0
- data/rules/dvla-ruby-styleguide.yml +663 -0
- data/rules/other-excludes.yml +19 -0
- data/rules/other-gemspec.yml +7 -0
- data/rules/other-lint.yml +161 -0
- data/rules/other-metrics.yml +11 -0
- data/rules/other-rails.yml +42 -0
- data/rules/other-security.yml +3 -0
- data/rules/other-style.yml +522 -0
- metadata +197 -0
@@ -0,0 +1,161 @@
|
|
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/ConstantOverwrittenInRescue:
|
22
|
+
Description: "Checks for overwriting an exception with an exception result by use `rescue =>`."
|
23
|
+
Enabled: false
|
24
|
+
|
25
|
+
Lint/Debugger:
|
26
|
+
Description: 'Check for debugger calls.'
|
27
|
+
Enabled: true
|
28
|
+
|
29
|
+
Lint/DeprecatedClassMethods:
|
30
|
+
Description: 'Check for deprecated class method calls.'
|
31
|
+
Enabled: true
|
32
|
+
|
33
|
+
Lint/EmptyEnsure:
|
34
|
+
Description: 'Checks for empty ensure block.'
|
35
|
+
Enabled: true
|
36
|
+
|
37
|
+
Lint/EmptyInterpolation:
|
38
|
+
Description: 'Checks for empty string interpolation.'
|
39
|
+
Enabled: true
|
40
|
+
|
41
|
+
Style/EndBlock:
|
42
|
+
Description: 'END blocks should not be placed inside method definitions.'
|
43
|
+
Enabled: true
|
44
|
+
|
45
|
+
Lint/EnsureReturn:
|
46
|
+
Description: 'Never use return in an ensure block.'
|
47
|
+
Enabled: true
|
48
|
+
|
49
|
+
Security/Eval:
|
50
|
+
Description: 'The use of eval represents a serious security risk.'
|
51
|
+
Enabled: true
|
52
|
+
|
53
|
+
Lint/SuppressedException:
|
54
|
+
Description: "Don't suppress exception."
|
55
|
+
Enabled: true
|
56
|
+
|
57
|
+
Lint/LiteralAsCondition:
|
58
|
+
Description: 'Checks of literals used in conditions.'
|
59
|
+
Enabled: true
|
60
|
+
|
61
|
+
Lint/LiteralInInterpolation:
|
62
|
+
Description: 'Checks for literals used in interpolation.'
|
63
|
+
Enabled: true
|
64
|
+
|
65
|
+
Lint/Loop:
|
66
|
+
Description: >-
|
67
|
+
Use Kernel#loop with break rather than begin/end/until or
|
68
|
+
begin/end/while for post-loop tests.
|
69
|
+
Enabled: true
|
70
|
+
|
71
|
+
Lint/NonAtomicFileOperation:
|
72
|
+
Description: Checks for non-atomic file operation and then replace it with a nearly equivalent and atomic method
|
73
|
+
Enabled: false
|
74
|
+
|
75
|
+
Lint/ParenthesesAsGroupedExpression:
|
76
|
+
Description: >-
|
77
|
+
Checks for method calls with a space before the opening
|
78
|
+
parenthesis.
|
79
|
+
Enabled: true
|
80
|
+
|
81
|
+
Lint/PercentStringArray:
|
82
|
+
Description: Checks for unwanted commas and quotes in %w/%W literals eg %w('foo', “bar”).
|
83
|
+
Enabled: true
|
84
|
+
|
85
|
+
Lint/RequireParentheses:
|
86
|
+
Description: >-
|
87
|
+
Use parentheses in the method call to avoid confusion
|
88
|
+
about precedence.
|
89
|
+
Enabled: true
|
90
|
+
|
91
|
+
Lint/RequireRangeParentheses:
|
92
|
+
Description: Checks that a range literal is enclosed in parentheses when the end of the range is at a line break.
|
93
|
+
Enabled: false
|
94
|
+
|
95
|
+
Lint/ShadowingOuterLocalVariable:
|
96
|
+
Description: >-
|
97
|
+
Do not use the same name as outer local variable
|
98
|
+
for block arguments or block local variables.
|
99
|
+
Enabled: true
|
100
|
+
|
101
|
+
Layout/SpaceBeforeFirstArg:
|
102
|
+
Description: >-
|
103
|
+
Put a space between a method name and the first argument
|
104
|
+
in a method call without parentheses.
|
105
|
+
Enabled: true
|
106
|
+
|
107
|
+
Lint/UnderscorePrefixedVariableName:
|
108
|
+
Description: 'Do not use prefix `_` for a variable that is used.'
|
109
|
+
Enabled: true
|
110
|
+
|
111
|
+
Lint/UnusedMethodArgument:
|
112
|
+
Description: 'Checks for unused method arguments.'
|
113
|
+
Enabled: true
|
114
|
+
|
115
|
+
Lint/UnreachableCode:
|
116
|
+
Description: 'Unreachable code.'
|
117
|
+
Enabled: true
|
118
|
+
|
119
|
+
Lint/UselessAccessModifier:
|
120
|
+
Description: 'Checks for useless access modifiers.'
|
121
|
+
Enabled: true
|
122
|
+
|
123
|
+
Lint/UselessAssignment:
|
124
|
+
Description: 'Checks for useless assignment to a local variable.'
|
125
|
+
Enabled: true
|
126
|
+
|
127
|
+
Lint/BinaryOperatorWithIdenticalOperands:
|
128
|
+
Description: 'Checks for comparison of something with itself.'
|
129
|
+
Enabled: true
|
130
|
+
|
131
|
+
Lint/UselessSetterCall:
|
132
|
+
Description: 'Checks for useless setter call to a local variable.'
|
133
|
+
Enabled: true
|
134
|
+
|
135
|
+
Lint/Void:
|
136
|
+
Description: 'Possible use of operator/literal/variable in void context.'
|
137
|
+
Enabled: true
|
138
|
+
|
139
|
+
Lint/RefinementImportMethods: # new in 1.27
|
140
|
+
Description: 'Checks if include or prepend is called in refine block'
|
141
|
+
Enabled: true
|
142
|
+
|
143
|
+
Lint/DuplicateMagicComment: # new in 1.37
|
144
|
+
Description: Checks for duplicated magic comments
|
145
|
+
Enabled: true
|
146
|
+
|
147
|
+
Lint/UselessRescue: # new in 1.43
|
148
|
+
Description: Checks for useless `rescue`s, which only reraise rescued exceptions
|
149
|
+
Enabled: true
|
150
|
+
|
151
|
+
Lint/MixedCaseRange: # new in 1.53.0
|
152
|
+
Description: Checks for mixed-case character ranges since they include likely unintended characters.
|
153
|
+
Enabled: true
|
154
|
+
|
155
|
+
Lint/RedundantRegexpQuantifiers: # new in 1.53.0
|
156
|
+
Description: Checks for redundant quantifiers inside Regexp literals.
|
157
|
+
Enabled: true
|
158
|
+
|
159
|
+
Lint/DuplicateMatchPattern: # new in 1.50.0
|
160
|
+
Description: Checks that there are no repeated patterns used in in keywords.
|
161
|
+
Enabled: true
|
@@ -0,0 +1,42 @@
|
|
1
|
+
##################### Rails ##################################
|
2
|
+
|
3
|
+
# By default Rails is switched off so this can be used by non-Rails apps,
|
4
|
+
# this can be enabled in a local .rubocop.yml file
|
5
|
+
Rails:
|
6
|
+
Enabled: false
|
7
|
+
|
8
|
+
Rails/ActionFilter:
|
9
|
+
Description: 'Enforces consistent use of action filter methods.'
|
10
|
+
Enabled: false
|
11
|
+
|
12
|
+
Rails/Delegate:
|
13
|
+
Description: 'Prefer delegate method for delegations.'
|
14
|
+
Enabled: false
|
15
|
+
|
16
|
+
Rails/HasAndBelongsToMany:
|
17
|
+
Description: 'Prefer has_many :through to has_and_belongs_to_many.'
|
18
|
+
Enabled: false
|
19
|
+
|
20
|
+
Rails/Output:
|
21
|
+
Description: 'Checks for calls to puts, print, etc.'
|
22
|
+
Enabled: false
|
23
|
+
|
24
|
+
Rails/ReadWriteAttribute:
|
25
|
+
Description: 'Checks for read_attribute(:attr) and write_attribute(:attr, val).'
|
26
|
+
Enabled: false
|
27
|
+
|
28
|
+
Rails/ScopeArgs:
|
29
|
+
Description: 'Checks the arguments of ActiveRecord scopes.'
|
30
|
+
Enabled: false
|
31
|
+
|
32
|
+
Rails/Validation:
|
33
|
+
Description: 'Use sexy validations.'
|
34
|
+
Enabled: false
|
35
|
+
|
36
|
+
Rails/SkipsModelValidations:
|
37
|
+
Description: 'Avoid methods that skip model validations.'
|
38
|
+
Enabled: false
|
39
|
+
|
40
|
+
Rails/ActiveRecordAliases:
|
41
|
+
Description: 'The direct method names are more clear and easier to read.'
|
42
|
+
Enabled: false
|