simplycop 1.14.6 → 1.14.8

data/.simplycop.yml

@@ -1,7 +1,15 @@
 inherit_from:
   - .custom_simplycop.yml
-  - .simplycop_security.yml
   - .simplycop_metaprogramming.yml
+  - .simplycop_bundler.yml
+  - .simplycop_gemspec.yml
+  - .simplycop_layout.yml
+  - .simplycop_lint.yml
+  - .simplycop_metrics.yml
+  - .simplycop_migration.yml
+  - .simplycop_naming.yml
+  - .simplycop_security.yml
+  - .simplycop_style.yml
 
 AllCops:
   Exclude:
@@ -20,1756 +28,3 @@ AllCops:
   MaxFilesInCache: 5000
   ExtraDetails: true
 # Adapted from: https://github.com/simplybusiness/how-we-roll/blob/master/development/RUBYSTYLEGUIDE.markdown
-
-Bundler/DuplicatedGem:
-  Enabled: true
-
-Bundler/GemComment:
-  Enabled: false
-
-Bundler/GemFilename:
-  Enabled: true
-
-# To discuss: Suggest true but 103 fails on Chopin, 10 on Rater
-Bundler/GemVersion:
-  Enabled: false
-
-Bundler/InsecureProtocolSource:
-  Enabled: true
-
-Bundler/OrderedGems:
-  Enabled: true
-
-# To discuss: Suggest true but 3 fails on Chopin, 21 on Rater
-Gemspec/DependencyVersion:
-  Enabled: false
-
-# TODO: 1 autocorrectable in Rater and it would pass
-Gemspec/DeprecatedAttributeAssignment:
-  Enabled: true
-
-# 2 fails on Chopin, 10 on Rater, so override if necessary
-Gemspec/DevelopmentDependencies:
-  Enabled: false
-
-Gemspec/DuplicatedAssignment:
-  Enabled: true
-
-Gemspec/OrderedDependencies:
-  Enabled: true
-
-# TODO: Suggest true but 4 fails on Chopin, 3 on Rater, overridden to false on Chopin
-Gemspec/RequireMFA:
-  Enabled: false
-
-# Will need Chopin override
-Gemspec/RequiredRubyVersion:
-  Enabled: true
-
-Gemspec/RubyVersionGlobalsUsage:
-  Enabled: true
-
-Layout/AccessModifierIndentation:
-  Enabled: true
-
-Layout/ArgumentAlignment:
-  Enabled: true
-
-Layout/ArrayAlignment:
-  Enabled: true
-
-Layout/AssignmentIndentation:
-  Enabled: true
-
-Layout/BeginEndAlignment:
-  Enabled: true
-
-Layout/BlockAlignment:
-  Enabled: true
-
-Layout/BlockEndNewline:
-  Enabled: true
-
-Layout/CaseIndentation:
-  Enabled: true
-
-# Discussion needed about preferred structure. Dozens of fails on each repo if we stick with default, all but 4 on Chopin are autocorrectable
-# Worth agreeing a style?
-Layout/ClassStructure:
-  Enabled: false
-
-Layout/ClosingHeredocIndentation:
-  Enabled: true
-
-Layout/ClosingParenthesisIndentation:
-  Enabled: true
-
-Layout/CommentIndentation:
-  Enabled: true
-
-Layout/ConditionPosition:
-  Enabled: true
-
-Layout/DefEndAlignment:
-  Enabled: true
-
-Layout/DotPosition:
-  Enabled: true
-
-Layout/ElseAlignment:
-  Enabled: true
-
-Layout/EmptyComment:
-  Enabled: true
-
-Layout/EmptyLineAfterGuardClause:
-  Enabled: true
-
-# Needs Chopin override 
-Layout/EmptyLineAfterMagicComment:
-  Enabled: true
-
-Layout/EmptyLineAfterMultilineCondition:
-  Enabled: true
-
-Layout/EmptyLineBetweenDefs:
-  Enabled: true
-
-Layout/EmptyLines:
-  Enabled: true
-
-Layout/EmptyLinesAroundAccessModifier:
-  Enabled: true
-
-Layout/EmptyLinesAroundArguments:
-  Enabled: true
-
-Layout/EmptyLinesAroundAttributeAccessor:
-  Enabled: true
-
-Layout/EmptyLinesAroundBeginBody:
-  Enabled: true
-
-Layout/EmptyLinesAroundBlockBody:
-  Enabled: true
-
-Layout/EmptyLinesAroundClassBody:
-  Enabled: true  
-
-Layout/EmptyLinesAroundExceptionHandlingKeywords:
-  Enabled: true
-
-Layout/EmptyLinesAroundMethodBody:
-  Enabled: true  
-
-Layout/EmptyLinesAroundModuleBody:
-  Enabled: true
-
-Layout/EndAlignment:
-  Enabled: true
-
-Layout/EndOfLine:
-  Enabled: true
-
-Layout/ExtraSpacing:
-  Enabled: true
-
-Layout/FirstArgumentIndentation:
-  Enabled: true
-
-Layout/FirstArrayElementIndentation:
-  Enabled: true
-
-Layout/FirstArrayElementLineBreak:
-  Enabled: true
-
-Layout/FirstHashElementIndentation:
-  Enabled: true
-
-Layout/FirstHashElementLineBreak:
-  Enabled: true
-
-Layout/FirstMethodArgumentLineBreak:
-  Enabled: true
-
-# Needs Chopin override
-Layout/FirstMethodParameterLineBreak:
-  Enabled: true
-
-Layout/FirstParameterIndentation:
-  Enabled: true
-
-Layout/HashAlignment:
-  Enabled: true
-
-Layout/HeredocArgumentClosingParenthesis:
-  Enabled: true
-
-Layout/HeredocIndentation:
-  Enabled: true
-
-Layout/IndentationConsistency:
-  Enabled: true
-
-Layout/IndentationStyle:
-  Enabled: true
-
-Layout/IndentationWidth:
-  Enabled: true
-
-Layout/InitialIndentation:
-  Enabled: true
-
-Layout/LeadingCommentSpace:
-  Enabled: true
-
-Layout/LeadingEmptyLines:
-  Enabled: true
-
-# Overridden to false in Chopin
-Layout/LineContinuationLeadingSpace:
-  Enabled: true
-
-# Overridden to false in Chopin
-Layout/LineContinuationSpacing:
-  Enabled: true
-
-# Overridden to false in Chopin
-Layout/LineEndStringConcatenationIndentation:
-  Enabled: true
-
-# To discuss: configure line length? At default 120: 6805 fails on Chopin, 3 on Rater, 179 on CoCo
-# Typically overridden/"to-do"-ed
-Layout/LineLength:
-  Enabled: true
-
-Layout/MultilineArrayBraceLayout:
-  Enabled: true
-  
-Layout/MultilineArrayLineBreaks:
-  Enabled: true
-
-# To discuss: 100s of fails on each repo but autocorrectable  
-Layout/MultilineAssignmentLayout:
-  Enabled: false
-  
-Layout/MultilineBlockLayout:
-  Enabled: true
-  
-Layout/MultilineHashBraceLayout:
-  Enabled: true
-
-# To discuss: 100s of fails on each repo but autocorrectable  gp 
-Layout/MultilineHashKeyLineBreaks:
-  Enabled: false
-
-# To discuss: 1000s of fails on each repo but autocorrectable  
-Layout/MultilineMethodArgumentLineBreaks:
-  Enabled: false
-  
-Layout/MultilineMethodCallBraceLayout:
-  Enabled: true
-  
-Layout/MultilineMethodCallIndentation:
-  Enabled: true
-  
-Layout/MultilineMethodDefinitionBraceLayout:
-  Enabled: true
-
-# To discuss: 100s of fails on each repo but autocorrectable
-Layout/MultilineMethodParameterLineBreaks:
-  Enabled: false
-  
-Layout/MultilineOperationIndentation:
-  Enabled: true
-
-Layout/ParameterAlignment:
-  Enabled: true
-
-# For discussion: 1000s of fails, all autocorrectable
-Layout/RedundantLineBreak:
-  Enabled: false
-
-Layout/RescueEnsureAlignment:
-  Enabled: true
-
-# For discussion: 1000s of fails, all autocorrectable
-Layout/SingleLineBlockChain:
-  Enabled: false
-
-Layout/SpaceAfterColon:
-  Enabled: true
-
-Layout/SpaceAfterComma:
-  Enabled: true
-
-Layout/SpaceAfterMethodName:
-  Enabled: true
-
-Layout/SpaceAfterNot:
-  Enabled: true
-
-Layout/SpaceAfterSemicolon:
-  Enabled: true
-
-Layout/SpaceAroundBlockParameters:
-  Enabled: true
-
-Layout/SpaceAroundEqualsInParameterDefault:
-  Enabled: true
-
-Layout/SpaceAroundKeyword:
-  Enabled: true
-
-Layout/SpaceAroundMethodCallOperator:
-  Enabled: true
-
-Layout/SpaceAroundOperators:
-  Enabled: true
-
-Layout/SpaceBeforeBlockBraces:
-  Enabled: true
-
-Layout/SpaceBeforeBrackets:
-  Enabled: true
-
-Layout/SpaceBeforeComma:
-  Enabled: true
-
-Layout/SpaceBeforeComment:
-  Enabled: true
-
-Layout/SpaceBeforeFirstArg:
-  Enabled: true
-
-Layout/SpaceBeforeSemicolon:
-  Enabled: true
-
-Layout/SpaceInLambdaLiteral:
-  Enabled: true
-
-Layout/SpaceInsideArrayLiteralBrackets:
-  Enabled: true
-
-Layout/SpaceInsideArrayPercentLiteral:
-  Enabled: true
-
-Layout/SpaceInsideBlockBraces:
-  Enabled: true
-
-Layout/SpaceInsideHashLiteralBraces:
-  Enabled: true
-
-Layout/SpaceInsideParens:
-  Enabled: true
-
-Layout/SpaceInsidePercentLiteralDelimiters:
-  Enabled: true
-
-Layout/SpaceInsideRangeLiteral:
-  Enabled: true
-
-Layout/SpaceInsideReferenceBrackets:
-  Enabled: true
-
-Layout/SpaceInsideStringInterpolation:
-  Enabled: true
-
-Layout/TrailingEmptyLines:
-  Enabled: true
-
-Layout/TrailingWhitespace:
-  Enabled: true
-
-Lint/AmbiguousAssignment:
-  Enabled: true
-
-Lint/AmbiguousBlockAssociation:
-  Exclude:
-    - spec/**/*
-
-Lint/AmbiguousOperator:
-  Enabled: true
-
-# Overridden to false in Chopin
-Lint/AmbiguousOperatorPrecedence:
-  Enabled: true
-
-# Overridden to false in Chopin
-Lint/AmbiguousRange:
-  Enabled: true
-
-# TODO: One fail on rater spec then would pass
-Lint/AmbiguousRegexpLiteral:
-  Enabled: true
-
-Lint/AssignmentInCondition:
-  Enabled: true
-  
-Lint/BigDecimalNew:
-  Enabled: true
-
-Lint/BinaryOperatorWithIdenticalOperands:
-  Enabled: true
-
-Lint/BooleanSymbol:
-  Enabled: true
-
-Lint/CircularArgumentReference:
-  Enabled: true
-
-# TODO: 6 fails in Rater, otherwise clear
-Lint/ConstantDefinitionInBlock:
-  Enabled: true
-
-Lint/ConstantOverwrittenInRescue: # new in 1.31
-  Enabled: true
-
-# Set to false as 10000s of fails on Rater and Chopin if enabled
-# Rubocop docs suggest enabling only for constants that may cause concern
-# so enable and set these if required
-Lint/ConstantResolution:
-  Enabled: false 
-
-Lint/Debugger:
-  Enabled: true 
-
-Lint/DeprecatedClassMethods:
-  Enabled: true 
-
-Lint/DeprecatedConstants:
-  Enabled: true
-
-Lint/DeprecatedOpenSSLConstant:
-  Enabled: true
-
-Lint/DisjunctiveAssignmentInConstructor:
-  Enabled: true
-
-Lint/DuplicateBranch:
-  Enabled: true
-
-Lint/DuplicateCaseCondition:
-  Enabled: true
-
-Lint/DuplicateElsifCondition:
-  Enabled: true
-
-Lint/DuplicateHashKey:
-  Enabled: true
-
-Lint/DuplicateMagicComment:
-  Enabled: true
-
-Lint/DuplicateMatchPattern:
-  Enabled: true
-
-Lint/DuplicateMethods:
-  Enabled: true
-
-Lint/DuplicateRegexpCharacterClassElement:
-  Enabled: true
-
-Lint/DuplicateRequire:
-  Enabled: true
-
-Lint/DuplicateRescueException:
-  Enabled: true
-
-Lint/EachWithObjectArgument:
-  Enabled: true
-
-Lint/ElseLayout:
-  Enabled: true
-
-Lint/EmptyBlock:
-  Enabled: true
-
-Lint/EmptyClass:
-  Enabled: true
-
-Lint/EmptyConditionalBody:
-  Enabled: true 
-
-Lint/EmptyEnsure:
-  Enabled: true 
-
-Lint/EmptyExpression:
-  Enabled: true 
-
-Lint/EmptyFile:
-  Enabled: true 
-
-Lint/EmptyInPattern: # new in 1.16
-  Enabled: true 
-
-Lint/EmptyInterpolation:
-  Enabled: true 
-
-Lint/EmptyWhen:
-  Enabled: true
-
-Lint/ErbNewArguments:
-  Enabled: true 
-
-Lint/FlipFlop:
-  Enabled: true 
-
-Lint/FloatComparison:
-  Enabled: true 
-
-Lint/FloatOutOfRange:
-  Enabled: true 
-
-Lint/FormatParameterMismatch:
-  Enabled: true 
-
-Lint/HashCompareByIdentity:
-  Enabled: true 
-
-Lint/HeredocMethodCallPosition:
-  Enabled: true
-
-Lint/IdentityComparison:
-  Enabled: true 
-
-Lint/ImplicitStringConcatenation:
-  Enabled: true 
-
-Lint/IncompatibleIoSelectWithFiberScheduler: # new in 1.21
-  Enabled: true
-
-# TODO: two fails on Rater, one on CoCo, otherwise fine
-Lint/IneffectiveAccessModifier:
-  Enabled: true 
-
-Lint/InheritException:
-  Enabled: true 
-
-Lint/InterpolationCheck:
-  Enabled: true
-
-Lint/LambdaWithoutLiteralBlock:
-  Enabled: true
-
-Lint/LiteralAsCondition:
-  Enabled: true
-
-Lint/LiteralInInterpolation:
-  Enabled: true
-
-Lint/Loop:
-  Enabled: true
-
-Lint/MissingCopEnableDirective:
-  Enabled: true
-
-# TODO: Handful of errors on Rater and CoCo
-Lint/MissingSuper:
-  Enabled: true
-
-Lint/MixedCaseRange:
-  Enabled: true
-
-Lint/MixedRegexpCaptureTypes:
-  Enabled: true
-
-Lint/MultipleComparison:
-  Enabled: true
-
-Lint/NestedMethodDefinition:
-  Enabled: true
-
-Lint/NestedPercentLiteral:
-  Enabled: true
-
-Lint/NextWithoutAccumulator:
-  Enabled: true
-
-Lint/NoReturnInBeginEndBlocks:
-  Enabled: true
-
-# Overridden to false in Chopin
-Lint/NonAtomicFileOperation:
-  Enabled: true
-
-Lint/NonDeterministicRequireOrder:
-  Enabled: true 
-
-Lint/NonLocalExitFromIterator:
-  Enabled: true 
-
-Lint/NumberConversion:
-  Enabled: false 
-
-Lint/NumberedParameterAssignment:
-  Enabled: true
-
-Lint/OrAssignmentToConstant:
-  Enabled: true
-
-Lint/OrderedMagicComments:
-  Enabled: true 
-
-Lint/OutOfRangeRegexpRef:
-  Enabled: true 
-
-Lint/ParenthesesAsGroupedExpression:
-  Enabled: true 
-
-Lint/PercentStringArray:
-  Enabled: true 
-
-Lint/PercentSymbolArray:
-  Enabled: true 
-
-Lint/RaiseException:
-  Enabled: true 
-
-Lint/RandOne:
-  Enabled: true 
-
-Lint/RedundantCopDisableDirective:
-  Enabled: true 
-
-Lint/RedundantCopEnableDirective:
-  Enabled: true 
-
-Lint/RedundantDirGlobSort:
-  Enabled: true
-
-Lint/RedundantRegexpQuantifiers:
-  Enabled: true
-
-Lint/RedundantRequireStatement:
-  Enabled: true
-
-Lint/RedundantSafeNavigation:
-  Enabled: true
-
-Lint/RedundantSplatExpansion:
-  Enabled: true
-
-Lint/RedundantStringCoercion:
-  Enabled: true
-
-Lint/RedundantWithIndex:
-  Enabled: true
-
-Lint/RedundantWithObject:
-  Enabled: true
-
-Lint/RefinementImportMethods:
-  Enabled: true
-
-Lint/RegexpAsCondition:
-  Enabled: true
-
-Lint/RequireParentheses:
-  Enabled: true
-
-Lint/RequireRangeParentheses:
-  Enabled: true
-
-# Overridden to false in Chopin
-Lint/RequireRelativeSelfPath:
-  Enabled: true
-
-Lint/RescueException:
-  Enabled: true
-
-Lint/RescueType:
-  Enabled: true
-
-Lint/ReturnInVoidContext:
-  Enabled: true
-
-Lint/SafeNavigationChain:
-  Enabled: true
-
-Lint/SafeNavigationConsistency:
-  Enabled: true
-
-Lint/SafeNavigationWithEmpty:
-  Enabled: true
-
-Lint/ScriptPermission:
-  Enabled: true
-
-Lint/SelfAssignment:
-  Enabled: true
-
-Lint/SendWithMixinArgument:
-  Enabled: true
-
-Lint/ShadowedArgument:
-  Enabled: true
-
-Lint/ShadowedException:
-  Enabled: true
-
-# Needs Chopin override
-Lint/ShadowingOuterLocalVariable:
-  Enabled: true
-
-Lint/StructNewOverride:
-  Enabled: true
-
-Lint/SuppressedException:
-  Enabled: true
-
-Lint/SymbolConversion:
-  Enabled: true
-
-Lint/Syntax:
-  Enabled: true
-
-Lint/ToEnumArguments:
-  Enabled: true
-
-Lint/ToJSON:
-  Enabled: true
-
-Lint/TopLevelReturnWithArgument:
-  Enabled: true
-
-Lint/TrailingCommaInAttributeDeclaration:
-  Enabled: true
-
-Lint/TripleQuotes:
-  Enabled: true
-
-Lint/UnderscorePrefixedVariableName:
-  Enabled: true
-
-Lint/UnexpectedBlockArity:
-  Enabled: true
-
-Lint/UnifiedInteger:
-  Enabled: true
-
-Lint/UnmodifiedReduceAccumulator:
-  Enabled: true
-
-Lint/UnreachableCode:
-  Enabled: true
-
-Lint/UnreachableLoop:
-  Enabled: true
-
-Lint/UnusedBlockArgument:
-  Enabled: true
-
-Lint/UnusedMethodArgument:
-  Enabled: true
-
-Lint/UriEscapeUnescape:
-  Enabled: true
-
-Lint/UriRegexp:
-  Enabled: true
-
-Lint/UselessAccessModifier:
-  Enabled: true
-
-Lint/UselessAssignment:
-  Enabled: true
-
-Lint/UselessElseWithoutRescue:
-  Enabled: true
-
-Lint/UselessMethodDefinition:
-  Enabled: true
-
-# TODO: Single fail on Chopin
-Lint/UselessRescue:
-  Enabled: true
-
-Lint/UselessRuby2Keywords:
-  Enabled: true
-
-Lint/UselessSetterCall:
-  Enabled: true
-
-Lint/UselessTimes:
-  Enabled: true
-
-Lint/Void:
-  Enabled: true
-
-# METRICS COPS
-
-# Metrics cops are enabled by default with the expectation that many repos will 
-# have to set their own limits or disable. #technical-excellence-community discussion
-# suggested some liked to work from them, and that overriding should be a conscious decision
-
-Metrics/AbcSize:
-  Enabled: true
-
-Metrics/BlockLength:
-  Enabled: true
-  Exclude:
-    - spec/**/*.rb
-
-Metrics/BlockNesting:
-  Enabled: true
-
-Metrics/ClassLength:
-  Enabled: true
-
-Metrics/CollectionLiteralLength:
-  Enabled: true 
-
-Metrics/CyclomaticComplexity:
-  Enabled: true
-
-Metrics/MethodLength:
-  Enabled: true
-  Exclude:
-    - spec/**/*.rb
-
-Metrics/ModuleLength:
-  Enabled: true
-
-Metrics/ParameterLists:
-  CountKeywordArgs: false
-  Enabled: true
-
-Metrics/PerceivedComplexity:
-  Enabled: true
-
-Migration/DepartmentName:
-  Enabled: true
-
-# Needs override or fix in Chopin
-Naming/AccessorMethodName:
-  Enabled: true
-
-Naming/AsciiIdentifiers:
-  Enabled: true
-
-Naming/BinaryOperatorParameterName:
-  Enabled: true
-
-Naming/BlockForwarding:
-  Enabled: true
-
-Naming/BlockParameterName:
-  Enabled: true
-
-Naming/ClassAndModuleCamelCase:
-  Enabled: true
-
-Naming/ConstantName:
-  Enabled: true
-
-Naming/FileName:
-  Enabled: true
-
-Naming/HeredocDelimiterCase:
-  Enabled: true
-
-Naming/HeredocDelimiterNaming:
-  Enabled: true
-
-Naming/InclusiveLanguage:
-  Enabled: true
-
-# Need override - 86 fails on Chopin
-Naming/MemoizedInstanceVariableName:
-  Enabled: true
-
-Naming/MethodName:
-  Enabled: true
-
-Naming/MethodParameterName:
-  Enabled: true
-  AllowedNames:
-    - as
-    - at
-    - ex
-    - id
-    - ip
-    - n
-    - to
-
-Naming/PredicateName:
-  Enabled: true
-  ForbiddenPrefixes: []
-
-# Need Chopin override
-Naming/RescuedExceptionsVariableName:
-  Enabled: true
-
-Naming/VariableName:
-  Enabled: true
-
-# snake_case: 311 fails on Chopin, normalcase: 1118 fails on Chopin
-# snake_case default on Rater so make default in simplycop
-Naming/VariableNumber:
-  Enabled: true
-  EnforcedStyle: snake_case
-  AllowedIdentifiers:
-    - capture3
-    - e164
-    - iso8601 
-    - oauth2
-    - rfc1123_date
-    - rfc822
-    - rfc2822
-    - rfc3339
-    - s3
-    - x86_64
-  AllowedPatterns:
-    - v\d+
-    
-# Overridden to false in Chopin
-Security/CompoundHash:
-  Enabled: true
-
-Security/Eval:
-  Enabled: true
-  Exclude:
-    - 'spec/**/*.rb'
-
-# Overridden to false on Chopin
-Security/IoMethods:
-  Enabled: true
-
-# Needs override in chopin spec
-Security/JSONLoad:
-  Enabled: true
-
-Security/MarshalLoad:
-  Enabled: true
-
-Security/Open:
-  Enabled: true
-
-Security/YAMLLoad:
-  Enabled: true
-
-Style/AccessModifierDeclarations:
-  Enabled: false
-
-Style/AccessorGrouping:
-  Enabled: true
-
-Style/Alias:
-  Enabled: false
-
-Style/AndOr:
-  Enabled: true
-
-Style/ArgumentsForwarding:
-  Enabled: true
-
-Style/ArrayCoercion:
-  Enabled: true
-
-# Overridden to false in Chopin
-Style/ArrayIntersect:
-  Enabled: true
-
-Style/ArrayJoin:
-  Enabled: true
-
-Style/AsciiComments:
-  Enabled: true
-  AllowedChars: 
-    - £
-    - ©
-
-Style/Attr:
-  Enabled: true
-
-# Requires fixes in Chopin and Rater
-Style/AutoResourceCleanup:
-  Enabled: true 
-
-Style/BarePercentLiterals:
-  Enabled: true
-
-Style/BeginBlock:
-  Enabled: true
-
-Style/BisectedAttrAccessor:
-  Enabled: true
-
-Style/BlockComments:
-  Enabled: true
-
-# Enabled by default and used in Rater so made explicit here
-# However, overridden by Chopin to do file and proposal to use semantic style:
-# See #technical-excellence-community discussion 28/02/23
-Style/BlockDelimiters:
-  Enabled: true
-
-Style/CaseEquality:
-  Enabled: true
-
-Style/CaseLikeIf:
-  Enabled: true
-
-Style/CharacterLiteral:
-  Enabled: true
-
-Style/ClassAndModuleChildren:
-  Enabled: true
-  Exclude:
-    - spec/**/*.rb
-
-Style/ClassCheck:
-  Enabled: true
-
-Style/ClassEqualityComparison:
-  Enabled: true
-
-Style/ClassMethods:
-  Enabled: true
-
-# Requires discussion - multiple fails on Chopin/Rater/CoCo when set true with default
-Style/ClassMethodsDefinitions:
-  Enabled: false
-
-Style/ClassVars:
-  Enabled: true
-
-Style/CollectionCompact:
-  Enabled: true
-
-# Requires discussion - multiple fails on Chopin/Rater/CoCo when set true with default
-Style/CollectionMethods:
-  Enabled: false
-
-Style/ColonMethodCall:
-  Enabled: true
-  Severity: warning
-
-Style/ColonMethodDefinition:
-  Enabled: true
-
-Style/CombinableLoops:
-  Enabled: true
-
-Style/CommandLiteral:
-  Enabled: true
-
-Style/CommentAnnotation:
-  Enabled: true
-
-Style/CommentedKeyword:
-  Enabled: true
-
-# TODO: Single fail on Rater
-Style/ComparableClamp:
-  Enabled: true
-
-Style/ConcatArrayLiterals:
-  Enabled: true
-
-Style/ConditionalAssignment:
-  Enabled: true
-
-# Requires discussion - 1000+ offences in Chopin alone, unless options set
-Style/ConstantVisibility:
-  Enabled: false
-
-# For discussion
-Style/Copyright:
-  Enabled: false
-
-Style/DataInheritance:
-  Enabled: true
-
-# Requires discussion - multiple fails on Chopin/Rater/CoCo if true
-Style/DateTime:
-  Enabled: false
-
-Style/DefWithParentheses:
-  Enabled: true
-
-Style/Dir:
-  Enabled: true
-
-Style/DirEmpty:
-  Enabled: true
-
-# For discussion - - multiple fails on Chopin/Rater/CoCo
-Style/DisableCopsWithinSourceCodeDirective:
-  Enabled: false
-
-Style/DocumentDynamicEvalDefinition:
-  Enabled: false
-
-Style/Documentation:
-  Enabled: false
-
-# Requires discussion - suggest false as 7000+ offences in Chopin alone
-Style/DocumentationMethod:
-  Enabled: false
-
-Style/DoubleCopDisableDirective:
-  Enabled: true
-
-Style/DoubleNegation:
-  Enabled: true
-
-# Need override in Rater
-Style/EachForSimpleLoop:
-  Enabled: true
-
-Style/EachWithObject:
-  Enabled: true
-
-Style/EmptyBlockParameter:
-  Enabled: true
-
-Style/EmptyCaseCondition:
-  Enabled: true
-
-Style/EmptyElse:
-  Enabled: true
-
-Style/EmptyHeredoc:
-  Enabled: true
-
-Style/EmptyLambdaParameter:
-  Enabled: true
-
-Style/EmptyLiteral:
-  Enabled: true
-
-Style/EmptyMethod:
-  Enabled: true
-
-Style/Encoding:
-  Enabled: true
-
-Style/EndBlock:
-  Enabled: true
-
-Style/EndlessMethod:
-  Enabled: true
-
-Style/EnvHome:
-  Enabled: true
-
-Style/EvalWithLocation:
-  Enabled: true
-
-Style/EvenOdd:
-  Enabled: true
-
-Style/ExactRegexpMatch:
-  Enabled: true
-
-Style/ExpandPathArguments:
-  Enabled: true
-
-Style/ExplicitBlockArgument:
-  Enabled: true
-
-Style/ExponentialNotation:
-  Enabled: false
-
-# Overridden to false in Chopin
-Style/FetchEnvVar:
-  Enabled: true
-
-Style/FileEmpty:
-  Enabled: true
-
-Style/FileRead: # new in 1.24
-  Enabled: true
-
-# Overridden to false in Chopin
-Style/FileWrite:
-  Enabled: true
-
-Style/FloatDivision:
-  Enabled: true
-
-Style/For:
-  Enabled: true
-
-Style/FormatString:
-  Enabled: false
-
-Style/FormatStringToken:
-  Enabled: false
-
-Style/FrozenStringLiteralComment:
-  Enabled: false
-
-Style/GlobalStdStream:
-  Enabled: true
-
-Style/GlobalVars:
-  Enabled: true
-
-Style/GuardClause:
-  Enabled: false
-
-Style/HashAsLastArrayItem:
-  Enabled: true
-
-Style/HashConversion:
-  Enabled: true
-
-Style/HashEachMethods:
-  Enabled: false
-
-Style/HashExcept:
-  Enabled: true
-
-Style/HashLikeCase:
-  Enabled: false
-
-Style/HashSyntax:
-  Enabled: true
-  EnforcedStyle: no_mixed_keys
-  EnforcedShorthandSyntax: consistent
-
-Style/HashTransformKeys:
-  Enabled: true
-
-Style/HashTransformValues:
-  Enabled: true
-
-Style/IdenticalConditionalBranches:
-  Enabled: true
-
-Style/IfInsideElse:
-  Enabled: true
-
-Style/IfUnlessModifier:
-  Enabled: false
-
-Style/IfUnlessModifierOfIfUnless:
-  Enabled: true
-
-Style/IfWithBooleanLiteralBranches:
-  Enabled: true
-
-Style/IfWithSemicolon:
-  Enabled: true
-
-# For dicussion - 100s of fails on Chopin, CoCo, Rater
-Style/ImplicitRuntimeError:
-  Enabled: false
-
-Style/InfiniteLoop:
-  Enabled: true
-
-Style/InlineComment:
-  Enabled: false
-
-Style/InverseMethods:
-  Enabled: true
-
-Style/InvertibleUnlessCondition:
-  Enabled: true
-
-Style/InPatternThen: # new in 1.16
-  Enabled: true
-
-# For discussion - disabled by default but could be enabled 
-# and mostly solved by excluding spec files
-Style/IpAddresses:
-  Enabled: false
-#   Enabled: true
-#   Exclude: [*spec.rb]
-
-Style/KeywordParametersOrder:
-  Enabled: true
-
-Style/Lambda:
-  Enabled: true
-
-Style/LambdaCall:
-  Enabled: true
-
-Style/LineEndConcatenation:
-  Enabled: true
-
-Style/MagicCommentFormat: # new in 1.35
-  Enabled: true
-
-Style/MapCompactWithConditionalBlock: # new in 1.30
-  Enabled: true
-
-# Overridden to false in Chopin
-Style/MapToHash:
-  Enabled: true
-
-# Overridden to false in Chopin
-Style/MapToSet:
-  Enabled: true
-
-# For discussion - 1000s of fails if set true 
-Style/MethodCallWithArgsParentheses:
-  Enabled: false
-
-Style/MethodCallWithoutArgsParentheses:
-  Enabled: false
-
-# For discussion - 100s of fails if set true
-Style/MethodCalledOnDoEndBlock:
-  Enabled: false
-
-Style/MethodDefParentheses:
-  Enabled: true
-
-Style/MinMax:
-  Enabled: true
-
-# Overridden to false in Chopin
-Style/MinMaxComparison:
-  Enabled: true
-
-Style/MissingElse:
-  Enabled: false
-
-Style/MissingRespondToMissing:
-  Enabled: true
-
-Style/MixinGrouping:
-  Enabled: true
-
-Style/MixinUsage:
-  Enabled: true
-
-Style/ModuleFunction:
-  Enabled: true
-
-Style/MultilineBlockChain:
-  Enabled: true
-
-Style/MultilineIfModifier:
-  Enabled: true
-
-Style/MultilineIfThen:
-  Enabled: true
-
-Style/MultilineInPatternThen: # new in 1.16
-  Enabled: true
-
-Style/MultilineMemoization:
-  Enabled: true
-
-# For discussion - suggest true, but 100s of fails in Chopin and handful in CoCo
-Style/MultilineMethodSignature:
-  Enabled: false
-
-Style/MultilineTernaryOperator:
-  Enabled: true
-
-Style/MultilineWhenThen:
-  Enabled: true
-
-Style/MultipleComparison:
-  Enabled: true
-
-Style/MutableConstant:
-  Enabled: false
-
-Style/NegatedIf:
-  Enabled: true
-
-Style/NegatedIfElseCondition:
-  Enabled: true
-
-Style/NegatedUnless:
-  Enabled: true
-
-Style/NegatedWhile:
-  Enabled: true
-
-Style/NestedFileDirname: 
-  Enabled: true
-
-Style/NestedModifier:
-  Enabled: true
-
-Style/NestedParenthesizedCalls:
-  Enabled: true
-
-Style/NestedTernaryOperator:
-  Enabled: true
-
-Style/Next:
-  Enabled: true
-
-Style/NilComparison:
-  Enabled: true
-
-Style/NilLambda:
-  Enabled: true
-
-Style/NonNilCheck:
-  Enabled: true
-
-Style/Not:
-  Enabled: true
-
-Style/NumberedParameters: 
-  Enabled: true
-
-Style/NumberedParametersLimit: 
-  Enabled: true
-
-Style/NumericLiteralPrefix:
-  Enabled: true
-
-Style/NumericLiterals:
-  Enabled: true
-
-Style/NumericPredicate:
-  Enabled: false
-
-Style/ObjectThen:
-  Enabled: true
-
-Style/OneLineConditional:
-  Enabled: true
-
-# Overridden to false in Chopin
-Style/OpenStructUse:
-  Enabled: true
-
-Style/OperatorMethodCall:
-  Enabled: true
-
-Style/OptionHash:
-  Enabled: true
-
-Style/OptionalArguments:
-  Enabled: true
-
-Style/OptionalBooleanParameter:
-  Enabled: true
-
-Style/OrAssignment:
-  Enabled: true
-
-Style/ParallelAssignment:
-  Enabled: true
-
-Style/ParenthesesAroundCondition:
-  Enabled: true
-
-Style/PercentLiteralDelimiters:
-  Enabled: true
-
-Style/PercentQLiterals:
-  Enabled: true
-
-Style/PerlBackrefs:
-  Enabled: true
-
-Style/PreferredHashMethods:
-  Enabled: true
-
-Style/Proc:
-  Enabled: true
-
-# Overridden to false in Chopin and Rater
-Style/QuotedSymbols:
-  Enabled: true
-
-Style/RaiseArgs:
-  Enabled: true
-
-Style/RandomWithOffset:
-  Enabled: true
-
-Style/RedundantArgument:
-  Enabled: false
-
-Style/RedundantArrayConstructor:
-  Enabled: true
-
-Style/RedundantAssignment:
-  Enabled: true
-
-Style/RedundantBegin:
-  Enabled: true
-
-Style/RedundantCapitalW:
-  Enabled: true
-
-Style/RedundantCondition:
-  Enabled: true
-
-Style/RedundantConditional:
-  Enabled: true
-
-Style/RedundantConstantBase:
-  Enabled: false
-
-Style/RedundantCurrentDirectoryInPath:
-  Enabled: false
-
-Style/RedundantDoubleSplatHashBraces:
-  Enabled: true
-
-Style/RedundantEach:
-  Enabled: true
-
-Style/RedundantException:
-  Enabled: true
-
-Style/RedundantFetchBlock:
-  Enabled: true
-
-Style/RedundantFileExtensionInRequire:
-  Enabled: true
-
-Style/RedundantFilterChain:
-  Enabled: true
-
-Style/RedundantFreeze:
-  Enabled: true
-
-Style/RedundantHeredocDelimiterQuotes:
-  Enabled: true
-
-Style/RedundantInitialize:
-  Enabled: true
-
-Style/RedundantInterpolation:
-  Enabled: false
-
-Style/RedundantLineContinuation:
-  Enabled: true
-
-Style/RedundantParentheses:
-  Enabled: true
-
-Style/RedundantPercentQ:
-  Enabled: true
-
-Style/RedundantRegexpArgument:
-  Enabled: true
-
-Style/RedundantRegexpCharacterClass:
-  Enabled: true
-
-Style/RedundantRegexpConstructor:
-  Enabled: true
-
-Style/RedundantRegexpEscape:
-  Enabled: true
-
-Style/RedundantReturn:
-  Enabled: true
-  Severity: warning
-
-Style/RedundantSelf:
-  Enabled: true
-
-Style/RedundantSelfAssignment:
-  Enabled: true
-
-# Overridden to false in Chopin
-Style/RedundantSelfAssignmentBranch:
-  Enabled: true
-
-Style/RedundantSort:
-  Enabled: true
-
-Style/RedundantSortBy:
-  Enabled: true
-
-# Overridden to false in Chopin
-Style/RedundantStringEscape:
-  Enabled: true
-
-Style/RegexpLiteral:
-  Enabled: true
-
-# For discussion - would improve legibility if set true
-Style/RequireOrder:
-  Enabled: false
-
-Style/RescueModifier:
-  Enabled: true
-
-Style/RescueStandardError:
-  Enabled: true
-
-# For discussion - agreed configuration would be nice
-Style/ReturnNil:
-  Enabled: false
-
-# For discussion - as ReturnNil agreed configuration would be nice
-Style/ReturnNilInPredicateMethodDefinition:
-  Enabled: false  
-
-Style/SafeNavigation:
-  Enabled: false
-
-Style/Sample:
-  Enabled: true
-
-# Overridden to false in Chopin
-Style/SelectByRegexp:
-  Enabled: true
-
-Style/SelfAssignment:
-  Enabled: true
-
-Style/Semicolon:
-  Enabled: true
-
-Style/Send:
-  Enabled: false
-
-Style/SignalException:
-  Enabled: true
-
-Style/SingleArgumentDig:
-  Enabled: true
-
-# If this is to be set true, desired param names need to be set for reduce and inject
-# (default is |acc, elem|) and current usage suggests domain relevant names are preferred
-Style/SingleLineBlockParams:
-  Enabled: false
-
-Style/SingleLineMethods:
-  Enabled: true
-
-Style/SlicingWithRange:
-  Enabled: false
-
-Style/SoleNestedConditional:
-  Enabled: true
-
-Style/SpecialGlobalVars:
-  Enabled: true
-  Severity: warning
-
-Style/StabbyLambdaParentheses:
-  Enabled: true
-
-# Unsafe autocorrect and many fails on multiple repos
-Style/StaticClass:
-  Enabled: false
-
-# TODO: Small number of autocorrects in Chopin
-Style/StderrPuts:
-  Enabled: true
-
-Style/StringChars:
-  Enabled: true
-
-Style/StringConcatenation:
-  Enabled: true
-
-Style/StringHashKeys:
-  Enabled: false
-
-Style/StringLiterals:
-  Enabled: false
-
-Style/StringLiteralsInInterpolation:
-  Enabled: true
-
-Style/StringMethods:
-  Enabled: true
-
-Style/Strip:
-  Enabled: true
-
-Style/StructInheritance:
-  Enabled: true
-
-Style/SwapValues:
-  Enabled: true
-
-Style/SymbolArray:
-  EnforcedStyle: brackets
-
-Style/SymbolLiteral:
-  Enabled: true
-
-Style/SymbolProc:
-  Enabled: true
-
-Style/TernaryParentheses:
-  Enabled: true
-
-# For discussion - suggest should be true or at least warn but large number of fails on repos
-Style/TopLevelMethodDefinition:
-  Enabled: false
-
-Style/TrailingBodyOnClass:
-  Enabled: true
-
-Style/TrailingBodyOnMethodDefinition:
-  Enabled: true
-
-Style/TrailingBodyOnModule:
-  Enabled: true
-
-# Will require Chopin override
-Style/TrailingCommaInArguments:
-  Enabled: true
-
-# Will require Chopin override
-Style/TrailingCommaInArrayLiteral:
-  Enabled: true
-
-Style/TrailingCommaInBlockArgs:
-  Enabled: true
-
-# Will require Chopin override
-Style/TrailingCommaInHashLiteral:
-  Enabled: true
-
-Style/TrailingMethodEndStatement:
-  Enabled: true
-
-Style/TrailingUnderscoreVariable:
-  Enabled: true
-
-Style/TrivialAccessors:
-  Enabled: true
-
-Style/UnlessElse:
-  Enabled: true
-
-Style/UnlessLogicalOperators:
-  Enabled: true
-
-Style/UnpackFirst:
-  Enabled: true
-
-Style/VariableInterpolation:
-  Enabled: true
-
-Style/WhenThen:
-  Enabled: true
-
-Style/WhileUntilDo:
-  Enabled: true
-
-Style/WhileUntilModifier:
-  Enabled: true
-
-# Overridden to false in Chopin
-Style/WordArray:
-  EnforcedStyle: brackets
-
-Style/YAMLFileRead:
-  Enabled: true
-
-Style/YodaCondition:
-  Enabled: false
-
-Style/YodaExpression:
-  Enabled: false
-
-Style/ZeroLengthPredicate:
-  Enabled: true