Move already in use Rails rules to basic cops
We were already applying the FindEach rule since commit cae210c1, while
the EnumUniqueness rule is essential when we use `enum`, and the
EnvironmentComparison rule is very useful since it forces us to use a
format the UnknownEnv rule will recognize.
This commit is contained in:
@@ -15,18 +15,9 @@ Performance/EndWith:
|
|||||||
Performance/StartWith:
|
Performance/StartWith:
|
||||||
Enabled: true
|
Enabled: true
|
||||||
|
|
||||||
Rails/EnumUniqueness:
|
|
||||||
Enabled: true
|
|
||||||
|
|
||||||
Rails/EnvironmentComparison:
|
|
||||||
Enabled: true
|
|
||||||
|
|
||||||
Rails/Exit:
|
Rails/Exit:
|
||||||
Enabled: true
|
Enabled: true
|
||||||
|
|
||||||
Rails/FindEach:
|
|
||||||
Enabled: true
|
|
||||||
|
|
||||||
Rails/HasAndBelongsToMany:
|
Rails/HasAndBelongsToMany:
|
||||||
Enabled: true
|
Enabled: true
|
||||||
|
|
||||||
|
|||||||
@@ -191,9 +191,18 @@ Rails/DynamicFindBy:
|
|||||||
Enabled: true
|
Enabled: true
|
||||||
Severity: refactor
|
Severity: refactor
|
||||||
|
|
||||||
|
Rails/EnumUniqueness:
|
||||||
|
Enabled: true
|
||||||
|
|
||||||
|
Rails/EnvironmentComparison:
|
||||||
|
Enabled: true
|
||||||
|
|
||||||
Rails/FindBy:
|
Rails/FindBy:
|
||||||
Enabled: true
|
Enabled: true
|
||||||
|
|
||||||
|
Rails/FindEach:
|
||||||
|
Enabled: true
|
||||||
|
|
||||||
Rails/HttpPositionalArguments:
|
Rails/HttpPositionalArguments:
|
||||||
Enabled: true
|
Enabled: true
|
||||||
|
|
||||||
|
|||||||
Reference in New Issue
Block a user