-
Notifications
You must be signed in to change notification settings - Fork 117
/
.rubocop.yml
139 lines (99 loc) · 2.33 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
#
# 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
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
Lint/RedundantCopDisableDirective:
Enabled: false
# Seems buggy - https://github.com/bbatsov/rubocop/issues/2690
MultilineOperationIndentation:
Enabled: false
# We don't use CAPS just for constants.
Style/MutableConstant:
Enabled: false
# Bug with constants? https://phabricator.fb.com/P56108678
Style/ConditionalAssignment:
Enabled: false
#
# Modified rules
#
Layout/LineLength:
Max: 80
Metrics/BlockLength:
Max: 250
Layout/DotPosition:
EnforcedStyle: trailing
Layout/EmptyLineAfterGuardClause:
Enabled: false
Style/HashSyntax:
EnforcedStyle: hash_rockets
Style/Documentation:
Enabled: false
Style/TrailingCommaInArrayLiteral:
EnforcedStyleForMultiline: comma
Style/TrailingCommaInHashLiteral:
EnforcedStyleForMultiline: comma
Style/TrailingCommaInArguments:
EnforcedStyleForMultiline: comma
Layout/FirstArrayElementIndentation:
EnforcedStyle: consistent
Style/SignalException:
EnforcedStyle: semantic
Style/FrozenStringLiteralComment:
EnforcedStyle: never