simplycop 1.12.18 → 1.12.21

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: bb0162eb0b49f4c38b6a0e5a3adb01ffcd6d58ba0ba4f0cf56fae6869229b974
-  data.tar.gz: 152f92c09cb44a1f852cd694e5676da8c5ea15766f9d0129010925663f485f11
+  metadata.gz: 68e568f306249b37bfb255cdd58ec03e209b2f1e0d81be18c8ef015dab756c4d
+  data.tar.gz: 84681f62a6470b474340dc9ab33310721a2caf41ada7245d2ab17c39e37b9038
 SHA512:
-  metadata.gz: 3b5387269617071c3b578ea6b9833e7431b256bdc05b79e82b59378ff6a9bfd1723b89aca949b4dc042169d6ef34cd88763021b4155b96c16726b6fdca047620
-  data.tar.gz: efdb8ba24eac8850c568844b3614451ad5acb51ac6f30298a1b38294a378921bef5c5aee68bdddf639b2d8208ed171e1041b2e34ce258776339761fcf11582d0
+  metadata.gz: f139da031074eb065eb31ecf446da61ce67c4fb33e7286dd601e8a236d44cdf3473e048b20f3936f615236e4a7ac866cfdb614bca263236fc17b33b04072898e
+  data.tar.gz: 8a02a4aef2f054cec5b66af68d2a922bf6cd0481c6337c51d3aff1de4ca82bbdfb6b0cd978537d639889475aec737e88f2bae86482bbc39b37d05caf7e402a84

data/.github/workflows/version_forget_me_not.yml

@@ -14,7 +14,7 @@ jobs:
     runs-on: ubuntu-latest
 
     steps:
-      - uses: simplybusiness/version-forget-me-not@v2.3.0
+      - uses: simplybusiness/version-forget-me-not@v2
         env:
           ACCESS_TOKEN: ${{ secrets.GITHUB_TOKEN }}
           VERSION_FILE_PATH: "lib/simplycop/version.rb"

data/.simplycop.yml

@@ -25,21 +25,166 @@ AllCops:
 Gemspec/DeprecatedAttributeAssignment: # new in 1.30
   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
+# Layout/ClassStructure:
+#   Enabled: true
+
+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
+
+# To discuss. Suggest should be true but 500+ autocorrectable fails on Chopin. None on others 
+# 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/FirstMethodArgumentLineBreak:
   Enabled: true
 
+Layout/FirstHashElementIndentation:
+  Enabled: true
+
 Layout/FirstHashElementLineBreak:
   Enabled: true
 
+# To discuss - 200+ fails on CoCo, none elsewhere
+# Layout/FirstMethodArgumentLineBreak:
+#   Enabled: true
+
+# To discuss - 100+ fails on Chopin, handful on others
+# 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
+
 Layout/LineContinuationLeadingSpace:
   Enabled: true
 
@@ -49,9 +194,148 @@ Layout/LineContinuationSpacing:
 Layout/LineEndStringConcatenationIndentation:
   Enabled: true
 
+# To discuss: configure line length? At default 120: 6805 fails on Chopin, 3 on Rater, 179 on CoCo
+# Layout/LineLength
+#   Enabled: false
+Layout/MultilineArrayBraceLayout:
+  Enabled: true
+  
+# To discuss: 100s of fails on each repo but autocorrectable  
+# Layout/MultilineArrayLineBreaks:
+#   Enabled: true
+
+# To discuss: 100s of fails on each repo but autocorrectable  
+# Layout/MultilineAssignmentLayout:
+#   Enabled: true
+  
+Layout/MultilineBlockLayout:
+  Enabled: true
+  
+Layout/MultilineHashBraceLayout:
+  Enabled: true
+
+# To discuss: 100s of fails on each repo but autocorrectable  
+# Layout/MultilineHashKeyLineBreaks:
+#   Enabled: true
+
+# To discuss: 1000s of fails on each repo but autocorrectable  
+# Layout/MultilineMethodArgumentLineBreaks:
+#   Enabled: true
+  
+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: true
+  
+Layout/MultilineOperationIndentation:
+  Enabled: true
+
+Layout/ParameterAlignment:
+  Enabled: true
+
+# For discussion: 1000s of fails, all autocorrectable
+# Layout/RedundantLineBreak:
+#   Enabled: true
+
+Layout/RescueEnsureAlignment:
+  Enabled: true
+
+# For discussion: 1000s of fails, all autocorrectable
+# Layout/SingleLineBlockChain:
+#   Enabled: true
+
+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
 
@@ -175,6 +459,9 @@ Style/AccessorGrouping:
 Style/Alias:
   Enabled: false
 
+Style/AndOr:
+  Enabled: true
+
 Style/ArgumentsForwarding:
   Enabled: true
 
@@ -184,43 +471,192 @@ Style/ArrayCoercion:
 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 
+
+# Requires discussion
+# Style/BarePercentLiterals:
+
+Style/BeginBlock:
+  Enabled: true
+
 Style/BisectedAttrAccessor:
   Enabled: true
 
+Style/BlockComments:
+  Enabled: true
+
+# Requires discussion - 1000+ offences in Chopin alone, unless options set
+# Style/BlockDelimiters:
+
+Style/CaseEquality:
+  Enabled: true
+
 Style/CaseLikeIf:
   Enabled: true
 
+Style/CharacterLiteral:
+  Enabled: true
+
+# Requires discussion
+# Style/ClassAndModuleChildren:
+
+Style/ClassCheck:
+  Enabled: true
+
+Style/ClassEqualityComparison:
+  Enabled: true
+
+Style/ClassMethods:
+  Enabled: true
+
+# Requires discussion - multiple fails on Chopin/Rater/CoCo when left at default
+# Style/ClassMethodsDefinitions:
+#   Enabled: true
+
+Style/ClassVars:
+  Enabled: true
+
 Style/CollectionCompact:
   Enabled: true
 
+# Requires discussion - multiple fails on Chopin/Rater/CoCo when left at default
+# Style/CollectionMethods:
+
 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:
+
+# For discussion
+# Style/Copyright:
+
+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:
+
 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
+
+# Suggest true but 100+ fails on Rater, none on Chopin or CoCo
+# Style/EachForSimpleLoop:
+#   Enabled: true
+
+Style/EachWithObject:
+  Enabled: true
+
+Style/EmptyBlockParameter:
+  Enabled: true
+
+Style/EmptyCaseCondition:
+  Enabled: true
+
+Style/EmptyElse:
+  Enabled: true
+
 Style/EmptyHeredoc: # new in 1.32
   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
+
+# Added in version 1.51, unable to check v Chopin, CoCo, Rater yet
+# Style/ExactRegexpMatch:
+#   Enabled: true
+
+Style/ExpandPathArguments:
+  Enabled: true
+
 Style/ExplicitBlockArgument:
   Enabled: true
 
@@ -230,12 +666,18 @@ Style/ExponentialNotation:
 Style/FetchEnvVar:
   Enabled: true
 
+Style/FileEmpty:
+  Enabled: true
+
 Style/FileRead: # new in 1.24
   Enabled: true
 
 Style/FileWrite:
   Enabled: true
 
+Style/FloatDivision:
+  Enabled: true
+
 Style/For:
   Enabled: true
 
@@ -251,6 +693,9 @@ Style/FrozenStringLiteralComment:
 Style/GlobalStdStream:
   Enabled: true
 
+Style/GlobalVars:
+  Enabled: true
+
 Style/GuardClause:
   Enabled: false
 
@@ -269,30 +714,72 @@ Style/HashExcept:
 Style/HashLikeCase:
   Enabled: false
 
+# For discussion - 11000+ fails on Chopin if true by default
+# Style/HashSyntax:
+#   Enabled: false
+
 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
+
+# For discussion - suggest false as 1000s of fails
+# Style/InlineComment:
+#   Enabled: false
+
+Style/InverseMethods:
+  Enabled: true
+
+# Manageable number of fails on Chopin, CoCo, Rater, suggest true
+# Style/InvertibleUnlessCondition:
+#   Enabled: true
+
 Style/InPatternThen: # new in 1.16
   Enabled: true
 
+# For discussion - multiple fails but most could be solved by excluding spec files
+# Style/IpAddresses:
+#   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
 
@@ -302,21 +789,90 @@ Style/MapCompactWithConditionalBlock: # new in 1.30
 Style/MapToHash:
   Enabled: true
 
+# TODO: small number of fixes in Chopin
+# Style/MapToSet:
+#   Enabled: true
+
+# For discussion - 1000s of fails, suggest false
+# Style/MethodCallWithArgsParentheses:
+#   Enabled: false
+
+Style/MethodCallWithoutArgsParentheses:
+  Enabled: false
+
+# For discussion - 100s of fails, suggest false
+# Style/MethodCalledOnDoEndBlock:
+#   Enabled: false
+
 Style/MethodDefParentheses:
   Enabled: true
 
+Style/MinMax:
+  Enabled: true
+
+# For discussion - multiple fails, suggest true
+# Style/MinMaxComparison:
+#   Enabled: true
+
+# For discussion - 100s of fails with default true
+# Style/MissingElse:
+#   Enabled: true
+
+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: true
+
 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: # new in 1.26
   Enabled: true
 
@@ -329,39 +885,91 @@ Style/NestedParenthesizedCalls:
 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: # new in 1.22
   Enabled: true
 
 Style/NumberedParametersLimit: # new in 1.22
   Enabled: true
 
+Style/NumericLiteralPrefix:
+  Enabled: true
+
+Style/NumericLiterals:
+  Enabled: true
+
 Style/NumericPredicate:
   Enabled: false
 
 Style/ObjectThen: # new in 1.28
   Enabled: true
 
+Style/OneLineConditional:
+  Enabled: true
+
 Style/OpenStructUse:
   Enabled: true
 
 Style/OperatorMethodCall:
   Enabled: true
 
+# For discussion - suggest true, but dozens of fails on Chopin and handful on Coco, Rater
+# 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
+
 Style/QuotedSymbols:
   Enabled: true
 
+Style/RaiseArgs:
+  Enabled: true
+
+Style/RandomWithOffset:
+  Enabled: true
+
 Style/RedundantArgument:
   Enabled: false
 
@@ -371,6 +979,15 @@ Style/RedundantAssignment:
 Style/RedundantBegin:
   Enabled: true
 
+Style/RedundantCapitalW:
+  Enabled: true
+
+Style/RedundantCondition:
+  Enabled: true
+
+Style/RedundantConditional:
+  Enabled: true
+
 Style/RedundantConstantBase:
   Enabled: false
 
@@ -389,15 +1006,27 @@ Style/RedundantFetchBlock:
 Style/RedundantFileExtensionInRequire:
   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/RedundantRegexpCharacterClass:
   Enabled: true
 
@@ -408,27 +1037,74 @@ Style/RedundantReturn:
   Enabled: true
   Severity: warning
 
+Style/RedundantSelf:
+  Enabled: true
+
 Style/RedundantSelfAssignment:
   Enabled: true
 
 Style/RedundantSelfAssignmentBranch:
   Enabled: true
 
+Style/RedundantSort:
+  Enabled: true
+
+Style/RedundantSortBy:
+  Enabled: true
+
 Style/RedundantStringEscape:
   Enabled: true
 
+# For discussion - CoCo, Rater OK, manageable amount of autocorrectable in Chopin if set true
+# Style/RegexpLiteral:
+#   Enabled: true
+
+# For discussion - dozens of fails, suggest false as autocorrect also unsafe
+# Style/RequireOrder:
+#   Enabled: false
+
+Style/RescueModifier:
+  Enabled: true
+
+Style/RescueStandardError:
+  Enabled: true
+
+# For discussion - suggest false as fails in all three if set true
+# Style/ReturnNil:
+#   Enabled: false
+
 Style/SafeNavigation:
   Enabled: false
 
+Style/Sample:
+  Enabled: true
+
 Style/SelectByRegexp:
   Enabled: true
 
 Style/SelfAssignment:
   Enabled: true
 
+Style/Semicolon:
+  Enabled: true
+
+# For discussion - 100s of fails on Chopin, some on CoCo
+# Style/Send:
+#   Enabled: true
+
+Style/SignalException:
+  Enabled: true
+
 Style/SingleArgumentDig:
   Enabled: true
 
+# TODO: Small number of autocorrectables on Chopin and Rater
+# Style/SingleLineBlockParams:
+#   Enabled: true
+
+Style/SingleLineMethods:
+  Enabled: true
+
 Style/SlicingWithRange:
   Enabled: false
 
@@ -439,26 +1115,126 @@ Style/SpecialGlobalVars:
   Enabled: true
   Severity: warning
 
+Style/StabbyLambdaParentheses:
+  Enabled: true
+
+# For discussion - 100s of fails across Rater, Chopin, CoCo, so suggest false
+# Style/StaticClass:
+#   Enabled: false
+
+# TODO: Small number of autocorrects in Chopin
+Style/StderrPuts:
+  Enabled: true
+
 Style/StringChars:
   Enabled: true
 
 Style/StringConcatenation:
   Enabled: true
 
+# For discussion - 10000s of fails in Chopin and Rater so suggest false
+# 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: true
+
+Style/TrailingBodyOnClass:
+  Enabled: true
+
+Style/TrailingBodyOnMethodDefinition:
+  Enabled: true
+
+Style/TrailingBodyOnModule:
+  Enabled: true
+
+# For discussion - 100s of fails on Chopin, none on others
+# Style/TrailingCommaInArguments:
+#   Enabled: true
+
+# For discussion - 72 fails on Chopin, none on others
+# Style/TrailingCommaInArrayLiteral:
+#   Enabled: true
+
+Style/TrailingCommaInBlockArgs:
+  Enabled: true
+
+# For discussion - 100s of fails on Chopin and Coco, none on Rater
+# Style/TrailingCommaInHashLiteral:
+#   Enabled: true
+
+Style/TrailingMethodEndStatement:
+  Enabled: true
+
+Style/TrailingUnderscoreVariable:
+  Enabled: true
+
+Style/TrivialAccessors:
+  Enabled: true
+
 Style/UnlessElse:
   Enabled: true
 
+# Single fail on each of Chopin and Rater
+# Style/UnlessLogicalOperators:
+#   Enabled: true
+
+Style/UnpackFirst:
+  Enabled: true
+
+Style/VariableInterpolation:
+  Enabled: true
+
+Style/WhenThen:
+  Enabled: true
+
+Style/WhileUntilDo:
+  Enabled: true
+
+Style/WhileUntilModifier:
+  Enabled: true
+
 Style/WordArray:
   EnforcedStyle: brackets
 
+# For discussion - causes no fails but maybe some may like it?
+# Style/YodaCondition:
+#   Enabled: true
+
+# For discussion with above - causes multiple fails
+# Style/YodaExpression:
+#   Enabled: true
+
 Style/ZeroLengthPredicate:
   Enabled: true

data/.simplycop_rspec.yml

@@ -117,6 +117,9 @@ Capybara/SpecificMatcher:
 FactoryBot/ConsistentParenthesesStyle:
   Enabled: true
 
+FactoryBot/CreateList:
+  Enabled: false
+
 FactoryBot/FactoryNameStyle:
   Enabled: true
 

data/lib/simplycop/version.rb

@@ -7,5 +7,5 @@
 #
 
 module Simplycop
-  VERSION = '1.12.18'
+  VERSION = '1.12.21'
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: simplycop
 version: !ruby/object:Gem::Version
-  version: 1.12.18
+  version: 1.12.21
 platform: ruby
 authors:
 - Simply Business
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2023-05-12 00:00:00.000000000 Z
+date: 2023-05-23 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rubocop