cookstyle 7.32.8 → 8.0.0

data/config/chefstyle.yml

@@ -1,741 +1,892 @@
-# This is an internal RuboCop configuration for Chef Ruby projects (chef-client, ohai,
-# mixlib-shellout, mixlib-config, etc).
-#
-# It is not meant for consumption by cookbooks or for any general purpose uses.
-# **Chef Users and Customers Should Generally Not Use This Configuration and
-# Should Use cookstyle.yml**.  It is not intended for any audience outside of
-# Chef Core Ruby development.
-
-AllCops:
-  TargetRubyVersion: 2.5
-  NewCops: disable
-  DisabledByDefault: true
-  SuggestExtensions: false
-
-#
-# Bundler
-#
-
-Bundler/DuplicatedGem:
-  Enabled: false
-
-Bundler/InsecureProtocolSource:
-  Enabled: true
-  AllowHttpProtocol: false
-
-#
-# Lint
-#
-
-Lint/AmbiguousOperator:
-  Enabled: true
-Lint/AmbiguousBlockAssociation:
-  Enabled: true
-Lint/AmbiguousRegexpLiteral:
-  Enabled: true
-Lint/AssignmentInCondition:
-  Enabled: true
-Layout/BlockAlignment:
-  EnforcedStyleAlignWith: start_of_block
-  Enabled: true
-Lint/CircularArgumentReference:
-  Enabled: true
-Layout/ConditionPosition:
-  Enabled: true
-Lint/Debugger:
-  Enabled: true
-Layout/DefEndAlignment:
-  Enabled: true
-  AutoCorrect: true
-Lint/DeprecatedClassMethods:
-  Enabled: true
-Lint/DuplicateCaseCondition:
-  Enabled: true
-Lint/DuplicateMethods:
-  Enabled: true
-# Lint/DuplicatedKey: TODO broken with rubocop 1.25.1 move
-#   Enabled: true
-Lint/EachWithObjectArgument:
-  Enabled: true
-Lint/ElseLayout:
-  Enabled: true
-Lint/EmptyEnsure:
-  Enabled: true
-Lint/EmptyExpression:
-  Enabled: true
-Lint/EmptyInterpolation:
-  Enabled: true
-Lint/EmptyWhen:
-  Enabled: true
-Layout/EndAlignment:
-  Enabled: true
-  AutoCorrect: true
-# Lint/EndInMethod: TODO broken with rubocop 1.25.1 move
-#  Enabled: true
-Lint/EnsureReturn:
-  Enabled: true
-Lint/FloatOutOfRange:
-  Enabled: true
-Lint/FormatParameterMismatch:
-  Enabled: true
-Lint/ImplicitStringConcatenation:
-  Enabled: true
-Lint/InheritException:
-  Enabled: true
-Lint/IneffectiveAccessModifier:
-  Enabled: true
-Lint/LiteralAsCondition:
-  Enabled: true
-Lint/LiteralInInterpolation:
-  Enabled: true
-Lint/Loop:
-  Enabled: true
-# Lint/MultipleCompare: TODO broken with rubocop 1.25.1 move
-#  Enabled: true
-Lint/NestedMethodDefinition:
-  Enabled: true
-Lint/NextWithoutAccumulator:
-  Enabled: true
-Lint/NonLocalExitFromIterator:
-  Enabled: true
-Lint/ParenthesesAsGroupedExpression:
-  Enabled: true
-Lint/PercentStringArray:
-  Enabled: true
-Lint/PercentSymbolArray:
-  Enabled: true
-Lint/RandOne:
-  Enabled: true
-Lint/RequireParentheses:
-  Enabled: true
-Lint/SafeNavigationChain:
-  Enabled: true
-Lint/ShadowedException:
-  Enabled: true
-Lint/ShadowingOuterLocalVariable:
-  Enabled: true
-# Lint/StringConversionInInterpolation: TODO broken with rubocop 1.25.1 move
-#  Enabled: true
-Lint/UnderscorePrefixedVariableName:
-  Enabled: true
-Lint/UnifiedInteger:
-  Enabled: true
-# Lint/UnneededSplatExpansion: TODO broken with rubocop 1.25.1 move
-#   Enabled: true
-Lint/UnreachableCode:
-  Enabled: true
-Lint/UriEscapeUnescape:
-  Enabled: true
-Lint/UselessAccessModifier:
-  Enabled: true
-Lint/UselessAssignment:
-  Enabled: true
-# Lint/UselessComparison: TODO broken with rubocop 1.25.1 move
-#   Enabled: true
-Lint/UselessElseWithoutRescue:
-  Enabled: true
-Lint/UselessSetterCall:
-  Enabled: true
-Lint/Void:
-  Enabled: true
-Layout/CommentIndentation:
-  Enabled: true
-
-#
-# Disabled Lint
-#
-
-# These are disabled per https://github.com/chef/chef/pull/3038
-Lint/UnusedBlockArgument:
-  Enabled: false
-Lint/UnusedMethodArgument:
-  Enabled: false
-
-# We ignore Exceptions a lot
-Lint/SuppressedException:
-  Enabled: false
-# We also decorate Exceptions a lot
-Lint/RescueException:
-  Enabled: false
-
-# disabling this will make it easier to stage chefstyle rollouts
-Lint/RedundantCopDisableDirective:
-  Enabled: false
-
-#
-# Security
-#
-
-Security/Eval:
-  Enabled: true
-
-#
-# Metrics
-#
-
-# Metrics cops are all disabled, there would be too much refactoring
-# work to do and we have much higher priorities.
-Metrics/AbcSize:
-  Enabled: false
-Metrics/BlockNesting:
-  Enabled: false
-Metrics/BlockLength:
-  Enabled: false
-Metrics/ClassLength:
-  Enabled: false
-Metrics/CyclomaticComplexity:
-  Enabled: false
-Layout/LineLength:
-  Enabled: false
-Metrics/MethodLength:
-  Enabled: false
-Metrics/ModuleLength:
-  Enabled: false
-Metrics/ParameterLists:
-  Enabled: false
-Metrics/PerceivedComplexity:
-  Enabled: false
-
-
-#
-# Style/Naming/Layout
-#
-
-Layout/AccessModifierIndentation:
-  Enabled: true
-# Layout/AlignArguments: TODO broken with rubocop 1.25.1 move
-#   Enabled: true
-#   EnforcedStyle: with_fixed_indentation
-# Layout/AlignParameters: TODO broken with rubocop 1.25.1 move
-#   Enabled: true
-#   EnforcedStyle: with_fixed_indentation
-# the "ignore_implict" is here for keyword args in method calls which are
-# "implicit" hashes, and those should not be treated like normal hashes
-# Layout/AlignHash: TODO broken with rubocop 1.25.1 move
-#   Enabled: true
-#   EnforcedLastArgumentHashStyle: ignore_implicit
-Style/AndOr:
-  Enabled: true
-Style/ArrayJoin:
-  Enabled: true
-Naming/AsciiIdentifiers:
-  Enabled: true
-Style/Attr:
-  Enabled: true
-Style/BeginBlock:
-  Enabled: true
-# there's debate over one-liner vs semantic blockstyle, chef projects should just pick one style
-# and stick with it.
-Style/BlockDelimiters:
-  Enabled: false
-Layout/BlockEndNewline:
-  Enabled: true
-Style/CharacterLiteral:
-  Enabled: true
-Style/ClassCheck:
-  Enabled: true
-Style/ClassMethods:
-  Enabled: true
-Layout/ClosingHeredocIndentation:
-  Enabled: true
-Layout/ClosingParenthesisIndentation:
-  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/EmptyLineAfterGuardClause:
-  Enabled: true
-Layout/EmptyLineBetweenDefs:
-  Enabled: true
-  AllowAdjacentOneLineDefs: true
-Layout/EmptyLines:
-  Enabled: true
-Layout/EmptyLinesAroundAccessModifier:
-  Enabled: true
-Layout/EmptyLinesAroundMethodBody:
-  Enabled: true
-Style/EmptyLiteral:
-  Enabled: true
-Style/EmptyMethod:
-  Enabled: true
-Style/EndBlock:
-  Enabled: true
-Layout/EndOfLine:
-  Enabled: true
-  EnforcedStyle: lf
-Style/EvenOdd:
-  Enabled: true
-Layout/ExtraSpacing:
-  Enabled: true
-Lint/FlipFlop:
-  Enabled: true
-Style/For:
-  Enabled: true
-Style/HashSyntax:
-  Enabled: true
-Naming/HeredocDelimiterCase:
-  Enabled: true
-Style/IfUnlessModifierOfIfUnless:
-  Enabled: true
-Style/IfWithSemicolon:
-  Enabled: true
-# Layout/IndentAssignment: TODO broken with rubocop 1.25.1 move
-#   Enabled: true
-# Layout/IndentFirstArgument: TODO broken with rubocop 1.25.1 move
-#   EnforcedStyle: consistent
-#   Enabled: true
-# Layout/IndentHeredoc: TODO broken with rubocop 1.25.1 move
-#   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
-Style/MethodCallWithoutArgsParentheses:
-  Enabled: true
-Layout/MultilineBlockLayout:
-  Enabled: true
-Style/MultilineIfModifier:
-  Enabled: true
-Style/MultilineIfThen:
-  Enabled: true
-Style/MultilineMemoization:
-  Enabled: true
-Layout/MultilineMethodCallBraceLayout:
-  Enabled: true
-Layout/MultilineMethodCallIndentation:
-  Enabled: true
-  EnforcedStyle: indented
-Layout/MultilineMethodDefinitionBraceLayout:
-  Enabled: true
-Layout/MultilineOperationIndentation:
-  Enabled: true
-  EnforcedStyle: indented
-Style/MultilineTernaryOperator:
-  Enabled: true
-Style/MutableConstant:
-  Enabled: true
-Style/NegatedIf:
-  Enabled: true
-Style/NegatedWhile:
-  Enabled: true
-Style/NegatedUnless:
-  Enabled: true
-Style/NestedModifier:
-  Enabled: true
-Style/NestedTernaryOperator:
-  Enabled: true
-Style/NilComparison:
-  Enabled: true
-Style/NonNilCheck:
-  Enabled: true
-# `foo.zero?` is slower and substantially more annoying to read than `foo == 0`
-Style/NumericPredicate:
-  Enabled: true
-  EnforcedStyle: comparison
-Style/OneLineConditional:
-  Enabled: true
-Naming/BinaryOperatorParameterName:
-  Enabled: true
-Style/OptionalArguments:
-  Enabled: true
-Style/ParenthesesAroundCondition:
-  Enabled: true
-# - The chef source code predominantly uses {} for %,%i,%q,%Q,%r,%w,%W
-# - The old 1.9 pickaxe book predominantly uses {} for %,%q,%Q,%r,%w,%W,%x
-# - The chef source code does not use %x enough to determine a preference
-# - The old 1.9 pickaxe book does not use %i
-# - %s is not used in chef code or the pickaxe book, and is %s{} for consistency
-Style/PercentLiteralDelimiters:
-  Enabled: true
-  PreferredDelimiters:
-    '%':  '{}'
-    '%i': '{}'
-    '%I': '{}'
-    '%q': '{}'
-    '%Q': '{}'
-    '%r': '{}'
-    '%s': '{}'
-    '%w': '{}'
-    '%W': '{}'
-    '%x': '{}'
-Style/PercentQLiterals:
-  Enabled: true
-Style/PreferredHashMethods:
-  Enabled: true
-Style/RedundantBegin:
-  Enabled: true
-Style/RedundantException:
-  Enabled: true
-Style/RedundantFreeze:
-  Enabled: true
-Style/RedundantReturn:
-  Enabled: true
-Style/RedundantSelf:
-  Enabled: true
-Layout/RescueEnsureAlignment:
-  Enabled: true
-Style/RegexpLiteral:
-  Enabled: true
-# temporarily disabled, but we should do this
-Style/SafeNavigation:
-  Enabled: false
-Style/SelfAssignment:
-  Enabled: true
-Style/Semicolon:
-  AllowAsExpressionSeparator: true
-  Enabled: true
-# we only 'raise' and do not 'fail'
-Style/SignalException:
-  EnforcedStyle: only_raise
-  Enabled: true
-Layout/SpaceAfterColon:
-  Enabled: true
-Layout/SpaceAfterComma:
-  Enabled: true
-Layout/SpaceAfterMethodName:
-  Enabled: true
-Layout/SpaceAfterSemicolon:
-  Enabled: true
-Layout/SpaceAroundBlockParameters:
-  Enabled: true
-Layout/SpaceAroundEqualsInParameterDefault:
-  Enabled: true
-Layout/SpaceAroundOperators:
-  Enabled: true
-Layout/SpaceBeforeBlockBraces:
-  Enabled: true
-Layout/SpaceBeforeComment:
-  Enabled: true
-Layout/SpaceBeforeSemicolon:
-  Enabled: true
-Layout/SpaceAroundKeyword:
-  Enabled: true
-Layout/SpaceInLambdaLiteral:
-  Enabled: true
-Layout/SpaceInsideArrayPercentLiteral:
-  Enabled: true
-Layout/SpaceInsideBlockBraces:
-  Enabled: true
-Layout/SpaceInsideHashLiteralBraces:
-  Enabled: true
-Layout/SpaceInsideRangeLiteral:
-  Enabled: true
-Layout/SpaceInsideStringInterpolation:
-  Enabled: true
-Style/StabbyLambdaParentheses:
-  Enabled: true
-# core chef community prefers double quotes over single quotes
-Style/StringLiterals:
-  EnforcedStyle: double_quotes
-  Enabled: true
-Style/StringLiteralsInInterpolation:
-  EnforcedStyle: double_quotes
-  Enabled: true
-Style/StructInheritance:
-  Enabled: true
-Style/SymbolLiteral:
-  Enabled: true
-Style/StringMethods:
-  Enabled: true
-Style/SymbolArray:
-  Enabled: true
-Style/SymbolProc:
-  Enabled: true
-Layout/IndentationStyle:
-  Enabled: true
-# Layout/TrailingBlankLines: TODO broken with rubocop 1.25.1 move
-#   Enabled: true
-Style/TrailingCommaInArguments:
-  Enabled: true
-# rubocop's default gets this completely backwards
-Style/TrailingCommaInArrayLiteral:
-  Enabled: true
-  EnforcedStyleForMultiline: comma
-Style/TrailingCommaInHashLiteral:
-  Enabled: true
-  EnforcedStyleForMultiline: comma
-Style/TrailingUnderscoreVariable:
-  Enabled: true
-Layout/TrailingWhitespace:
-  Enabled: true
-# Style/UnneededCapitalW: TODO broken with rubocop 1.25.1 move
-#   Enabled: true
-Style/RedundantInterpolation:
-  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:
-  Enabled: true
-Style/WhenThen:
-  Enabled: true
-Style/WhileUntilDo:
-  Enabled: true
-Style/WhileUntilModifier:
-  Enabled: true
-Style/WordArray:
-  Enabled: true
-
-# we are ruby > 2.0 only so we can remove encoding comments for utf-8
-Style/Encoding:
-  Enabled: true
-
-#
-# Disabled Style
-#
-
-# reduces memory usage, but isn't a simple autocorrect so we need to do this one project at a time
-Style/FrozenStringLiteralComment:
-  Enabled: false
-
-# As of this commit we have 686 modules and classes without docs.
-# This is a cop that we /should/ have enabled, but tactically we can't really enable.
-Style/Documentation:
-  Enabled: false
-Style/DocumentationMethod:
-  Enabled: false
-
-# this makes whitespace formatting of DSL code impossible
-Layout/SpaceBeforeFirstArg:
-  Enabled: false
-
-# whitespace in expressions is useful to enhance readability
-Layout/SpaceInsideArrayLiteralBrackets:
-  Enabled: false
-Layout/SpaceInsideReferenceBrackets:
-  Enabled: false
-Layout/SpaceInsideParens:
-  Enabled: false
-Layout/SpaceInsidePercentLiteralDelimiters:
-  Enabled: false
-
-# more whitespace cops that we allow since they can enhance readability
-Layout/EmptyLinesAroundBlockBody:
-  Enabled: false
-Layout/EmptyLinesAroundClassBody:
-  Enabled: false
-Layout/EmptyLinesAroundModuleBody:
-  Enabled: false
-
-# double negation is perfectly idiomatic ruby
-Style/DoubleNegation:
-  Enabled: false
-# format strings are also perfectly idiomatic ruby
-Style/FormatString:
-  Enabled: false
-# single line if/unless blocks may read more naturally than modifiers
-Style/IfUnlessModifier:
-  Enabled: false
-
-# Dan is -1 on this one: https://github.com/chef/chef/pull/4526#issuecomment-179950045
-Layout/FirstHashElementIndentation:
-  Enabled: false
-
-# This is overly aggressive and autofix broke stuff, would need to exclude classes
-#  - https://github.com/chef/chef/pull/4541
-Style/TrivialAccessors:
-  Enabled: false
-
-# Not a lot of offenses and this seems to catch code that looks otherwise fine to my eye
-Style/MultilineBlockChain:
-  Enabled: false
-
-# We see nothing at all wrong with a trailing `rescue nil`
-Style/RescueModifier:
-  Enabled: false
-
-# Resulted in a bit of a bikeshed over names with unicode chars vs silently hidden unicode chars in comments
-Style/AsciiComments:
-  Enabled: false
-
-# Parens around ternaries often make them more readable and reduces cognitive load over operator precedence
-Style/TernaryParentheses:
-  Enabled: false
-
-# Underscores in numbers are super ugly for port numbers, freebsd versions, etc.
-# If we could mandate that this gets turned on only for numbers >= 1e9 which require three _'s then maybe.
-Style/NumericLiterals:
-  Enabled: false
-
-# I tend to find this more readable than long elsif chains
-Style/EmptyCaseCondition:
-  Enabled: false
-
-# This didn't quite do what we need
-Layout/MultilineArrayBraceLayout:
-  Enabled: false
-
-# This rule isn't smart enough to figure out accessors vs. things that set stuff on the system (e.g. set_member method in group providers)
-Naming/AccessorMethodName:
-  Enabled: false
-
-# Perl backrefs are just fine and the alternative is a lot of typing.
-Style/PerlBackrefs:
-  Enabled: false
-
-# This has an annoying auto-fix which fails to `require "english"` and then fails silently
-Style/SpecialGlobalVars:
-  Enabled: false
-
-# This cop is horrible and I hate the code it produces (even though I tend to like GuardClauses in lots of cases)
-Style/GuardClause:
-  Enabled: false
-
-Style/RedundantSortBy:
-  Enabled: true
-
-Style/Sample:
-  Enabled: true
-
-# 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/MethodParameterName:
-  Enabled: false
-Naming/MemoizedInstanceVariableName:
-  Enabled: false
-Naming/VariableNumber:
-  Enabled: false
-
-# We almost never use format strings but this cop triggers on any string with "%{whatever}" in it and is 99% false positives
-Style/FormatStringToken:
-  Enabled: false
-
-# This cop is annoying.  The rubocop default of course goes against the most popular community guidelines due to contrariness
-# and then the autocorrect mangles specs in chef/chef.  Burned more time than it was worth looking at this one so its off.
-Style/Alias:
-  Enabled: false
-
-# No autofix for this one, and I don't care to fix 138 instances of it in the chef/chef codebase.  Don't think its worth wasting
-# anyone else's time either.
-Style/CommentedKeyword:
-  Enabled: false
-
-# make sure we catch this Ruby 3.0 breaking change now
-Lint/DeprecatedOpenSSLConstant:
-  Enabled: true
-
-# alert on invalid ruby synatx
-Lint/Syntax:
-  Enabled: true
-
-# remove extra requires like 'thread'
-Lint/RedundantRequireStatement:
-  Enabled: true
-
-# simplify stripping strings
-Style/Strip:
-  Enabled: true
-
-# simplify getting min/max
-Style/RedundantSort:
-  Enabled: true
-
-# no need for .rb in requires
-Style/RedundantFileExtensionInRequire:
-  Enabled: true
-
-# more code you don't need
-Style/RedundantCondition:
-  Enabled: true
-
-# Use  __dir__ to simplify things
-Style/Dir:
-  Enabled: true
-
-# Use __FILE__ or __dir__ to simplify expand_paths
-Style/ExpandPathArguments:
-  Enabled: true
-
-# more code you don't need
-Lint/RedundantSafeNavigation:
-  Enabled: true
-
-# Eases readability
-Lint/RedundantStringCoercion:
-  Enabled: true
-
-# unnecessary complexity
-Lint/RedundantWithObject:
-  Enabled: true
-
-# unnecessary complexity
-Lint/RedundantWithIndex:
-  Enabled: true
-
-# avoid requiring twice
-Lint/DuplicateRequire:
-  Enabled: true
-
-# detect assignment typos
-Lint/AmbiguousAssignment:
-  Enabled: true
-
-Chef/Deprecations/Ruby27KeywordArgumentWarnings:
-  Description: Pass options to shell_out helpers without the brackets to avoid Ruby 2.7 deprecation warnings.
-  Enabled: true
-  VersionAdded: '1.3.0'
-
-Chef/Ruby/UnlessDefinedRequire:
-  Description: Workaround RubyGems' slow requires by only running require if the constant isn't already defined
-  Enabled: true
-  VersionAdded: '1.3.0'
-  Include:
-    - 'lib/**/*'
-
-Chef/Ruby/GemspecRequireRubygems:
-  Description: Rubygems does not need to be required in a Gemspec
-  Enabled: true
-  VersionAdded: '1.3.0'
-  Include:
-    - '**/*.gemspec'
-
-Chef/Ruby/GemspecLicense:
-  Description: 'All gemspec files should define their license.'
-  Enabled: true
-  VersionAdded: '1.7.0'
-  Include:
-    - '**/*.gemspec'
-  Exclude:
-    - '/**/spec/**/*'
-    - '/**/test/**/*'
-
-Chef/Ruby/RequireNetHttps:
-  Description: net/https is deprecated and just includes net/http and openssl. We should include those directly instead
-  Enabled: true
-  VersionAdded: '1.3.0'
-
-Chef/Ruby/LegacyPowershellOutMethods:
-  Description: Use powershell_exec!/powershell_exec instead of the slower legacy powershell_out!/powershell_out methods.
-  Enabled: true
-  VersionAdded: '1.6.0'
+# This is an internal RuboCop configuration for Chef Ruby projects (chef-client, ohai,
+# mixlib-shellout, mixlib-config, etc).
+#
+# It is not meant for consumption by cookbooks or for any general purpose uses.
+# **Chef Users and Customers Should Generally Not Use This Configuration and
+# Should Use cookstyle.yml**.  It is not intended for any audience outside of
+# Chef Core Ruby development.
+
+AllCops:
+  TargetRubyVersion: 3.0
+  NewCops: disable
+  DisabledByDefault: true
+  SuggestExtensions: false
+
+#
+# Bundler
+#
+
+Bundler/DuplicatedGem:
+  Enabled: false
+
+Bundler/InsecureProtocolSource:
+  Enabled: true
+  AllowHttpProtocol: false
+
+#
+# Lint
+#
+
+Lint/AmbiguousOperator:
+  Enabled: true
+Lint/AmbiguousBlockAssociation:
+  Enabled: true
+Lint/AmbiguousRegexpLiteral:
+  Enabled: true
+Lint/AssignmentInCondition:
+  Enabled: true
+Layout/BlockAlignment:
+  EnforcedStyleAlignWith: start_of_block
+  Enabled: true
+Lint/CircularArgumentReference:
+  Enabled: true
+Layout/ConditionPosition:
+  Enabled: true
+Lint/Debugger:
+  Enabled: true
+Layout/DefEndAlignment:
+  Enabled: true
+  AutoCorrect: true
+Lint/DeprecatedClassMethods:
+  Enabled: true
+Lint/DuplicateCaseCondition:
+  Enabled: true
+Lint/DuplicateMethods:
+  Enabled: true
+Lint/DuplicateHashKey:
+  Enabled: true
+Lint/EachWithObjectArgument:
+  Enabled: true
+Lint/ElseLayout:
+  Enabled: true
+Lint/EmptyEnsure:
+  Enabled: true
+Lint/EmptyExpression:
+  Enabled: true
+Lint/EmptyInterpolation:
+  Enabled: true
+Lint/EmptyWhen:
+  Enabled: true
+Layout/EndAlignment:
+  Enabled: true
+  AutoCorrect: true
+Lint/EnsureReturn:
+  Enabled: true
+Lint/FloatOutOfRange:
+  Enabled: true
+Lint/FormatParameterMismatch:
+  Enabled: true
+Lint/ImplicitStringConcatenation:
+  Enabled: true
+Lint/InheritException:
+  Enabled: true
+Lint/IneffectiveAccessModifier:
+  Enabled: true
+Lint/LiteralAsCondition:
+  Enabled: true
+Lint/LiteralInInterpolation:
+  Enabled: true
+Lint/Loop:
+  Enabled: true
+Lint/MultipleComparison:
+  Enabled: true
+Lint/NestedMethodDefinition:
+  Enabled: true
+Lint/NextWithoutAccumulator:
+  Enabled: true
+Lint/NonLocalExitFromIterator:
+  Enabled: true
+Lint/ParenthesesAsGroupedExpression:
+  Enabled: true
+Lint/PercentStringArray:
+  Enabled: true
+Lint/PercentSymbolArray:
+  Enabled: true
+Lint/RandOne:
+  Enabled: true
+Lint/RequireParentheses:
+  Enabled: true
+Lint/SafeNavigationChain:
+  Enabled: true
+Lint/ShadowedException:
+  Enabled: true
+Lint/ShadowingOuterLocalVariable:
+  Enabled: true
+Lint/UnderscorePrefixedVariableName:
+  Enabled: true
+Lint/UnifiedInteger:
+  Enabled: true
+Lint/RedundantSplatExpansion:
+  Enabled: true
+Lint/UnreachableCode:
+  Enabled: true
+Lint/UriEscapeUnescape:
+  Enabled: true
+Lint/UselessAccessModifier:
+  Enabled: true
+Lint/UselessAssignment:
+  Enabled: true
+Lint/BinaryOperatorWithIdenticalOperands:
+  Enabled: true
+Lint/UselessElseWithoutRescue:
+  Enabled: true
+Lint/UselessSetterCall:
+  Enabled: true
+Lint/Void:
+  Enabled: true
+Layout/CommentIndentation:
+  Enabled: true
+
+#
+# Disabled Lint
+#
+
+# These are disabled per https://github.com/chef/chef/pull/3038
+Lint/UnusedBlockArgument:
+  Enabled: false
+Lint/UnusedMethodArgument:
+  Enabled: false
+
+# We ignore Exceptions a lot
+Lint/SuppressedException:
+  Enabled: false
+# We also decorate Exceptions a lot
+Lint/RescueException:
+  Enabled: false
+
+# disabling this will make it easier to stage chefstyle rollouts
+Lint/RedundantCopDisableDirective:
+  Enabled: false
+
+#
+# Security
+#
+
+Security/Eval:
+  Enabled: true
+
+#
+# Metrics
+#
+
+# Metrics cops are all disabled, there would be too much refactoring
+# work to do and we have much higher priorities.
+Metrics/AbcSize:
+  Enabled: false
+Metrics/BlockNesting:
+  Enabled: false
+Metrics/BlockLength:
+  Enabled: false
+Metrics/ClassLength:
+  Enabled: false
+Metrics/CyclomaticComplexity:
+  Enabled: false
+Layout/LineLength:
+  Enabled: false
+Metrics/MethodLength:
+  Enabled: false
+Metrics/ModuleLength:
+  Enabled: false
+Metrics/ParameterLists:
+  Enabled: false
+Metrics/PerceivedComplexity:
+  Enabled: false
+
+#
+# Style/Naming/Layout
+#
+
+Layout/AccessModifierIndentation:
+  Enabled: true
+Layout/ArgumentAlignment:
+  Enabled: true
+  EnforcedStyle: with_fixed_indentation
+Layout/ParameterAlignment:
+  Enabled: true
+  EnforcedStyle: with_fixed_indentation
+# the "ignore_implict" is here for keyword args in method calls which are
+# "implicit" hashes, and those should not be treated like normal hashes
+Layout/HashAlignment:
+  Enabled: true
+  EnforcedLastArgumentHashStyle: ignore_implicit
+Style/AndOr:
+  Enabled: true
+Style/ArrayJoin:
+  Enabled: true
+Naming/AsciiIdentifiers:
+  Enabled: true
+Style/Attr:
+  Enabled: true
+Style/BeginBlock:
+  Enabled: true
+# there's debate over one-liner vs semantic blockstyle, chef projects should just pick one style
+# and stick with it.
+Style/BlockDelimiters:
+  Enabled: false
+Layout/BlockEndNewline:
+  Enabled: true
+Style/CharacterLiteral:
+  Enabled: true
+Style/ClassCheck:
+  Enabled: true
+Style/ClassMethods:
+  Enabled: true
+Layout/ClosingHeredocIndentation:
+  Enabled: true
+Layout/ClosingParenthesisIndentation:
+  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/EmptyLineAfterGuardClause:
+  Enabled: true
+Layout/EmptyLineBetweenDefs:
+  Enabled: true
+  AllowAdjacentOneLineDefs: true
+Layout/EmptyLines:
+  Enabled: true
+Layout/EmptyLinesAroundAccessModifier:
+  Enabled: true
+Layout/EmptyLinesAroundMethodBody:
+  Enabled: true
+Style/EmptyLiteral:
+  Enabled: true
+Style/EmptyMethod:
+  Enabled: true
+Style/EndBlock:
+  Enabled: true
+Layout/EndOfLine:
+  Enabled: true
+  EnforcedStyle: lf
+Style/EvenOdd:
+  Enabled: true
+Layout/ExtraSpacing:
+  Enabled: true
+Lint/FlipFlop:
+  Enabled: true
+Style/For:
+  Enabled: true
+Style/HashSyntax:
+  Enabled: true
+Naming/HeredocDelimiterCase:
+  Enabled: true
+Style/IfUnlessModifierOfIfUnless:
+  Enabled: true
+Style/IfWithSemicolon:
+  Enabled: true
+Layout/AssignmentIndentation:
+  Enabled: true
+Layout/FirstArgumentIndentation:
+  EnforcedStyle: consistent
+  Enabled: true
+Layout/HeredocIndentation:
+  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
+Style/MethodCallWithoutArgsParentheses:
+  Enabled: true
+Layout/MultilineBlockLayout:
+  Enabled: true
+Style/MultilineIfModifier:
+  Enabled: true
+Style/MultilineIfThen:
+  Enabled: true
+Style/MultilineMemoization:
+  Enabled: true
+Layout/MultilineMethodCallBraceLayout:
+  Enabled: true
+Layout/MultilineMethodCallIndentation:
+  Enabled: true
+  EnforcedStyle: indented
+Layout/MultilineMethodDefinitionBraceLayout:
+  Enabled: true
+Layout/MultilineOperationIndentation:
+  Enabled: true
+  EnforcedStyle: indented
+Style/MultilineTernaryOperator:
+  Enabled: true
+Style/MutableConstant:
+  Enabled: true
+Style/NegatedIf:
+  Enabled: true
+Style/NegatedWhile:
+  Enabled: true
+Style/NegatedUnless:
+  Enabled: true
+Style/NestedModifier:
+  Enabled: true
+Style/NestedTernaryOperator:
+  Enabled: true
+Style/NilComparison:
+  Enabled: true
+Style/NonNilCheck:
+  Enabled: true
+# `foo.zero?` is slower and substantially more annoying to read than `foo == 0`
+Style/NumericPredicate:
+  Enabled: true
+  EnforcedStyle: comparison
+Style/OneLineConditional:
+  Enabled: true
+Naming/BinaryOperatorParameterName:
+  Enabled: true
+Style/OptionalArguments:
+  Enabled: true
+Style/ParenthesesAroundCondition:
+  Enabled: true
+# - The chef source code predominantly uses {} for %,%i,%q,%Q,%r,%w,%W
+# - The old 1.9 pickaxe book predominantly uses {} for %,%q,%Q,%r,%w,%W,%x
+# - The chef source code does not use %x enough to determine a preference
+# - The old 1.9 pickaxe book does not use %i
+# - %s is not used in chef code or the pickaxe book, and is %s{} for consistency
+Style/PercentLiteralDelimiters:
+  Enabled: true
+  PreferredDelimiters:
+    '%':  '{}'
+    '%i': '{}'
+    '%I': '{}'
+    '%q': '{}'
+    '%Q': '{}'
+    '%r': '{}'
+    '%s': '{}'
+    '%w': '{}'
+    '%W': '{}'
+    '%x': '{}'
+Style/PercentQLiterals:
+  Enabled: true
+Style/PreferredHashMethods:
+  Enabled: true
+Style/RedundantBegin:
+  Enabled: true
+Style/RedundantException:
+  Enabled: true
+Style/RedundantFreeze:
+  Enabled: true
+Style/RedundantReturn:
+  Enabled: true
+Style/RedundantSelf:
+  Enabled: true
+Layout/RescueEnsureAlignment:
+  Enabled: true
+Style/RegexpLiteral:
+  Enabled: true
+# temporarily disabled, but we should do this
+Style/SafeNavigation:
+  Enabled: false
+Style/SelfAssignment:
+  Enabled: true
+Style/Semicolon:
+  AllowAsExpressionSeparator: true
+  Enabled: true
+# we only 'raise' and do not 'fail'
+Style/SignalException:
+  EnforcedStyle: only_raise
+  Enabled: true
+Layout/SpaceAfterColon:
+  Enabled: true
+Layout/SpaceAfterComma:
+  Enabled: true
+Layout/SpaceAfterMethodName:
+  Enabled: true
+Layout/SpaceAfterSemicolon:
+  Enabled: true
+Layout/SpaceAroundBlockParameters:
+  Enabled: true
+Layout/SpaceAroundEqualsInParameterDefault:
+  Enabled: true
+Layout/SpaceAroundOperators:
+  Enabled: true
+Layout/SpaceBeforeBlockBraces:
+  Enabled: true
+Layout/SpaceBeforeComment:
+  Enabled: true
+Layout/SpaceBeforeSemicolon:
+  Enabled: true
+Layout/SpaceAroundKeyword:
+  Enabled: true
+Layout/SpaceInLambdaLiteral:
+  Enabled: true
+Layout/SpaceInsideArrayPercentLiteral:
+  Enabled: true
+Layout/SpaceInsideBlockBraces:
+  Enabled: true
+Layout/SpaceInsideHashLiteralBraces:
+  Enabled: true
+Layout/SpaceInsideRangeLiteral:
+  Enabled: true
+Layout/SpaceInsideStringInterpolation:
+  Enabled: true
+Style/StabbyLambdaParentheses:
+  Enabled: true
+# core chef community prefers double quotes over single quotes
+Style/StringLiterals:
+  EnforcedStyle: double_quotes
+  Enabled: true
+Style/StringLiteralsInInterpolation:
+  EnforcedStyle: double_quotes
+  Enabled: true
+Style/StructInheritance:
+  Enabled: true
+Style/SymbolLiteral:
+  Enabled: true
+Style/StringMethods:
+  Enabled: true
+Style/SymbolArray:
+  Enabled: true
+Style/SymbolProc:
+  Enabled: true
+Layout/IndentationStyle:
+  Enabled: true
+Layout/TrailingEmptyLines:
+  Enabled: true
+Style/TrailingCommaInArguments:
+  Enabled: true
+# rubocop's default gets this completely backwards
+Style/TrailingCommaInArrayLiteral:
+  Enabled: true
+  EnforcedStyleForMultiline: comma
+Style/TrailingCommaInHashLiteral:
+  Enabled: true
+  EnforcedStyleForMultiline: comma
+Style/TrailingUnderscoreVariable:
+  Enabled: true
+Layout/TrailingWhitespace:
+  Enabled: true
+Style/RedundantCapitalW:
+  Enabled: true
+Style/RedundantInterpolation:
+  Enabled: false # buggy: https://github.com/rubocop-hq/rubocop/issues/6099
+#Style/RedundantPercentQ:  # would like to enable this one but its buggy as of 0.35.1
+#  Enabled: true
+Naming/VariableName:
+  Enabled: true
+Style/VariableInterpolation:
+  Enabled: true
+Style/WhenThen:
+  Enabled: true
+Style/WhileUntilDo:
+  Enabled: true
+Style/WhileUntilModifier:
+  Enabled: true
+Style/WordArray:
+  Enabled: true
+
+# we are ruby > 2.0 only so we can remove encoding comments for utf-8
+Style/Encoding:
+  Enabled: true
+
+#
+# Disabled Style
+#
+
+# reduces memory usage, but isn't a simple autocorrect so we need to do this one project at a time
+Style/FrozenStringLiteralComment:
+  Enabled: false
+
+# As of this commit we have 686 modules and classes without docs.
+# This is a cop that we /should/ have enabled, but tactically we can't really enable.
+Style/Documentation:
+  Enabled: false
+Style/DocumentationMethod:
+  Enabled: false
+
+# this makes whitespace formatting of DSL code impossible
+Layout/SpaceBeforeFirstArg:
+  Enabled: false
+
+# whitespace in expressions is useful to enhance readability
+Layout/SpaceInsideArrayLiteralBrackets:
+  Enabled: false
+Layout/SpaceInsideReferenceBrackets:
+  Enabled: false
+Layout/SpaceInsideParens:
+  Enabled: false
+Layout/SpaceInsidePercentLiteralDelimiters:
+  Enabled: false
+
+# more whitespace cops that we allow since they can enhance readability
+Layout/EmptyLinesAroundBlockBody:
+  Enabled: false
+Layout/EmptyLinesAroundClassBody:
+  Enabled: false
+Layout/EmptyLinesAroundModuleBody:
+  Enabled: false
+
+# double negation is perfectly idiomatic ruby
+Style/DoubleNegation:
+  Enabled: false
+# format strings are also perfectly idiomatic ruby
+Style/FormatString:
+  Enabled: false
+# single line if/unless blocks may read more naturally than modifiers
+Style/IfUnlessModifier:
+  Enabled: false
+
+# Dan is -1 on this one: https://github.com/chef/chef/pull/4526#issuecomment-179950045
+Layout/FirstHashElementIndentation:
+  Enabled: false
+
+# This is overly aggressive and autofix broke stuff, would need to exclude classes
+#  - https://github.com/chef/chef/pull/4541
+Style/TrivialAccessors:
+  Enabled: false
+
+# Not a lot of offenses and this seems to catch code that looks otherwise fine to my eye
+Style/MultilineBlockChain:
+  Enabled: false
+
+# We see nothing at all wrong with a trailing `rescue nil`
+Style/RescueModifier:
+  Enabled: false
+
+# Resulted in a bit of a bikeshed over names with unicode chars vs silently hidden unicode chars in comments
+Style/AsciiComments:
+  Enabled: false
+
+# Parens around ternaries often make them more readable and reduces cognitive load over operator precedence
+Style/TernaryParentheses:
+  Enabled: false
+
+# Underscores in numbers are super ugly for port numbers, freebsd versions, etc.
+# If we could mandate that this gets turned on only for numbers >= 1e9 which require three _'s then maybe.
+Style/NumericLiterals:
+  Enabled: false
+
+# I tend to find this more readable than long elsif chains
+Style/EmptyCaseCondition:
+  Enabled: false
+
+# This didn't quite do what we need
+Layout/MultilineArrayBraceLayout:
+  Enabled: false
+
+# This rule isn't smart enough to figure out accessors vs. things that set stuff on the system (e.g. set_member method in group providers)
+Naming/AccessorMethodName:
+  Enabled: false
+
+# Perl backrefs are just fine and the alternative is a lot of typing.
+Style/PerlBackrefs:
+  Enabled: false
+
+# This has an annoying auto-fix which fails to `require "english"` and then fails silently
+Style/SpecialGlobalVars:
+  Enabled: false
+
+# This cop is horrible and I hate the code it produces (even though I tend to like GuardClauses in lots of cases)
+Style/GuardClause:
+  Enabled: false
+
+Style/RedundantSortBy:
+  Enabled: true
+
+Style/Sample:
+  Enabled: true
+
+# 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/MethodParameterName:
+  Enabled: false
+Naming/MemoizedInstanceVariableName:
+  Enabled: false
+Naming/VariableNumber:
+  Enabled: false
+
+# We almost never use format strings but this cop triggers on any string with "%{whatever}" in it and is 99% false positives
+Style/FormatStringToken:
+  Enabled: false
+
+# This cop is annoying.  The rubocop default of course goes against the most popular community guidelines due to contrariness
+# and then the autocorrect mangles specs in chef/chef.  Burned more time than it was worth looking at this one so its off.
+Style/Alias:
+  Enabled: false
+
+# No autofix for this one, and I don't care to fix 138 instances of it in the chef/chef codebase.  Don't think its worth wasting
+# anyone else's time either.
+Style/CommentedKeyword:
+  Enabled: false
+
+# make sure we catch this Ruby 3.0 breaking change now
+Lint/DeprecatedOpenSSLConstant:
+  Enabled: true
+
+# alert on invalid ruby syntax
+Lint/Syntax:
+  Enabled: true
+
+# remove extra requires like 'thread'
+Lint/RedundantRequireStatement:
+  Enabled: true
+
+# simplify stripping strings
+Style/Strip:
+  Enabled: true
+
+# simplify getting min/max
+Style/RedundantSort:
+  Enabled: true
+
+# no need for .rb in requires
+Style/RedundantFileExtensionInRequire:
+  Enabled: true
+
+# more code you don't need
+Style/RedundantCondition:
+  Enabled: true
+
+# Use  __dir__ to simplify things
+Style/Dir:
+  Enabled: true
+
+# Use __FILE__ or __dir__ to simplify expand_paths
+Style/ExpandPathArguments:
+  Enabled: true
+
+# more code you don't need
+Lint/RedundantSafeNavigation:
+  Enabled: true
+
+# Eases readability
+Lint/RedundantStringCoercion:
+  Enabled: true
+
+# unnecessary complexity
+Lint/RedundantWithObject:
+  Enabled: true
+
+# unnecessary complexity
+Lint/RedundantWithIndex:
+  Enabled: true
+
+# avoid requiring twice
+Lint/DuplicateRequire:
+  Enabled: true
+
+# detect assignment typos
+Lint/AmbiguousAssignment:
+  Enabled: true
+
+# File.dirname(path, 2)
+# instead of
+# File.dirname(File.dirname(path))
+Style/NestedFileDirname:
+  Enabled: true
+
+# Use strings instead of regexes when possible
+Style/ExactRegexpMatch:
+  Enabled: true
+
+# Don't use redundant `any?`, `empty?`, `none?` or `one?` with `select`
+Style/RedundantFilterChain:
+  Enabled: true
+
+# Simplify creating arrays
+Style/RedundantArrayConstructor:
+  Enabled: true
+
+# Catch regex that tries to catch A-Z and a-z but fails
+Lint/MixedCaseRange:
+  Enabled: true
+
+# Use require_relative '/path'
+# not
+# require_relative './path
+Style/RedundantCurrentDirectoryInPath:
+  Enabled: true
+
+# Use strings instead of regex with common helper methods
+Style/RedundantRegexpArgument:
+  Enabled: true
+
+# Simplify loading of YAML from a file
+Style/YAMLFileRead:
+  Enabled: true
+
+# You can't have the same named group in a gemfile twice
+Bundler/DuplicatedGroup:
+  Enabled: true
+
+# Avoid creating an array and iterating it when .map can be used
+Style/MapIntoArray:
+  Enabled: true
+
+# Don't send method names when you don't need to
+Style/SendWithLiteralMethodName:
+  Enabled: true
+
+# add_runtime_dependencies is not preferred in gemspecs
+Gemspec/AddRuntimeDependency:
+  Enabled: true
+
+# Avoid numeric operators that do nothing to the value
+Lint/UselessNumericOperation:
+  Enabled: true
+
+# Don't define a set with duplicate elements since that defeats the point of a set
+Lint/DuplicateSetElement:
+  Enabled: true
+
+# Avoid calling defined twice when you don't need both
+Style/CombinableDefined:
+  Enabled: true
+
+# Catch malformed regexes
+Lint/UnescapedBracketInRegexp:
+  Enabled: true
+
+# Avoid defined calls that are always truthy
+Lint/UselessDefined:
+  Enabled: true
+
+# Call dig with multiple args instead of chaining it
+Style/DigChain:
+  Enabled: true
+
+# Catch deprecation hash creation in Ruby 3.4
+Lint/HashNewWithKeywordArgumentsAsDefault:
+  Enabled: true
+
+# Consistent magic comment formatting
+Style/MagicCommentFormat:
+  Enabled: true
+
+# Reduce duplicate comments
+Lint/DuplicateMagicComment:
+  Enabled: true
+
+# Don't escape strings when you don't need to
+Style/RedundantStringEscape:
+  Enabled: true
+
+# Don't double each something when you don't need to
+Style/RedundantEach:
+  Enabled: true
+
+# Use array.intersect? when on Ruby 3.1+
+Style/ArrayIntersect:
+  Enabled: true
+
+# Use .push to add to arrays which is much faster than .concat
+Style/ConcatArrayLiterals:
+  Enabled: true
+
+# Use .min and .max instead of rolling your own
+Style/MinMaxComparison:
+  Enabled: true
+
+# use .to_set instead of .map/.collect and then .to_set
+Style/MapToSet:
+  Enabled: true
+
+# Simplify how you check for empty files
+Style/FileEmpty:
+  Enabled: true
+
+# Simplify checking for empty directories
+Style/DirEmpty:
+  Enabled: true
+
+# Make it easier to read line continuations
+Layout/LineContinuationSpacing:
+  Enabled: true
+
+# Catch range statements that produce a Ruby warning
+Lint/RequireRangeParentheses:
+  Enabled: true
+
+# Don't use attributes in the gemspec that are deprecated and/or ignored now
+Gemspec/DeprecatedAttributeAssignment:
+  Enabled: true
+
+# Don't .map and then .compact when you can use .select or .reject instead
+Style/MapCompactWithConditionalBlock:
+  Enabled: true
+
+# Don't allow case statements with the same condition twice
+Lint/DuplicateMatchPattern:
+  Enabled: true
+
+# Use the performant .between? helper when possible
+Style/ComparableBetween:
+  Enabled: true
+
+# Simplify fetching data by using .dig
+Style/HashFetchChain:
+  Enabled: true
+
+# Don't use .format or .sprintf when not required
+Style/RedundantFormat:
+  Enabled: true
+
+# Don't convert types if we don't need to
+Lint/RedundantTypeConversion:
+  Enabled: true
+
+Chef/Deprecations/Ruby27KeywordArgumentWarnings:
+  Description: Pass options to shell_out helpers without the brackets to avoid Ruby 2.7 deprecation warnings.
+  Enabled: true
+  VersionAdded: '1.3.0'
+
+Chef/Ruby/UnlessDefinedRequire:
+  Description: Workaround RubyGems' slow requires by only running require if the constant isn't already defined
+  Enabled: true
+  VersionAdded: '1.3.0'
+  Include:
+    - 'lib/**/*'
+
+Chef/Ruby/GemspecRequireRubygems:
+  Description: Rubygems does not need to be required in a Gemspec
+  Enabled: true
+  VersionAdded: '1.3.0'
+  Include:
+    - '**/*.gemspec'
+
+Chef/Ruby/GemspecLicense:
+  Description: 'All gemspec files should define their license.'
+  Enabled: true
+  VersionAdded: '1.7.0'
+  Include:
+    - '**/*.gemspec'
+  Exclude:
+    - '/**/spec/**/*'
+    - '/**/test/**/*'
+
+Chef/Ruby/RequireNetHttps:
+  Description: net/https is deprecated and just includes net/http and openssl. We should include those directly instead
+  Enabled: true
+  VersionAdded: '1.3.0'
+
+Chef/Ruby/LegacyPowershellOutMethods:
+  Description: Use powershell_exec!/powershell_exec instead of the slower legacy powershell_out!/powershell_out methods.
+  Enabled: true
+  VersionAdded: '1.6.0'