forked from uber/client-platform-engineering
-
Notifications
You must be signed in to change notification settings - Fork 0
/
.rubocop.yml
149 lines (108 loc) · 2.59 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
#
# Disabled rules
#
Encoding:
Enabled: false
NumericLiterals:
Enabled: false
MultilineTernaryOperator:
Enabled: false
ModuleLength:
Enabled: false
MethodLength:
Enabled: false
ClassLength:
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
# 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 don't use CAPS just for constants.
Style/MutableConstant:
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
# No more 'Missing frozen string literal comment.' linting violations
Style/FrozenStringLiteralComment:
Enabled: false
#
# Modified rules
#
LineLength:
Max: 120
DotPosition:
EnforcedStyle: trailing
HashSyntax:
EnforcedStyle: hash_rockets
Style/Documentation:
Enabled: false
TrailingCommaInArrayLiteral:
EnforcedStyleForMultiline: comma
TrailingCommaInHashLiteral:
EnforcedStyleForMultiline: comma
TrailingCommaInArguments:
EnforcedStyleForMultiline: comma
Layout/IndentArray:
EnforcedStyle: consistent
Style/SignalException:
EnforcedStyle: semantic
Metrics/BlockLength:
Max: 80