rubocop-amagi 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/lib/cops/duplicate_constant_cop.rb +40 -0
- data/lib/cops/logger_format_cop.rb +67 -0
- data/lib/cops/readme.md +59 -0
- data/rubocop.yml +242 -183
- metadata +6 -3
checksums.yaml
CHANGED
|
@@ -1,7 +1,7 @@
|
|
|
1
1
|
---
|
|
2
2
|
SHA256:
|
|
3
|
-
metadata.gz:
|
|
4
|
-
data.tar.gz:
|
|
3
|
+
metadata.gz: 3948acf3c16d99d00f4d0d583691b817cebde99fd51e13d2103c584973ebe1d3
|
|
4
|
+
data.tar.gz: 04c64ff3bceb28524d44d2bbabef49562a3b33bc315376440bdf619d3c842523
|
|
5
5
|
SHA512:
|
|
6
|
-
metadata.gz:
|
|
7
|
-
data.tar.gz:
|
|
6
|
+
metadata.gz: 5ca7aeaaaca366eb83baa552c9093aea4c2e48830206e37bfb90f693e9f80044a9865df0b76f73a9ea677efcfffe2a3f15818917daf507dd857ce018b293af91
|
|
7
|
+
data.tar.gz: c637e37d4602e07552482f11862b950771bc2d92b56890dc280914d8ef7062d913d544c1cc5fa945cb19f5d8215e269d80c91daa5c898d39a8d8ce36aa19be23
|
|
@@ -0,0 +1,40 @@
|
|
|
1
|
+
# frozen_string_literal: true
|
|
2
|
+
|
|
3
|
+
# This cop will checks the constant for the duplicate values in a file
|
|
4
|
+
# if two constants have same values it may give wrong constant name if we fetch constants using values
|
|
5
|
+
|
|
6
|
+
module RuboCop
|
|
7
|
+
module Cop
|
|
8
|
+
module CustomCops
|
|
9
|
+
class DuplicateConstantCop < Cop
|
|
10
|
+
|
|
11
|
+
# on every file reset the hash
|
|
12
|
+
def on_new_investigation
|
|
13
|
+
@occurance = {}
|
|
14
|
+
end
|
|
15
|
+
|
|
16
|
+
# on getting constant assignment
|
|
17
|
+
# check if there is any constant already present with the same value
|
|
18
|
+
# in that particular file
|
|
19
|
+
# ignore if the const value is other than string
|
|
20
|
+
# example of node on_casgn
|
|
21
|
+
# (casgn nil :MILLI_SECOND
|
|
22
|
+
# (int 1000000))
|
|
23
|
+
# node.children[1] consists of const name(ex: MILLI_SECOND)
|
|
24
|
+
# node.children[2] consists of type and value (ex: (int 1000000))
|
|
25
|
+
# node.children[2].source gives the value of the constant
|
|
26
|
+
def on_casgn(node)
|
|
27
|
+
const = node.children[1]
|
|
28
|
+
value = node.children[2].source
|
|
29
|
+
# if value is already present return error
|
|
30
|
+
if @occurance[value].present?
|
|
31
|
+
add_offense(node, location: :expression, message: "Duplicate value #{value} for the const #{const}. Value already present in the file as #{@occurance[value]}")
|
|
32
|
+
# populate hash only if the value type string
|
|
33
|
+
elsif node.children[2].str_type?
|
|
34
|
+
@occurance[value] = const
|
|
35
|
+
end
|
|
36
|
+
end
|
|
37
|
+
end
|
|
38
|
+
end
|
|
39
|
+
end
|
|
40
|
+
end
|
|
@@ -0,0 +1,67 @@
|
|
|
1
|
+
module RuboCop
|
|
2
|
+
module Cop
|
|
3
|
+
module CustomCops
|
|
4
|
+
class PutLoggerFormatCop < Cop
|
|
5
|
+
# Constant required for Rubocop
|
|
6
|
+
MSG = 'Log Format should be <module/class>#method_name:<space><message>'.freeze
|
|
7
|
+
def get_classname(node)
|
|
8
|
+
# for getting complete names of controllers
|
|
9
|
+
@name = ""
|
|
10
|
+
if node.children[0] == nil
|
|
11
|
+
@name += node.children[1].to_s + "::"
|
|
12
|
+
else
|
|
13
|
+
get_classname(node.children[0])
|
|
14
|
+
@name += node.children[1].to_s + "::"
|
|
15
|
+
end
|
|
16
|
+
return @name[0...-2]
|
|
17
|
+
end
|
|
18
|
+
|
|
19
|
+
def on_module(node)
|
|
20
|
+
# gives module name for a particular node
|
|
21
|
+
@class_name = node.children[0].children[1]
|
|
22
|
+
end
|
|
23
|
+
|
|
24
|
+
def on_class(node)
|
|
25
|
+
# gives class name for a particular node
|
|
26
|
+
@class_name = get_classname(node.children[0])
|
|
27
|
+
end
|
|
28
|
+
|
|
29
|
+
def on_def(node)
|
|
30
|
+
# gives method_name for each node
|
|
31
|
+
@method_name = node.children[0]
|
|
32
|
+
end
|
|
33
|
+
|
|
34
|
+
def on_defs(node)
|
|
35
|
+
# gives method_name for class methods
|
|
36
|
+
@method_name = node.children[1]
|
|
37
|
+
end
|
|
38
|
+
|
|
39
|
+
def on_block(node)
|
|
40
|
+
# give name of a block defined without any method
|
|
41
|
+
@method_name = node.children[0].children[1].to_s if node.parent&.parent && node.parent.parent&.class_type?
|
|
42
|
+
end
|
|
43
|
+
|
|
44
|
+
def on_send(node)
|
|
45
|
+
if node.children[1] == :puts
|
|
46
|
+
str = get_string(node.children[2])
|
|
47
|
+
if str != "#{@class_name}##{@method_name}:"
|
|
48
|
+
# add offense if format not correct
|
|
49
|
+
add_offense(node, location: :expression)
|
|
50
|
+
end
|
|
51
|
+
end
|
|
52
|
+
end
|
|
53
|
+
|
|
54
|
+
def get_string(node)
|
|
55
|
+
string = if node&.str_type?
|
|
56
|
+
node.children[0].to_s
|
|
57
|
+
elsif node&.dstr_type?
|
|
58
|
+
node.children[0].children[0].to_s
|
|
59
|
+
else
|
|
60
|
+
'false_string'
|
|
61
|
+
end
|
|
62
|
+
string.partition(' ').first
|
|
63
|
+
end
|
|
64
|
+
end
|
|
65
|
+
end
|
|
66
|
+
end
|
|
67
|
+
end
|
data/lib/cops/readme.md
ADDED
|
@@ -0,0 +1,59 @@
|
|
|
1
|
+
# Amagi Rubocop CustomCop Rules
|
|
2
|
+
|
|
3
|
+
## How to use CustomCops?
|
|
4
|
+
***
|
|
5
|
+
**Using any cop**
|
|
6
|
+
---
|
|
7
|
+
Open .rubocop.yml in your working directory and set the enabled option as true.
|
|
8
|
+
~~~ruby
|
|
9
|
+
CustomCop/Copname:
|
|
10
|
+
Enabled: true
|
|
11
|
+
~~~
|
|
12
|
+
### PutLoggerFormatCop
|
|
13
|
+
PutLoggerFormatCop ensures that the message format received on encountering **puts** is of the standard format
|
|
14
|
+
~~~ruby
|
|
15
|
+
"<module/class>#method_name:<space><message>".
|
|
16
|
+
~~~
|
|
17
|
+
Enabling the cop inside .rubocop.yml :
|
|
18
|
+
~~~ruby
|
|
19
|
+
CustomCop/PutLoggerFormatCop:
|
|
20
|
+
Enabled: true
|
|
21
|
+
~~~
|
|
22
|
+
|
|
23
|
+
### DuplicateConstantCop
|
|
24
|
+
DuplicateConstantCop ensures that there is no two constants in a file which has same string value
|
|
25
|
+
|
|
26
|
+
#### Reason to add this cop
|
|
27
|
+
if we have a same value for two constants and we try to fetch constant name using its value which ends in picking up the same constant always ignoring the other one.
|
|
28
|
+
|
|
29
|
+
~~~ruby
|
|
30
|
+
# not allowed
|
|
31
|
+
TEST = "abc"
|
|
32
|
+
TEST1 = "abc"
|
|
33
|
+
~~~
|
|
34
|
+
|
|
35
|
+
~~~ruby
|
|
36
|
+
# allowed
|
|
37
|
+
TEST = "abc"
|
|
38
|
+
TEST1 = "abcd"
|
|
39
|
+
~~~
|
|
40
|
+
|
|
41
|
+
~~~ruby
|
|
42
|
+
# allowed
|
|
43
|
+
TEST = 10
|
|
44
|
+
TEST1 = 10
|
|
45
|
+
~~~
|
|
46
|
+
|
|
47
|
+
~~~ruby
|
|
48
|
+
# allowed
|
|
49
|
+
TEST = ["abc", "def"]
|
|
50
|
+
TEST1 = ["abc", "def"]
|
|
51
|
+
~~~
|
|
52
|
+
|
|
53
|
+
Enabled by default
|
|
54
|
+
|
|
55
|
+
to disable it add the following lines in .rubocop.yml of your repo:
|
|
56
|
+
~~~ruby
|
|
57
|
+
CustomCop/DuplicateConstantCop:
|
|
58
|
+
Enabled: false
|
|
59
|
+
~~~
|
data/rubocop.yml
CHANGED
|
@@ -1,117 +1,130 @@
|
|
|
1
|
+
inherit_gem:
|
|
2
|
+
rubocop-shopify: rubocop.yml
|
|
3
|
+
|
|
1
4
|
AllCops:
|
|
2
5
|
TargetRubyVersion: 2.7.1
|
|
3
6
|
Exclude:
|
|
4
7
|
- 'vendor/**/*'
|
|
5
8
|
- 'node_modules/**/*'
|
|
6
|
-
|
|
9
|
+
|
|
10
|
+
require:
|
|
11
|
+
- './lib/cops/logger_format_cop.rb'
|
|
12
|
+
- './lib/cops/duplicate_constant_cop.rb'
|
|
13
|
+
- 'rubocop-rails'
|
|
14
|
+
|
|
15
|
+
CustomCops/PutLoggerFormatCop:
|
|
16
|
+
Enabled: false
|
|
17
|
+
|
|
18
|
+
CustomCops/DuplicateConstantCop:
|
|
19
|
+
Enabled: true
|
|
7
20
|
|
|
8
21
|
Bundler/DuplicatedGem:
|
|
9
22
|
Enabled: true
|
|
10
23
|
|
|
11
24
|
Bundler/OrderedGems:
|
|
12
|
-
Enabled:
|
|
25
|
+
Enabled: false
|
|
13
26
|
|
|
14
27
|
Layout/AccessModifierIndentation:
|
|
15
|
-
Enabled:
|
|
28
|
+
Enabled: false
|
|
16
29
|
|
|
17
30
|
Layout/ArgumentAlignment:
|
|
18
|
-
Enabled:
|
|
31
|
+
Enabled: false
|
|
19
32
|
|
|
20
33
|
Layout/ArrayAlignment:
|
|
21
|
-
Enabled:
|
|
34
|
+
Enabled: false
|
|
22
35
|
|
|
23
36
|
Layout/AssignmentIndentation:
|
|
24
|
-
Enabled:
|
|
37
|
+
Enabled: false
|
|
25
38
|
|
|
26
39
|
Layout/BlockAlignment:
|
|
27
|
-
Enabled:
|
|
40
|
+
Enabled: false
|
|
28
41
|
|
|
29
42
|
Layout/BlockEndNewline:
|
|
30
|
-
Enabled:
|
|
43
|
+
Enabled: false
|
|
31
44
|
|
|
32
45
|
Layout/CaseIndentation:
|
|
33
|
-
Enabled:
|
|
46
|
+
Enabled: false
|
|
34
47
|
|
|
35
48
|
Layout/ClosingHeredocIndentation:
|
|
36
|
-
Enabled:
|
|
49
|
+
Enabled: false
|
|
37
50
|
|
|
38
51
|
Layout/ClosingParenthesisIndentation:
|
|
39
|
-
Enabled:
|
|
52
|
+
Enabled: false
|
|
40
53
|
|
|
41
54
|
Layout/CommentIndentation:
|
|
42
|
-
Enabled:
|
|
55
|
+
Enabled: false
|
|
43
56
|
|
|
44
57
|
Layout/DefEndAlignment:
|
|
45
|
-
Enabled:
|
|
58
|
+
Enabled: false
|
|
46
59
|
|
|
47
60
|
Layout/DotPosition:
|
|
48
|
-
Enabled:
|
|
61
|
+
Enabled: false
|
|
49
62
|
|
|
50
63
|
Layout/ElseAlignment:
|
|
51
|
-
Enabled:
|
|
64
|
+
Enabled: false
|
|
52
65
|
|
|
53
66
|
Layout/EmptyComment:
|
|
54
|
-
Enabled:
|
|
67
|
+
Enabled: false
|
|
55
68
|
|
|
56
69
|
Layout/EmptyLineAfterGuardClause:
|
|
57
|
-
Enabled:
|
|
70
|
+
Enabled: false
|
|
58
71
|
|
|
59
72
|
Layout/EmptyLineBetweenDefs:
|
|
60
73
|
Enabled: true
|
|
61
74
|
|
|
62
75
|
Layout/EmptyLines:
|
|
63
|
-
Enabled:
|
|
76
|
+
Enabled: false
|
|
64
77
|
|
|
65
78
|
Layout/EmptyLinesAroundAccessModifier:
|
|
66
|
-
Enabled:
|
|
79
|
+
Enabled: false
|
|
67
80
|
|
|
68
81
|
Layout/EmptyLinesAroundBeginBody:
|
|
69
|
-
Enabled:
|
|
82
|
+
Enabled: false
|
|
70
83
|
|
|
71
84
|
Layout/EmptyLinesAroundBlockBody:
|
|
72
|
-
Enabled:
|
|
85
|
+
Enabled: false
|
|
73
86
|
|
|
74
87
|
Layout/EmptyLinesAroundClassBody:
|
|
75
|
-
Enabled:
|
|
88
|
+
Enabled: false
|
|
76
89
|
|
|
77
90
|
Layout/EmptyLinesAroundExceptionHandlingKeywords:
|
|
78
|
-
Enabled:
|
|
91
|
+
Enabled: false
|
|
79
92
|
|
|
80
93
|
Layout/EmptyLineAfterMagicComment:
|
|
81
94
|
Enabled: true
|
|
82
95
|
|
|
83
96
|
Layout/EmptyLinesAroundMethodBody:
|
|
84
|
-
Enabled:
|
|
97
|
+
Enabled: false
|
|
85
98
|
|
|
86
99
|
Layout/EmptyLinesAroundModuleBody:
|
|
87
|
-
Enabled:
|
|
100
|
+
Enabled: false
|
|
88
101
|
|
|
89
102
|
Layout/EndAlignment:
|
|
90
103
|
Enabled: true
|
|
91
104
|
|
|
92
105
|
Layout/EndOfLine:
|
|
93
|
-
Enabled:
|
|
106
|
+
Enabled: false
|
|
94
107
|
|
|
95
108
|
Layout/ExtraSpacing:
|
|
96
|
-
Enabled:
|
|
109
|
+
Enabled: false
|
|
97
110
|
|
|
98
111
|
Layout/FirstArgumentIndentation:
|
|
99
|
-
Enabled:
|
|
112
|
+
Enabled: false
|
|
100
113
|
|
|
101
114
|
Layout/FirstArrayElementIndentation:
|
|
102
|
-
Enabled:
|
|
115
|
+
Enabled: false
|
|
103
116
|
|
|
104
117
|
Layout/FirstHashElementIndentation:
|
|
105
|
-
Enabled:
|
|
118
|
+
Enabled: false
|
|
106
119
|
|
|
107
120
|
Layout/HashAlignment:
|
|
108
|
-
Enabled:
|
|
121
|
+
Enabled: false
|
|
109
122
|
|
|
110
123
|
Layout/HeredocIndentation:
|
|
111
|
-
Enabled:
|
|
124
|
+
Enabled: false
|
|
112
125
|
|
|
113
126
|
Layout/IndentationConsistency:
|
|
114
|
-
Enabled:
|
|
127
|
+
Enabled: false
|
|
115
128
|
|
|
116
129
|
Layout/IndentationStyle:
|
|
117
130
|
Enabled: true
|
|
@@ -127,88 +140,88 @@ Layout/LeadingCommentSpace:
|
|
|
127
140
|
Enabled: true
|
|
128
141
|
|
|
129
142
|
Layout/LeadingEmptyLines:
|
|
130
|
-
Enabled:
|
|
143
|
+
Enabled: false
|
|
131
144
|
|
|
132
145
|
Layout/LineLength:
|
|
133
|
-
Enabled:
|
|
146
|
+
Enabled: false
|
|
134
147
|
|
|
135
148
|
Layout/MultilineArrayBraceLayout:
|
|
136
|
-
Enabled:
|
|
149
|
+
Enabled: false
|
|
137
150
|
|
|
138
151
|
Layout/MultilineBlockLayout:
|
|
139
|
-
Enabled:
|
|
152
|
+
Enabled: false
|
|
140
153
|
|
|
141
154
|
Layout/MultilineHashBraceLayout:
|
|
142
|
-
Enabled:
|
|
155
|
+
Enabled: false
|
|
143
156
|
|
|
144
157
|
Layout/MultilineMethodCallBraceLayout:
|
|
145
|
-
Enabled:
|
|
158
|
+
Enabled: false
|
|
146
159
|
|
|
147
160
|
Layout/MultilineMethodCallIndentation:
|
|
148
|
-
Enabled:
|
|
161
|
+
Enabled: false
|
|
149
162
|
|
|
150
163
|
Layout/MultilineOperationIndentation:
|
|
151
|
-
Enabled:
|
|
164
|
+
Enabled: false
|
|
152
165
|
|
|
153
166
|
Layout/RescueEnsureAlignment:
|
|
154
167
|
Enabled: true
|
|
155
168
|
|
|
156
169
|
Layout/SpaceAfterColon:
|
|
157
|
-
Enabled:
|
|
170
|
+
Enabled: false
|
|
158
171
|
|
|
159
172
|
Layout/SpaceAfterComma:
|
|
160
173
|
Enabled: true
|
|
161
174
|
|
|
162
175
|
Layout/SpaceAfterMethodName:
|
|
163
|
-
Enabled:
|
|
176
|
+
Enabled: false
|
|
164
177
|
|
|
165
178
|
Layout/SpaceAfterNot:
|
|
166
|
-
Enabled:
|
|
179
|
+
Enabled: false
|
|
167
180
|
|
|
168
181
|
Layout/SpaceAfterSemicolon:
|
|
169
|
-
Enabled:
|
|
182
|
+
Enabled: false
|
|
170
183
|
|
|
171
184
|
Layout/SpaceAroundBlockParameters:
|
|
172
|
-
Enabled:
|
|
185
|
+
Enabled: false
|
|
173
186
|
|
|
174
187
|
Layout/SpaceAroundEqualsInParameterDefault:
|
|
175
|
-
Enabled:
|
|
188
|
+
Enabled: false
|
|
176
189
|
|
|
177
190
|
Layout/SpaceAroundKeyword:
|
|
178
|
-
Enabled:
|
|
191
|
+
Enabled: false
|
|
179
192
|
|
|
180
193
|
Layout/SpaceAroundOperators:
|
|
181
|
-
Enabled:
|
|
194
|
+
Enabled: false
|
|
182
195
|
|
|
183
196
|
Layout/SpaceBeforeBlockBraces:
|
|
184
|
-
Enabled:
|
|
197
|
+
Enabled: false
|
|
185
198
|
|
|
186
199
|
Layout/SpaceBeforeComma:
|
|
187
|
-
Enabled:
|
|
200
|
+
Enabled: false
|
|
188
201
|
|
|
189
202
|
Layout/SpaceBeforeComment:
|
|
190
|
-
Enabled:
|
|
203
|
+
Enabled: false
|
|
191
204
|
|
|
192
205
|
Layout/SpaceBeforeFirstArg:
|
|
193
|
-
Enabled:
|
|
206
|
+
Enabled: false
|
|
194
207
|
|
|
195
208
|
Layout/SpaceInsideArrayLiteralBrackets:
|
|
196
|
-
Enabled:
|
|
209
|
+
Enabled: false
|
|
197
210
|
|
|
198
211
|
Layout/SpaceInsideArrayPercentLiteral:
|
|
199
|
-
Enabled:
|
|
212
|
+
Enabled: false
|
|
200
213
|
|
|
201
214
|
Layout/SpaceInsideBlockBraces:
|
|
202
|
-
Enabled:
|
|
215
|
+
Enabled: false
|
|
203
216
|
|
|
204
217
|
Layout/SpaceInsideHashLiteralBraces:
|
|
205
|
-
Enabled:
|
|
218
|
+
Enabled: false
|
|
206
219
|
|
|
207
220
|
Layout/SpaceInsideParens:
|
|
208
|
-
Enabled:
|
|
221
|
+
Enabled: false
|
|
209
222
|
|
|
210
223
|
Layout/SpaceInsideStringInterpolation:
|
|
211
|
-
Enabled:
|
|
224
|
+
Enabled: false
|
|
212
225
|
|
|
213
226
|
Layout/TrailingEmptyLines:
|
|
214
227
|
Enabled: true
|
|
@@ -220,7 +233,7 @@ Lint/AmbiguousBlockAssociation:
|
|
|
220
233
|
Enabled: true
|
|
221
234
|
|
|
222
235
|
Lint/AmbiguousRegexpLiteral:
|
|
223
|
-
Enabled:
|
|
236
|
+
Enabled: false
|
|
224
237
|
|
|
225
238
|
Lint/AmbiguousOperator:
|
|
226
239
|
Enabled: true
|
|
@@ -229,7 +242,7 @@ Lint/AssignmentInCondition:
|
|
|
229
242
|
Enabled: true
|
|
230
243
|
|
|
231
244
|
Lint/BinaryOperatorWithIdenticalOperands:
|
|
232
|
-
Enabled:
|
|
245
|
+
Enabled: false
|
|
233
246
|
|
|
234
247
|
Lint/DeprecatedClassMethods:
|
|
235
248
|
Enabled: true
|
|
@@ -258,10 +271,10 @@ Lint/EmptyWhen:
|
|
|
258
271
|
AllowComments: true
|
|
259
272
|
|
|
260
273
|
Lint/EnsureReturn:
|
|
261
|
-
Enabled:
|
|
274
|
+
Enabled: false
|
|
262
275
|
|
|
263
276
|
Lint/ErbNewArguments:
|
|
264
|
-
Enabled:
|
|
277
|
+
Enabled: false
|
|
265
278
|
|
|
266
279
|
Lint/ImplicitStringConcatenation:
|
|
267
280
|
Enabled: true
|
|
@@ -273,40 +286,40 @@ Lint/LiteralAsCondition:
|
|
|
273
286
|
Enabled: true
|
|
274
287
|
|
|
275
288
|
Lint/Loop:
|
|
276
|
-
Enabled:
|
|
289
|
+
Enabled: false
|
|
277
290
|
|
|
278
291
|
Lint/MissingSuper:
|
|
279
|
-
Enabled:
|
|
292
|
+
Enabled: false
|
|
280
293
|
|
|
281
294
|
Lint/NestedMethodDefinition:
|
|
282
|
-
Enabled:
|
|
295
|
+
Enabled: false
|
|
283
296
|
|
|
284
297
|
Lint/NonDeterministicRequireOrder:
|
|
285
|
-
Enabled:
|
|
298
|
+
Enabled: false
|
|
286
299
|
|
|
287
300
|
Lint/NonLocalExitFromIterator:
|
|
288
301
|
Enabled: true
|
|
289
302
|
|
|
290
303
|
Lint/ParenthesesAsGroupedExpression:
|
|
291
|
-
Enabled:
|
|
304
|
+
Enabled: false
|
|
292
305
|
|
|
293
306
|
Lint/RedundantStringCoercion:
|
|
294
307
|
Enabled: true
|
|
295
308
|
|
|
296
309
|
Lint/RedundantWithIndex:
|
|
297
|
-
Enabled:
|
|
310
|
+
Enabled: false
|
|
298
311
|
|
|
299
312
|
Lint/RescueException:
|
|
300
313
|
Enabled: true
|
|
301
314
|
|
|
302
315
|
Lint/RequireParentheses:
|
|
303
|
-
Enabled:
|
|
316
|
+
Enabled: false
|
|
304
317
|
|
|
305
318
|
Lint/ReturnInVoidContext:
|
|
306
|
-
Enabled:
|
|
319
|
+
Enabled: false
|
|
307
320
|
|
|
308
321
|
Lint/ScriptPermission:
|
|
309
|
-
Enabled:
|
|
322
|
+
Enabled: false
|
|
310
323
|
|
|
311
324
|
Lint/ShadowedArgument:
|
|
312
325
|
Enabled: true
|
|
@@ -318,25 +331,25 @@ Lint/SuppressedException:
|
|
|
318
331
|
Enabled: true
|
|
319
332
|
|
|
320
333
|
Lint/ToJSON:
|
|
321
|
-
Enabled:
|
|
334
|
+
Enabled: false
|
|
322
335
|
|
|
323
336
|
Lint/UnderscorePrefixedVariableName:
|
|
324
337
|
Enabled: true
|
|
325
338
|
|
|
326
339
|
Lint/UnifiedInteger:
|
|
327
|
-
Enabled:
|
|
340
|
+
Enabled: false
|
|
328
341
|
|
|
329
342
|
Lint/UnreachableCode:
|
|
330
343
|
Enabled: true
|
|
331
344
|
|
|
332
345
|
Lint/UnusedBlockArgument:
|
|
333
|
-
Enabled:
|
|
346
|
+
Enabled: false
|
|
334
347
|
|
|
335
348
|
Lint/UnusedMethodArgument:
|
|
336
|
-
Enabled:
|
|
349
|
+
Enabled: false
|
|
337
350
|
|
|
338
351
|
Lint/UselessAssignment:
|
|
339
|
-
Enabled:
|
|
352
|
+
Enabled: false
|
|
340
353
|
|
|
341
354
|
Lint/UriEscapeUnescape:
|
|
342
355
|
Enabled: true
|
|
@@ -348,41 +361,75 @@ Lint/RedundantCopDisableDirective:
|
|
|
348
361
|
Enabled: true
|
|
349
362
|
|
|
350
363
|
Metrics:
|
|
351
|
-
Enabled:
|
|
364
|
+
Enabled: false
|
|
352
365
|
|
|
353
366
|
Naming/AccessorMethodName:
|
|
354
|
-
Enabled:
|
|
367
|
+
Enabled: false
|
|
355
368
|
|
|
356
369
|
Naming/FileName:
|
|
357
|
-
Enabled:
|
|
370
|
+
Enabled: false
|
|
358
371
|
|
|
359
372
|
Naming/HeredocDelimiterNaming:
|
|
360
|
-
Enabled:
|
|
373
|
+
Enabled: false
|
|
361
374
|
|
|
362
375
|
Naming/MemoizedInstanceVariableName:
|
|
363
|
-
Enabled:
|
|
376
|
+
Enabled: false
|
|
364
377
|
|
|
365
378
|
Naming/MethodName:
|
|
366
|
-
Enabled:
|
|
379
|
+
Enabled: false
|
|
367
380
|
|
|
368
381
|
Naming/MethodParameterName:
|
|
369
|
-
Enabled:
|
|
382
|
+
Enabled: false
|
|
370
383
|
|
|
371
384
|
Naming/PredicateName:
|
|
372
|
-
Enabled:
|
|
385
|
+
Enabled: false
|
|
373
386
|
|
|
374
387
|
Naming/RescuedExceptionsVariableName:
|
|
375
|
-
Enabled:
|
|
388
|
+
Enabled: false
|
|
376
389
|
|
|
377
390
|
Naming/VariableName:
|
|
378
|
-
Enabled:
|
|
391
|
+
Enabled: false
|
|
379
392
|
|
|
380
393
|
Naming/VariableNumber:
|
|
394
|
+
Enabled: false
|
|
395
|
+
|
|
396
|
+
Rails/ActiveRecordOverride:
|
|
381
397
|
Enabled: true
|
|
382
398
|
|
|
383
|
-
|
|
399
|
+
Rails/AfterCommitOverride:
|
|
400
|
+
Enabled: true
|
|
401
|
+
|
|
402
|
+
Rails/Blank:
|
|
403
|
+
Enabled: true
|
|
404
|
+
|
|
405
|
+
Rails/Date:
|
|
406
|
+
Enabled: true
|
|
407
|
+
|
|
408
|
+
Rails/EnvironmentComparison:
|
|
409
|
+
Enabled: true
|
|
410
|
+
|
|
411
|
+
Rails/Exit:
|
|
412
|
+
Enabled: true
|
|
413
|
+
|
|
414
|
+
Rails/FilePath:
|
|
415
|
+
Enabled: true
|
|
416
|
+
EnforcedStyle: arguments
|
|
417
|
+
|
|
418
|
+
Rails/FindEach:
|
|
419
|
+
Enabled: true
|
|
420
|
+
|
|
421
|
+
Rails/Output:
|
|
422
|
+
Enabled: true
|
|
423
|
+
|
|
424
|
+
Rails/Present:
|
|
425
|
+
Enabled: true
|
|
426
|
+
|
|
427
|
+
Rails/SaveBang:
|
|
384
428
|
Enabled: true
|
|
385
429
|
|
|
430
|
+
Security/Eval:
|
|
431
|
+
Enabled: false
|
|
432
|
+
|
|
386
433
|
Security/JSONLoad:
|
|
387
434
|
Enabled: true
|
|
388
435
|
|
|
@@ -393,236 +440,248 @@ Security/YAMLLoad:
|
|
|
393
440
|
Enabled: true
|
|
394
441
|
|
|
395
442
|
Style/Alias:
|
|
396
|
-
Enabled:
|
|
443
|
+
Enabled: false
|
|
397
444
|
|
|
398
445
|
Style/AndOr:
|
|
399
|
-
Enabled:
|
|
446
|
+
Enabled: false
|
|
400
447
|
|
|
401
448
|
Style/AutoResourceCleanup:
|
|
402
449
|
Enabled: true
|
|
403
450
|
|
|
404
451
|
Style/BarePercentLiterals:
|
|
405
|
-
Enabled:
|
|
452
|
+
Enabled: false
|
|
406
453
|
|
|
407
454
|
Style/BisectedAttrAccessor:
|
|
408
455
|
Enabled: true
|
|
409
456
|
|
|
410
457
|
Style/BlockComments:
|
|
411
|
-
Enabled:
|
|
458
|
+
Enabled: false
|
|
412
459
|
|
|
413
460
|
Style/BlockDelimiters:
|
|
414
|
-
Enabled:
|
|
461
|
+
Enabled: false
|
|
415
462
|
|
|
416
463
|
Style/CaseEquality:
|
|
417
|
-
Enabled:
|
|
464
|
+
Enabled: false
|
|
418
465
|
|
|
419
466
|
Style/CharacterLiteral:
|
|
420
|
-
Enabled:
|
|
467
|
+
Enabled: false
|
|
421
468
|
|
|
422
469
|
Style/ClassAndModuleChildren:
|
|
423
|
-
Enabled:
|
|
470
|
+
Enabled: false
|
|
424
471
|
|
|
425
472
|
Style/ClassCheck:
|
|
426
|
-
Enabled:
|
|
473
|
+
Enabled: false
|
|
427
474
|
|
|
428
475
|
Style/ClassMethods:
|
|
429
|
-
Enabled:
|
|
476
|
+
Enabled: false
|
|
430
477
|
|
|
431
478
|
Style/ClassVars:
|
|
432
|
-
Enabled:
|
|
479
|
+
Enabled: false
|
|
433
480
|
|
|
434
481
|
Style/ColonMethodCall:
|
|
435
|
-
Enabled:
|
|
482
|
+
Enabled: false
|
|
436
483
|
|
|
437
484
|
Style/CommandLiteral:
|
|
438
|
-
Enabled:
|
|
485
|
+
Enabled: false
|
|
439
486
|
|
|
440
487
|
Style/CommentAnnotation:
|
|
441
|
-
Enabled:
|
|
488
|
+
Enabled: false
|
|
442
489
|
|
|
443
490
|
Style/CommentedKeyword:
|
|
444
|
-
Enabled:
|
|
491
|
+
Enabled: false
|
|
445
492
|
|
|
446
493
|
Style/ConditionalAssignment:
|
|
447
|
-
Enabled:
|
|
494
|
+
Enabled: false
|
|
448
495
|
|
|
449
496
|
Style/DateTime:
|
|
450
497
|
Enabled: true
|
|
451
498
|
|
|
452
499
|
Style/DefWithParentheses:
|
|
453
|
-
Enabled:
|
|
500
|
+
Enabled: false
|
|
454
501
|
|
|
455
502
|
Style/Dir:
|
|
456
|
-
Enabled:
|
|
503
|
+
Enabled: false
|
|
457
504
|
|
|
458
505
|
Style/Documentation:
|
|
459
|
-
Enabled:
|
|
506
|
+
Enabled: false
|
|
460
507
|
|
|
461
508
|
Style/DoubleNegation:
|
|
462
|
-
Enabled:
|
|
509
|
+
Enabled: false
|
|
463
510
|
|
|
464
511
|
Style/EachWithObject:
|
|
465
|
-
Enabled:
|
|
512
|
+
Enabled: false
|
|
466
513
|
|
|
467
514
|
Style/EmptyBlockParameter:
|
|
468
|
-
Enabled:
|
|
515
|
+
Enabled: false
|
|
469
516
|
|
|
470
517
|
Style/EmptyElse:
|
|
471
|
-
Enabled:
|
|
518
|
+
Enabled: false
|
|
472
519
|
|
|
473
520
|
Style/EmptyLiteral:
|
|
474
|
-
Enabled:
|
|
521
|
+
Enabled: false
|
|
475
522
|
|
|
476
523
|
Style/EmptyMethod:
|
|
477
524
|
Enabled: true
|
|
478
525
|
EnforcedStyle: expanded
|
|
479
526
|
|
|
480
527
|
Style/EvalWithLocation:
|
|
481
|
-
Enabled:
|
|
528
|
+
Enabled: false
|
|
482
529
|
|
|
483
530
|
Style/EvenOdd:
|
|
484
|
-
Enabled:
|
|
531
|
+
Enabled: false
|
|
485
532
|
|
|
486
533
|
Style/ExpandPathArguments:
|
|
487
|
-
Enabled:
|
|
534
|
+
Enabled: false
|
|
488
535
|
|
|
489
536
|
Style/FloatDivision:
|
|
490
|
-
Enabled:
|
|
537
|
+
Enabled: false
|
|
491
538
|
|
|
492
539
|
Style/For:
|
|
493
|
-
Enabled:
|
|
540
|
+
Enabled: false
|
|
494
541
|
|
|
495
542
|
Style/FormatString:
|
|
496
|
-
Enabled:
|
|
543
|
+
Enabled: false
|
|
497
544
|
|
|
498
545
|
Style/FormatStringToken:
|
|
499
|
-
Enabled:
|
|
546
|
+
Enabled: false
|
|
500
547
|
|
|
501
548
|
Style/FrozenStringLiteralComment:
|
|
502
|
-
Enabled:
|
|
549
|
+
Enabled: false
|
|
503
550
|
|
|
504
551
|
Style/GlobalVars:
|
|
505
|
-
Enabled:
|
|
552
|
+
Enabled: false
|
|
506
553
|
|
|
507
554
|
Style/GuardClause:
|
|
508
|
-
Enabled:
|
|
555
|
+
Enabled: false
|
|
509
556
|
|
|
510
557
|
Style/HashSyntax:
|
|
511
|
-
Enabled:
|
|
558
|
+
Enabled: false
|
|
512
559
|
|
|
513
560
|
Style/IdenticalConditionalBranches:
|
|
514
|
-
Enabled:
|
|
561
|
+
Enabled: false
|
|
515
562
|
|
|
516
563
|
Style/IfInsideElse:
|
|
517
|
-
Enabled:
|
|
564
|
+
Enabled: false
|
|
518
565
|
|
|
519
566
|
Style/IfUnlessModifier:
|
|
520
|
-
Enabled:
|
|
567
|
+
Enabled: false
|
|
521
568
|
|
|
522
569
|
Style/ImplicitRuntimeError:
|
|
523
570
|
Enabled: true
|
|
524
571
|
|
|
525
572
|
Style/InfiniteLoop:
|
|
526
|
-
Enabled:
|
|
573
|
+
Enabled: false
|
|
527
574
|
|
|
528
575
|
Style/InverseMethods:
|
|
529
|
-
Enabled:
|
|
576
|
+
Enabled: false
|
|
530
577
|
|
|
531
578
|
Style/IpAddresses:
|
|
532
579
|
Enabled: true
|
|
533
580
|
|
|
534
581
|
Style/Lambda:
|
|
535
|
-
Enabled:
|
|
582
|
+
Enabled: false
|
|
536
583
|
|
|
537
584
|
Style/LineEndConcatenation:
|
|
538
|
-
Enabled:
|
|
585
|
+
Enabled: false
|
|
539
586
|
|
|
540
587
|
Style/MethodCallWithoutArgsParentheses:
|
|
541
|
-
Enabled:
|
|
588
|
+
Enabled: false
|
|
542
589
|
|
|
543
590
|
Style/MethodCallWithArgsParentheses:
|
|
544
|
-
Enabled:
|
|
591
|
+
Enabled: false
|
|
545
592
|
|
|
546
593
|
Style/MethodDefParentheses:
|
|
547
|
-
Enabled:
|
|
594
|
+
Enabled: false
|
|
548
595
|
|
|
549
596
|
Style/MixinUsage:
|
|
550
|
-
Enabled:
|
|
597
|
+
Enabled: false
|
|
598
|
+
|
|
599
|
+
Style/MultilineIfThen:
|
|
600
|
+
Enabled: false
|
|
601
|
+
|
|
602
|
+
Style/MultilineTernaryOperator:
|
|
603
|
+
Enabled: false
|
|
604
|
+
|
|
605
|
+
Style/MultipleComparison:
|
|
606
|
+
Enabled: false
|
|
551
607
|
|
|
552
608
|
Style/MutableConstant:
|
|
553
609
|
Enabled: true
|
|
554
610
|
|
|
555
611
|
Style/NegatedIf:
|
|
556
|
-
Enabled:
|
|
612
|
+
Enabled: false
|
|
557
613
|
|
|
558
614
|
Style/NegatedWhile:
|
|
559
|
-
Enabled:
|
|
615
|
+
Enabled: false
|
|
560
616
|
|
|
561
617
|
Style/NestedParenthesizedCalls:
|
|
562
|
-
Enabled:
|
|
618
|
+
Enabled: false
|
|
563
619
|
|
|
564
620
|
Style/NestedTernaryOperator:
|
|
565
|
-
Enabled:
|
|
621
|
+
Enabled: false
|
|
566
622
|
|
|
567
623
|
Style/Next:
|
|
568
|
-
Enabled:
|
|
624
|
+
Enabled: false
|
|
569
625
|
|
|
570
626
|
Style/NilComparison:
|
|
571
|
-
Enabled:
|
|
627
|
+
Enabled: false
|
|
572
628
|
|
|
573
629
|
Style/NonNilCheck:
|
|
574
|
-
Enabled:
|
|
630
|
+
Enabled: false
|
|
575
631
|
|
|
576
632
|
Style/Not:
|
|
577
|
-
Enabled:
|
|
633
|
+
Enabled: false
|
|
578
634
|
|
|
579
635
|
Style/NumericLiterals:
|
|
580
|
-
Enabled:
|
|
636
|
+
Enabled: false
|
|
581
637
|
|
|
582
638
|
Style/NumericPredicate:
|
|
583
|
-
Enabled:
|
|
639
|
+
Enabled: false
|
|
584
640
|
|
|
585
641
|
Style/OneLineConditional:
|
|
586
|
-
Enabled:
|
|
642
|
+
Enabled: false
|
|
587
643
|
|
|
588
644
|
Style/OptionalArguments:
|
|
589
|
-
Enabled:
|
|
645
|
+
Enabled: false
|
|
590
646
|
|
|
591
647
|
Style/OrAssignment:
|
|
592
|
-
Enabled:
|
|
648
|
+
Enabled: false
|
|
593
649
|
|
|
594
650
|
Style/ParallelAssignment:
|
|
595
|
-
Enabled:
|
|
651
|
+
Enabled: false
|
|
596
652
|
|
|
597
653
|
Style/ParenthesesAroundCondition:
|
|
598
|
-
Enabled:
|
|
654
|
+
Enabled: false
|
|
599
655
|
|
|
600
656
|
Style/PercentLiteralDelimiters:
|
|
601
|
-
Enabled:
|
|
657
|
+
Enabled: false
|
|
602
658
|
|
|
603
659
|
Style/PerlBackrefs:
|
|
604
|
-
Enabled:
|
|
660
|
+
Enabled: false
|
|
605
661
|
|
|
606
662
|
Style/PreferredHashMethods:
|
|
607
|
-
Enabled:
|
|
663
|
+
Enabled: false
|
|
608
664
|
|
|
609
665
|
Style/Proc:
|
|
610
|
-
Enabled:
|
|
666
|
+
Enabled: false
|
|
667
|
+
|
|
668
|
+
Style/RaiseArgs:
|
|
669
|
+
Enabled: false
|
|
611
670
|
|
|
612
671
|
Style/RandomWithOffset:
|
|
613
|
-
Enabled:
|
|
672
|
+
Enabled: false
|
|
614
673
|
|
|
615
674
|
Style/RedundantBegin:
|
|
616
|
-
Enabled:
|
|
675
|
+
Enabled: false
|
|
617
676
|
|
|
618
677
|
Style/RedundantCondition:
|
|
619
|
-
Enabled:
|
|
678
|
+
Enabled: false
|
|
620
679
|
|
|
621
680
|
Style/RedundantConditional:
|
|
622
|
-
Enabled:
|
|
681
|
+
Enabled: false
|
|
623
682
|
|
|
624
683
|
Style/RedundantFreeze:
|
|
625
|
-
Enabled:
|
|
684
|
+
Enabled: false
|
|
626
685
|
|
|
627
686
|
Style/RedundantInterpolation:
|
|
628
687
|
Enabled: true
|
|
@@ -635,19 +694,19 @@ Style/RedundantParentheses:
|
|
|
635
694
|
Enabled: true
|
|
636
695
|
|
|
637
696
|
Style/RedundantReturn:
|
|
638
|
-
Enabled:
|
|
697
|
+
Enabled: false
|
|
639
698
|
|
|
640
699
|
Style/RedundantSelf:
|
|
641
|
-
Enabled:
|
|
700
|
+
Enabled: false
|
|
642
701
|
|
|
643
702
|
Style/RedundantSort:
|
|
644
|
-
Enabled:
|
|
703
|
+
Enabled: false
|
|
645
704
|
|
|
646
705
|
Style/RegexpLiteral:
|
|
647
|
-
Enabled:
|
|
706
|
+
Enabled: false
|
|
648
707
|
|
|
649
708
|
Style/RescueModifier:
|
|
650
|
-
Enabled:
|
|
709
|
+
Enabled: false
|
|
651
710
|
|
|
652
711
|
Style/RescueStandardError:
|
|
653
712
|
Enabled: true
|
|
@@ -657,74 +716,74 @@ Style/SafeNavigation:
|
|
|
657
716
|
Enabled: true
|
|
658
717
|
|
|
659
718
|
Style/SelfAssignment:
|
|
660
|
-
Enabled:
|
|
719
|
+
Enabled: false
|
|
661
720
|
|
|
662
721
|
Style/Semicolon:
|
|
663
|
-
Enabled:
|
|
722
|
+
Enabled: false
|
|
664
723
|
|
|
665
724
|
Style/SignalException:
|
|
666
725
|
Enabled: true
|
|
667
726
|
|
|
668
727
|
Style/SingleLineMethods:
|
|
669
|
-
Enabled:
|
|
728
|
+
Enabled: false
|
|
670
729
|
|
|
671
730
|
Style/SpecialGlobalVars:
|
|
672
|
-
Enabled:
|
|
731
|
+
Enabled: false
|
|
673
732
|
|
|
674
733
|
Style/StderrPuts:
|
|
675
|
-
Enabled:
|
|
734
|
+
Enabled: false
|
|
676
735
|
|
|
677
736
|
Style/SymbolArray:
|
|
678
|
-
Enabled:
|
|
737
|
+
Enabled: false
|
|
679
738
|
|
|
680
739
|
Style/StringLiterals:
|
|
681
|
-
Enabled:
|
|
740
|
+
Enabled: false
|
|
682
741
|
|
|
683
742
|
Style/StringLiteralsInInterpolation:
|
|
684
|
-
Enabled:
|
|
743
|
+
Enabled: false
|
|
685
744
|
|
|
686
745
|
Style/SymbolLiteral:
|
|
687
|
-
Enabled:
|
|
746
|
+
Enabled: false
|
|
688
747
|
|
|
689
748
|
Style/SymbolProc:
|
|
690
|
-
Enabled:
|
|
749
|
+
Enabled: false
|
|
691
750
|
|
|
692
751
|
Style/TernaryParentheses:
|
|
693
|
-
Enabled:
|
|
752
|
+
Enabled: false
|
|
694
753
|
|
|
695
754
|
Style/TrailingCommaInArguments:
|
|
696
755
|
Enabled: true
|
|
697
756
|
|
|
698
757
|
Style/TrailingCommaInArrayLiteral:
|
|
699
|
-
Enabled:
|
|
758
|
+
Enabled: false
|
|
700
759
|
|
|
701
760
|
Style/TrailingCommaInHashLiteral:
|
|
702
761
|
Enabled: true
|
|
703
762
|
EnforcedStyleForMultiline: comma
|
|
704
763
|
|
|
705
764
|
Style/TrailingUnderscoreVariable:
|
|
706
|
-
Enabled:
|
|
765
|
+
Enabled: false
|
|
707
766
|
|
|
708
767
|
Style/TrivialAccessors:
|
|
709
|
-
Enabled:
|
|
768
|
+
Enabled: false
|
|
710
769
|
|
|
711
770
|
Style/UnlessElse:
|
|
712
|
-
Enabled:
|
|
771
|
+
Enabled: false
|
|
713
772
|
|
|
714
773
|
Style/UnpackFirst:
|
|
715
|
-
Enabled:
|
|
774
|
+
Enabled: false
|
|
716
775
|
|
|
717
776
|
Style/WhileUntilDo:
|
|
718
|
-
Enabled:
|
|
777
|
+
Enabled: false
|
|
719
778
|
|
|
720
779
|
Style/WhileUntilModifier:
|
|
721
|
-
Enabled:
|
|
780
|
+
Enabled: false
|
|
722
781
|
|
|
723
782
|
Style/WordArray:
|
|
724
|
-
Enabled:
|
|
783
|
+
Enabled: false
|
|
725
784
|
|
|
726
785
|
Style/YodaCondition:
|
|
727
|
-
Enabled:
|
|
786
|
+
Enabled: false
|
|
728
787
|
|
|
729
788
|
Style/ZeroLengthPredicate:
|
|
730
789
|
Enabled: true
|
metadata
CHANGED
|
@@ -1,14 +1,14 @@
|
|
|
1
1
|
--- !ruby/object:Gem::Specification
|
|
2
2
|
name: rubocop-amagi
|
|
3
3
|
version: !ruby/object:Gem::Version
|
|
4
|
-
version: 0.0
|
|
4
|
+
version: 1.0.0
|
|
5
5
|
platform: ruby
|
|
6
6
|
authors:
|
|
7
7
|
- Cloudport Team
|
|
8
8
|
autorequire:
|
|
9
9
|
bindir: bin
|
|
10
10
|
cert_chain: []
|
|
11
|
-
date:
|
|
11
|
+
date: 2022-07-12 00:00:00.000000000 Z
|
|
12
12
|
dependencies: []
|
|
13
13
|
description: amagi rubocop rules
|
|
14
14
|
email: cloudport.team@amagi.com
|
|
@@ -16,6 +16,9 @@ executables: []
|
|
|
16
16
|
extensions: []
|
|
17
17
|
extra_rdoc_files: []
|
|
18
18
|
files:
|
|
19
|
+
- lib/cops/duplicate_constant_cop.rb
|
|
20
|
+
- lib/cops/logger_format_cop.rb
|
|
21
|
+
- lib/cops/readme.md
|
|
19
22
|
- rubocop.yml
|
|
20
23
|
homepage: https://github.com/amagimedia/rubocop-amagi
|
|
21
24
|
licenses:
|
|
@@ -36,7 +39,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
|
|
|
36
39
|
- !ruby/object:Gem::Version
|
|
37
40
|
version: '0'
|
|
38
41
|
requirements: []
|
|
39
|
-
rubygems_version: 3.2.
|
|
42
|
+
rubygems_version: 3.2.22
|
|
40
43
|
signing_key:
|
|
41
44
|
specification_version: 4
|
|
42
45
|
summary: amagi rubocop rules
|