forked from facebook/chef-cookbooks
-
Notifications
You must be signed in to change notification settings - Fork 0
/
.rubocop.yml
193 lines (141 loc) · 3.48 KB
/
.rubocop.yml
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
AllCops:
TargetRubyVersion: 2.1
Exclude:
# template files named `rb` instead of `erb` are a sin against ruby-nature.
- '**/templates/**/*.rb'#
# Disabled rules
#
Encoding:
Enabled: false
NumericLiterals:
Enabled: false
MultilineTernaryOperator:
Enabled: false
ModuleLength:
Enabled: false
MethodLength:
Enabled: false
ClassLength:
Enabled: false
# It would forbid action [:enable, :start]
SymbolArray:
Enabled: false
# This recommends using external gems to parse heredocs
Layout/IndentHeredoc:
Enabled: false
CyclomaticComplexity:
Enabled: false
# So just keep iterating instead of breaking? wtf.
Next:
Enabled: false
# While this can be nice, it also can promote errors. Let people
# use what's comfortable for them
GuardClause:
Enabled: false
AbcSize:
Enabled: false
# less readable, not more
IfUnlessModifier:
Enabled: false
# Really?
PerlBackrefs:
Enabled: false
# Unrealistic
BlockNesting:
Enabled: false
# Disabled because of the way 'variables' works.
BracesAroundHashParameters:
Enabled: false
WordArray:
Enabled: false
RedundantReturn:
Enabled: false
RedundantSelf:
Enabled: false
CommentAnnotation:
Enabled: false
# this trips on *any* method called 'get_*' wtf.
AccessorMethodName:
Enabled: false
# backslash is extra dumb in ruby, we want the OPPOSITE of this rule
LineEndConcatenation:
Enabled: false
# this isn't testing for consistency it always wants %w() which is dumb
PercentLiteralDelimiters:
Enabled: false
# it wants File.exist? instead of File.exists?
DeprecatedClassMethods:
Enabled: false
# This blows up on things like base_packages-redhat
FileName:
Enabled: false
# I don't know what it's metric for "human complexity" is, but it's wrong.
PerceivedComplexity:
Enabled: false
UnneededDisable:
Enabled: false
# Seems buggy - https://github.com/bbatsov/rubocop/issues/2690
MultilineOperationIndentation:
Enabled: false
# buggy: https://github.com/bbatsov/rubocop/issues/2639
Performance/RedundantMatch:
Enabled: false
# We'll .times.map all we want.
Performance/TimesMap:
Enabled: false
# https://github.com/bbatsov/rubocop/issues/2676
Performance/RedundantMerge:
Enabled: false
# Bug with constants? https://phabricator.fb.com/P56108678
Style/ConditionalAssignment:
Enabled: false
# Heredocs are fine with "EOF"
Naming/HeredocDelimiterNaming:
Enabled: false
# This double-reports what happens in EmptyLines
Layout/EmptyLineBetweenDefs:
Enabled: false
# this is supposedly the default, but turns not not really
# https://github.com/bbatsov/rubocop/issues/5277
Layout/SpaceBeforeBlockBraces:
EnforcedStyleForEmptyBraces: space
#
# Modified rules
#
LineLength:
Max: 80
DotPosition:
EnforcedStyle: trailing
HashSyntax:
EnforcedStyle: hash_rockets
Style/Documentation:
Enabled: false
TrailingCommaInLiteral:
EnforcedStyleForMultiline: comma
TrailingCommaInArguments:
EnforcedStyleForMultiline: comma
Style/FormatStringToken:
Enabled: false
Layout/IndentArray:
EnforcedStyle: consistent
Style/SignalException:
EnforcedStyle: semantic
Style/NumericLiteralPrefix:
Enabled: false
Naming/VariableNumber:
Enabled: false
Style/RegexpLiteral:
EnforcedStyle: mixed
Metrics/BlockLength:
Enabled: false
Style/PercentLiteralDelimiters:
PreferredDelimiters:
default: {}
'%i': {}
'%w': {}
# no, we're not putting parens around `lazy`
Lint/AmbiguousBlockAssociation:
Enabled: false
# this is misfiring in multiple cookbooks
Layout/EmptyLinesAroundArguments:
Enabled: false