Merge pull request #688 from Shopify/rm-layout-department

Enable cops in the Layout Department

Author: rafaelfranca

rubocop.yml

@@ -41,11 +41,7 @@ Gemspec/RequiredRubyVersion:
 
 
 # Layout Department
-Layout/AccessModifierIndentation:
-  Enabled: false
-
 Layout/ArgumentAlignment:
-  Enabled: false
   EnforcedStyle: with_fixed_indentation
 
 Layout/ArrayAlignment:
@@ -70,73 +66,19 @@ Layout/CaseIndentation:
 Layout/ClosingHeredocIndentation:
   Enabled: false
 
-Layout/ClosingParenthesisIndentation:
-  Enabled: false
-
-Layout/CommentIndentation:
-  Enabled: false
-
-Layout/ConditionPosition:
-  Enabled: false
-
-Layout/DefEndAlignment:
-  Enabled: false
-
-Layout/DotPosition:
-  Enabled: false
-
-Layout/ElseAlignment:
-  Enabled: false
-
 Layout/EmptyComment:
   Enabled: false
 
 Layout/EmptyLineAfterGuardClause:
   Enabled: false
 
-Layout/EmptyLineAfterMagicComment:
-  Enabled: false
-
-Layout/EmptyLineBetweenDefs:
-  Enabled: false
-
 Layout/EmptyLines:
   Enabled: false
 
-Layout/EmptyLinesAroundAccessModifier:
-  Enabled: false
-
-Layout/EmptyLinesAroundArguments:
-  Enabled: false
-
-Layout/EmptyLinesAroundAttributeAccessor:
-  Enabled: false
-
-Layout/EmptyLinesAroundBeginBody:
-  Enabled: false
-
-Layout/EmptyLinesAroundBlockBody:
-  Enabled: false
-
-Layout/EmptyLinesAroundClassBody:
-  Enabled: false
-
-Layout/EmptyLinesAroundExceptionHandlingKeywords:
-  Enabled: false
-
-Layout/EmptyLinesAroundMethodBody:
-  Enabled: false
-
-Layout/EmptyLinesAroundModuleBody:
-  Enabled: false
-
 Layout/EndAlignment:
   Enabled: false
   EnforcedStyleAlignWith: variable
 
-Layout/EndOfLine:
-  Enabled: false
-
 Layout/ExtraSpacing:
   Enabled: false
 
@@ -168,29 +110,16 @@ Layout/HashAlignment:
 Layout/HeredocIndentation:
   Enabled: false
 
-Layout/IndentationConsistency:
-  Enabled: false
-
-Layout/IndentationStyle:
-  Enabled: false
-
-Layout/IndentationWidth:
-  Enabled: false
-
-Layout/InitialIndentation:
-  Enabled: false
-
+<% if rubocop_version >= "1.67" %>
 Layout/LeadingCommentSpace:
-  Enabled: false
-
-Layout/LeadingEmptyLines:
-  Enabled: false
+  AllowRBSInlineAnnotation: true
+<% end %>
 
 Layout/LineContinuationLeadingSpace:
-  Enabled: false
+  Enabled: true
 
 Layout/LineContinuationSpacing:
-  Enabled: false
+  Enabled: true
 
 Layout/LineEndStringConcatenationIndentation:
   Enabled: false
@@ -208,9 +137,6 @@ Layout/MultilineArrayBraceLayout:
 Layout/MultilineArrayLineBreaks:
   AllowMultilineFinalElement: true
 
-Layout/MultilineBlockLayout:
-  Enabled: false
-
 Layout/MultilineHashBraceLayout:
   Enabled: false
 
@@ -236,89 +162,8 @@ Layout/ParameterAlignment:
 Layout/RescueEnsureAlignment:
   Enabled: false
 
-Layout/SpaceAfterColon:
-  Enabled: false
-
-Layout/SpaceAfterComma:
-  Enabled: false
-
-Layout/SpaceAfterMethodName:
-  Enabled: false
-
-Layout/SpaceAfterNot:
-  Enabled: false
-
-Layout/SpaceAfterSemicolon:
-  Enabled: false
-
-Layout/SpaceAroundBlockParameters:
-  Enabled: false
-
-Layout/SpaceAroundEqualsInParameterDefault:
-  Enabled: false
-
-Layout/SpaceAroundKeyword:
-  Enabled: false
-
-Layout/SpaceAroundMethodCallOperator:
-  Enabled: false
-
-Layout/SpaceAroundOperators:
-  Enabled: false
-
-Layout/SpaceBeforeBlockBraces:
-  Enabled: false
-
 Layout/SpaceBeforeBrackets:
-  Enabled: false
-
-Layout/SpaceBeforeComma:
-  Enabled: false
-
-Layout/SpaceBeforeComment:
-  Enabled: false
-
-Layout/SpaceBeforeFirstArg:
-  Enabled: false
-
-Layout/SpaceBeforeSemicolon:
-  Enabled: false
-
-Layout/SpaceInLambdaLiteral:
-  Enabled: false
-
-Layout/SpaceInsideArrayLiteralBrackets:
-  Enabled: false
-
-Layout/SpaceInsideArrayPercentLiteral:
-  Enabled: false
-
-Layout/SpaceInsideBlockBraces:
-  Enabled: false
-
-Layout/SpaceInsideHashLiteralBraces:
-  Enabled: false
-
-Layout/SpaceInsideParens:
-  Enabled: false
-
-Layout/SpaceInsidePercentLiteralDelimiters:
-  Enabled: false
-
-Layout/SpaceInsideRangeLiteral:
-  Enabled: false
-
-Layout/SpaceInsideReferenceBrackets:
-  Enabled: false
-
-Layout/SpaceInsideStringInterpolation:
-  Enabled: false
-
-Layout/TrailingEmptyLines:
-  Enabled: false
-
-Layout/TrailingWhitespace:
-  Enabled: false
+  Enabled: true
 
 
 # Lint Department