config/chefstyle.yml in chefstyle-0.10.0 vs config/chefstyle.yml in chefstyle-0.11.0

- old
+ new

@@ -267,10 +267,11 @@ Layout/AccessModifierIndentation: Enabled: true Layout/AlignHash: Enabled: true + EnforcedLastArgumentHashStyle: ignore_implicit Style/AndOr: Enabled: true Style/ArrayJoin: Enabled: true Naming/AsciiIdentifiers: @@ -283,18 +284,22 @@ Enabled: true Style/CharacterLiteral: Enabled: true Style/ClassMethods: Enabled: true +Layout/ClosingHeredocIndentation: + Enabled: true Style/ColonMethodCall: Enabled: true Style/CommandLiteral: Enabled: true Naming/ConstantName: Enabled: true Style/DefWithParentheses: Enabled: true +Layout/DotPosition: + Enabled: True Style/EachForSimpleLoop: Enabled: true Layout/ElseAlignment: Enabled: true Layout/EmptyLineBetweenDefs: @@ -318,26 +323,34 @@ Enabled: true Style/For: Enabled: true Style/FrozenStringLiteralComment: Enabled: true +Style/HashSyntax: + Enabled: true +Naming/HeredocDelimiterCase: + Enabled: true Style/IfUnlessModifierOfIfUnless: Enabled: true Style/IfWithSemicolon: Enabled: true Layout/IndentAssignment: Enabled: true +Layout/IndentHeredoc: + Enabled: true Layout/IndentationConsistency: Enabled: true Layout/IndentationWidth: Enabled: true Style/InfiniteLoop: Enabled: true Layout/InitialIndentation: Enabled: true Style/LambdaCall: Enabled: true +Layout/LeadingCommentSpace: + Enabled: true Style/MethodDefParentheses: Enabled: true Layout/MultilineBlockLayout: Enabled: true Style/MultilineIfModifier: @@ -351,10 +364,12 @@ Layout/MultilineOperationIndentation: Enabled: true EnforcedStyle: indented Style/MultilineTernaryOperator: Enabled: true +Style/MutableConstant: + Enabled: true Style/NegatedWhile: Enabled: true Style/NestedModifier: Enabled: true Style/NestedTernaryOperator: @@ -393,10 +408,12 @@ '%w': '{}' '%W': '{}' '%x': '{}' Style/PercentQLiterals: Enabled: true +Style/PreferredHashMethods: + Enabled: true Style/RedundantBegin: Enabled: true Style/RedundantException: Enabled: true Style/RedundantFreeze: @@ -469,20 +486,25 @@ Enabled: true Layout/TrailingBlankLines: Enabled: true Style/TrailingCommaInArguments: Enabled: true +Style/TrailingCommaInHashLiteral: + Enabled: true + EnforcedStyleForMultiline: comma # rubocop's default gets this completely backwards Style/TrailingCommaInLiteral: Enabled: true EnforcedStyleForMultiline: comma Style/TrailingUnderscoreVariable: Enabled: true Layout/TrailingWhitespace: Enabled: true Style/UnneededCapitalW: Enabled: true +Style/UnneededInterpolation: + Enabled: false # buggy: https://github.com/rubocop-hq/rubocop/issues/6099 #Style/UnneededPercentQ: # would like to enable this one but its buggy as of 0.35.1 # Enabled: true Naming/VariableName: Enabled: true Style/VariableInterpolation: @@ -520,10 +542,12 @@ Enabled: false Layout/SpaceInsideParens: Enabled: false Layout/SpaceInsidePercentLiteralDelimiters: Enabled: false +Layout/SpaceInsideArrayLiteralBrackets: + Enabled: false # more whitespace cops that we allow since they can enhance readability Layout/EmptyLinesAroundBlockBody: Enabled: false Layout/EmptyLinesAroundClassBody: @@ -599,6 +623,24 @@ Style/GuardClause: Enabled: false # Let authors order gems however makes sense Bundler/OrderedGems: + Enabled: false + +# Naming cops produce fairly pointless makework (e.g. EOF on HEREDOC's is +# perfectly fine and understandable, using 'ip' as a parameter to a function +# is also perfectly descriptive even though its only 2 characters...). +Naming/HeredocDelimiterNaming: + Enabled: false +Naming/PredicateName: + Enabled: false +Naming/UncommunicativeMethodParamName: + Enabled: false +Naming/MemoizedInstanceVariableName: + Enabled: false +Naming/VariableNumber: + Enabled: false + +# This breaks all kinds of specs in chef (i don't think it will ever quite work correctly) +Style/BracesAroundHashParameters: Enabled: false