armitage-rubocop 0.87.1.2 → 0.92.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: a3aaa361e243152b6d6ac78800cf0906e16538056654622eeb0c7492900f2ab8
-  data.tar.gz: f9e69f06fd500f7570303f31500da4b24119103439379a7c171b71c96457031f
+  metadata.gz: fbc0557d76b3ad87b560318672e7d972683e28746c2940bb53f4c2e927bfa304
+  data.tar.gz: 211abc70c147c6d5b27d3953daf4980d8265f795db6a857723bda539daec31eb
 SHA512:
-  metadata.gz: ef62f974dbb6c9165943f3e4584adbf1489ee6ea3de2edae9a69cdf12fbfa039fa2be5211ad17057c859d4c836a2ad81a8f9b2a43da270fa52fbfec39d8c1b62
-  data.tar.gz: 9e759bce179f0ad0330d801ce30b9e9d36b4031f624ec328f41e69d40f15146059c998f772cf81bdb3c8a86962714fecd0648b11a585a1af94bd1c589f83d88c
+  metadata.gz: c060c8bd643347ebad317d6a43ecfabda9db037c2c109f0efe1c8e6abab9abb6ff539eadb194da313dac137e9e737fe565ddca77275095f202119cfcaf31a316
+  data.tar.gz: 971daba115f8495e949e3bce691586de6db7d28f42b291efda6f19361976b676f4ba5171f2021e18fd423b03236951f86d283644060c04bf259568586e18eae9

data/armitage-rubocop.gemspec

@@ -4,7 +4,7 @@ lib = File.expand_path('lib', __dir__)
 $LOAD_PATH.unshift(lib) unless $LOAD_PATH.include?(lib)
 
 Gem::Specification.new do |spec|
-  spec.version     = '0.87.1.2'
+  spec.version     = '0.92.0'
 
   spec.name        = 'armitage-rubocop'
   spec.license     = 'MIT'
@@ -22,10 +22,10 @@ Gem::Specification.new do |spec|
     `git ls-files -z`.split("\x0").reject { |f| f.match(%r{^(test|spec|features)/}) }
   end
 
-  spec.add_dependency 'rubocop',             '= 0.87.1'
-  spec.add_dependency 'rubocop-performance', '= 1.7.0'
-  spec.add_dependency 'rubocop-rails',       '= 2.6.0'
-  spec.add_dependency 'rubocop-rspec',       '= 1.42.0'
+  spec.add_dependency 'rubocop',             '= 0.92.0'
+  spec.add_dependency 'rubocop-performance', '= 1.8.1'
+  spec.add_dependency 'rubocop-rails',       '= 2.8.1'
+  spec.add_dependency 'rubocop-rspec',       '= 1.43.2'
   spec.add_dependency 'rubocop-rake',        '= 0.5.1'
 
   spec.add_development_dependency 'bundler'

data/lib/common/layout.yml

@@ -24,6 +24,11 @@ Layout/ParameterAlignment:
   EnforcedStyle: with_first_parameter
   IndentationWidth: ~
 
+Layout/BeginEndAlignment:
+  Enabled: true
+  EnforcedStyleAlignWith: start_of_line
+  Severity: warning
+
 Layout/BlockAlignment:
   Enabled: true
   EnforcedStyleAlignWith: either
@@ -90,6 +95,9 @@ Layout/EmptyLineAfterGuardClause:
 Layout/EmptyLineAfterMagicComment:
   Enabled: true
 
+Layout/EmptyLineAfterMultilineCondition:
+  Enabled: false
+
 Layout/EmptyLinesAroundAttributeAccessor:
   Enabled: true
   AllowAliasSyntax: true
@@ -367,6 +375,9 @@ Layout/IndentationStyle:
   EnforcedStyle: spaces
   IndentationWidth: 2
 
+Layout/SpaceAroundMethodCallOperator:
+  Enabled: true
+
 Layout/TrailingEmptyLines:
   Enabled: true
   EnforcedStyle: final_newline

data/lib/common/lint.yml

@@ -14,12 +14,18 @@ Lint/AssignmentInCondition:
 Lint/BigDecimalNew:
   Enabled: true
 
+Lint/BinaryOperatorWithIdenticalOperands:
+  Enabled: true
+
 Lint/BooleanSymbol:
   Enabled: false
 
 Lint/CircularArgumentReference:
   Enabled: true
 
+Lint/ConstantDefinitionInBlock:
+  Enabled: false
+
 Lint/ConstantResolution:
   Enabled: false
   Only: []
@@ -34,6 +40,10 @@ Lint/DeprecatedClassMethods:
 Lint/DeprecatedOpenSSLConstant:
   Enabled: true
 
+Lint/DisjunctiveAssignmentInConstructor:
+  Enabled: true
+  AutoCorrect: false
+
 Lint/DuplicateCaseCondition:
   Enabled: true
 
@@ -43,12 +53,25 @@ Lint/DuplicateMethods:
 Lint/DuplicateHashKey:
   Enabled: true
 
+Lint/DuplicateElsifCondition:
+  Enabled: true
+
+Lint/DuplicateRequire:
+  Enabled: true
+
 Lint/EachWithObjectArgument:
   Enabled: true
 
+Lint/DuplicateRescueException:
+  Enabled: true
+
 Lint/ElseLayout:
   Enabled: true
 
+Lint/EmptyConditionalBody:
+  Enabled: true
+  AllowComments: true
+
 Lint/EmptyEnsure:
   Enabled: true
   AutoCorrect: true
@@ -56,6 +79,10 @@ Lint/EmptyEnsure:
 Lint/EmptyExpression:
   Enabled: true
 
+Lint/EmptyFile:
+  Enabled: true
+  AllowComments: true
+
 Lint/EmptyInterpolation:
   Enabled: true
 
@@ -65,7 +92,6 @@ Lint/EmptyWhen:
 
 Lint/EnsureReturn:
   Enabled: true
-  AutoCorrect: false
 
 Lint/ErbNewArguments:
   Enabled: true
@@ -77,10 +103,13 @@ Lint/FlipFlop:
 Lint/FloatOutOfRange:
   Enabled: true
 
+Lint/FloatComparison:
+  Enabled: false
+
 Lint/FormatParameterMismatch:
   Enabled: true
 
-Layout/SpaceAroundMethodCallOperator:
+Lint/SelfAssignment:
   Enabled: true
 
 Lint/StructNewOverride:
@@ -92,6 +121,9 @@ Lint/SuppressedException:
 Lint/HeredocMethodCallPosition:
   Enabled: true
 
+Lint/IdentityComparison:
+  Enabled: false
+
 Lint/ImplicitStringConcatenation:
   Enabled: true
 
@@ -114,6 +146,7 @@ Lint/LiteralInInterpolation:
 
 Lint/Loop:
   Enabled: true
+  AutoCorrect: true
 
 Lint/MissingCopEnableDirective:
   Enabled: true
@@ -122,6 +155,9 @@ Lint/MixedRegexpCaptureTypes:
   Enabled: true
   # NOTE: if you use named captures - numbered captures will be ignored. it is bad. validate it.
 
+Lint/MissingSuper:
+  Enabled: false
+
 Lint/MultipleComparison:
   Enabled: true
 
@@ -146,6 +182,9 @@ Lint/NumberConversion:
 Lint/OrderedMagicComments:
   Enabled: true
 
+Lint/OutOfRangeRegexpRef:
+  Enabled: true
+
 Lint/ParenthesesAsGroupedExpression:
   Enabled: true
   AutoCorrect: true
@@ -222,12 +261,21 @@ Lint/Syntax:
 Lint/ToJSON:
   Enabled: true
 
+Lint/TopLevelReturnWithArgument:
+  Enabled: true
+
+Lint/TrailingCommaInAttributeDeclaration:
+  Enabled: true
+
 Lint/UnderscorePrefixedVariableName:
   Enabled: false
 
 Lint/UnifiedInteger:
   Enabled: true
 
+Lint/UnreachableLoop:
+  Enabled: true
+
 Lint/RedundantCopDisableDirective:
   Enabled: true
 
@@ -272,15 +320,19 @@ Lint/UselessAccessModifier:
 Lint/UselessAssignment:
   Enabled: true
 
-Lint/UselessComparison:
+Lint/UselessElseWithoutRescue:
   Enabled: true
 
-Lint/UselessElseWithoutRescue:
+Lint/UselessMethodDefinition:
   Enabled: true
+  AllowComments: true
 
 Lint/UselessSetterCall:
   Enabled: true
 
+Lint/UselessTimes:
+  Enabled: true
+
 Lint/Void:
   Enabled: true
   CheckForMethodsWithNoSideEffects: false

data/lib/common/performance.yml

@@ -23,6 +23,10 @@ Performance/ChainArrayAllocation:
 Performance/CompareWithBlock:
   Enabled: true
 
+Performance/CollectionLiteralInLoop:
+  Enabled: true
+  MinSize: 1
+
 Performance/Count:
   Enabled: true
   SafeAutoCorrect: false
@@ -108,6 +112,9 @@ Performance/StringInclude:
 Performance/StringReplacement:
   Enabled: true
 
+Performance/Sum:
+  Enabled: true
+
 Performance/TimesMap:
   Enabled: true
   AutoCorrect: false

data/lib/common/rails.yml

@@ -5,6 +5,11 @@ Rails/ActionFilter:
 Rails/ActiveRecordAliases:
   Enabled: true
 
+Rails/ActiveRecordCallbacksOrder:
+  Enabled: true
+  Include:
+    - app/models/**/*.rb
+
 Rails/ActiveRecordOverride:
   Enabled: true
   Include:
@@ -13,6 +18,9 @@ Rails/ActiveRecordOverride:
 Rails/ActiveSupportAliases:
   Enabled: true
 
+Rails/AfterCommitOverride:
+  Enabled: true
+
 Rails/ApplicationJob:
   Enabled: true
 
@@ -52,6 +60,9 @@ Rails/Date:
   Enabled: true
   EnforcedStyle: flexible
 
+Rails/DefaultScope:
+  Enabled: false
+
 Rails/Delegate:
   Enabled: false
 
@@ -88,6 +99,9 @@ Rails/FilePath:
 Rails/FindBy:
   Enabled: true
 
+Rails/FindById:
+  Enabled: true
+
 Rails/FindEach:
   Enabled: true
 
@@ -112,6 +126,9 @@ Rails/IndexBy:
 Rails/IndexWith:
   Enabled: true
 
+Rails/Inquiry:
+  Enabled: false
+
 Rails/InverseOf:
   Enabled: true
 
@@ -121,9 +138,22 @@ Rails/LexicallyScopedActionFilter:
 Rails/LinkToBlank:
   Enabled: true
 
+Rails/MailerName:
+  Enabled: true
+  # mailer views are mapped to the mailer class with Mailer postfix in the name of a class
+
+Rails/MatchRoute:
+  Enabled: true
+
+Rails/NegateInclude:
+  Enabled: true
+
 Rails/NotNullColumn:
   Enabled: true
 
+Rails/OrderById:
+  Enabled: false
+
 Rails/Output:
   Enabled: true
 
@@ -133,6 +163,16 @@ Rails/OutputSafety:
 Rails/Pick:
   Enabled: true
 
+Rails/Pluck:
+  Enabled: true
+
+Rails/PluckId:
+  Enabled: false
+
+Rails/PluckInWhere:
+  Enabled: true
+  EnforcedStyle: conservative
+
 Rails/PluralizationGrammar:
   Enabled: true
 
@@ -170,6 +210,13 @@ Rails/RelativeDateConstant:
   Enabled: true
   AutoCorrect: false
 
+Rails/RenderInline:
+  Enabled: false
+
+Rails/RenderPlainText:
+  Enabled: true
+  ContentTypeCompatibility: true
+
 Rails/RequestReferer:
   Enabled: true
   EnforcedStyle: referer
@@ -190,9 +237,16 @@ Rails/SaveBang:
 Rails/ScopeArgs:
   Enabled: true
 
+Rails/ShortI18n:
+  Enabled: true
+  EnforcedStyle: conservative
+
 Rails/SkipsModelValidations:
   Enabled: false
 
+Rails/SquishedSQLHeredocs:
+  Enabled: true
+
 Rails/TimeZone:
   Enabled: true
   EnforcedStyle: flexible
@@ -216,3 +270,10 @@ Rails/UnknownEnv:
 
 Rails/Validation:
   Enabled: true
+
+Rails/WhereExists:
+  Enabled: false
+  EnforcedStyle: exists
+
+Rails/WhereNot:
+  Enabled: false

data/lib/common/style.yml

@@ -11,6 +11,9 @@ Style/Alias:
   Enabled: true
   EnforcedStyle: prefer_alias_method
 
+Style/ArrayCoercion:
+  Enabled: true
+
 Style/AndOr:
   Enabled: false
   EnforcedStyle: always
@@ -49,6 +52,12 @@ Style/BlockDelimiters:
 
 Style/CaseEquality:
   Enabled: true
+  AllowOnConstant: false
+  AutoCorrect: true
+
+Style/CaseLikeIf:
+  Enabled: true
+  AutoCorrect: true
 
 Style/CharacterLiteral:
   Enabled: true
@@ -63,6 +72,10 @@ Style/ClassCheck:
 Style/ClassMethods:
   Enabled: true
 
+Style/ClassMethodsDefinitions:
+  Enabled: false
+  EnforcedStyle: self_class
+
 Style/ClassVars:
   Enabled: true
 
@@ -75,6 +88,9 @@ Style/ColonMethodCall:
 Style/ColonMethodDefinition:
   Enabled: true
 
+Style/CombinableLoops:
+  Enabled: false
+
 Style/CommandLiteral:
   Enabled: false
 
@@ -96,6 +112,7 @@ Style/Copyright:
 Style/DateTime:
   Enabled: false
   AllowCoercion: false
+  AutoCorrect: false
 
 Style/DefWithParentheses:
   Enabled: true
@@ -162,6 +179,9 @@ Style/ExponentialNotation:
   Enabled: true
   EnforcedStyle: scientific
 
+Style/ExplicitBlockArgument:
+  Enabled: true
+
 Style/FloatDivision:
   Enabled: false
 
@@ -186,10 +206,22 @@ Style/GlobalVars:
   Enabled: true
   AllowedVariables: []
 
+Style/GlobalStdStream:
+  Enabled: false
+
 Style/GuardClause:
   Enabled: false
   MinBodyLength: 1
 
+Style/HashAsLastArrayItem:
+  Enabled: true
+  EnforcedStyle: braces
+  AutoCorrect: true
+
+Style/HashLikeCase:
+  Enabled: false
+  MinBranchesCount: 4
+
 Style/HashEachMethods:
   Enabled: true
 
@@ -238,6 +270,9 @@ Style/InverseMethods:
 Style/IpAddresses:
   Enabled: false
 
+Style/KeywordParametersOrder:
+  Enabled: false
+
 Style/Lambda:
   Enabled: true
   EnforcedStyle: line_count_dependent
@@ -262,9 +297,6 @@ Style/MethodDefParentheses:
   Enabled: true
   EnforcedStyle: require_parentheses
 
-Style/MethodMissingSuper:
-  Enabled: false
-
 Style/MinMax:
   Enabled: true
 
@@ -341,7 +373,7 @@ Style/Next:
 
 Style/NilComparison:
   Enabled: true
-  EnforcedStyle: predicate
+  EnforcedStyle: comparison
 
 Style/NonNilCheck:
   Enabled: true
@@ -358,9 +390,9 @@ Style/NumericLiterals:
   Enabled: false
 
 Style/NumericPredicate:
-  Enabled: true
-  EnforcedStyle: predicate
-  AutoCorrect: false
+  Enabled: false # strongly situative cop
+  EnforcedStyle: comparison
+  AutoCorrect: true
   IgnoredMethods: []
 
 Style/OneLineConditional:
@@ -372,6 +404,11 @@ Style/OptionHash:
 Style/OptionalArguments:
   Enabled: true
 
+Style/OptionalBooleanParameter:
+  Enabled: false
+  AllowedMethods:
+    - respond_to_missing?
+
 Style/ParallelAssignment:
   Enabled: false
 
@@ -411,6 +448,9 @@ Style/RedundantBegin:
 Style/RedundantConditional:
   Enabled: true
 
+Style/RedundantFileExtensionInRequire:
+  Enabled: true
+
 Style/RedundantFetchBlock:
   Enabled: true
   SafeForConstants: true
@@ -437,6 +477,9 @@ Style/RedundantReturn:
 Style/RedundantSelf:
   Enabled: true
 
+Style/RedundantSelfAssignment:
+  Enabled: false
+
 Style/RedundantSortBy:
   Enabled: true
 
@@ -473,6 +516,9 @@ Style/SignalException:
   Enabled: true
   EnforcedStyle: only_raise
 
+Style/SingleArgumentDig:
+  Enabled: true
+
 Style/SingleLineBlockParams:
   Enabled: false
 
@@ -495,6 +541,9 @@ Style/StabbyLambdaParentheses:
 Style/StderrPuts:
   Enabled: true
 
+Style/StringConcatenation:
+  Enabled: true
+
 Style/StringHashKeys:
   Enabled: false
 
@@ -555,6 +604,7 @@ Style/TrailingCommaInBlockArgs:
 Style/TrailingCommaInHashLiteral:
   Enabled: true
   EnforcedStyleForMultiline: no_comma
+  AutoCorrect: true
 
 Style/TrailingMethodEndStatement:
   Enabled: true
@@ -584,6 +634,10 @@ Style/RedundantPercentQ:
 Style/RedundantSort:
   Enabled: true
 
+Style/SoleNestedConditional:
+  Enabled: true
+  AllowModifier: true
+
 Style/UnpackFirst:
   Enabled: true
 

data/lib/rspec/rspec.yml

@@ -160,6 +160,9 @@ RSpec/MultipleDescribes:
 RSpec/MultipleExpectations:
   Enabled: false
 
+RSpec/MultipleMemoizedHelpers:
+  Enabled: false
+
 RSpec/MultipleSubjects:
   Enabled: true
 
@@ -240,6 +243,7 @@ RSpec/VariableDefinition:
 RSpec/VariableName:
   Enabled: true
   EnforcedStyle: snake_case
+  IgnoredPatterns: []
 
 RSpec/VerifiedDoubles:
   Enabled: false

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: armitage-rubocop
 version: !ruby/object:Gem::Version
-  version: 0.87.1.2
+  version: 0.92.0
 platform: ruby
 authors:
 - Rustam Ibragimov
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2020-07-10 00:00:00.000000000 Z
+date: 2020-10-04 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rubocop
@@ -16,56 +16,56 @@ dependencies:
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 0.87.1
+        version: 0.92.0
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 0.87.1
+        version: 0.92.0
 - !ruby/object:Gem::Dependency
   name: rubocop-performance
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 1.7.0
+        version: 1.8.1
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 1.7.0
+        version: 1.8.1
 - !ruby/object:Gem::Dependency
   name: rubocop-rails
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 2.6.0
+        version: 2.8.1
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 2.6.0
+        version: 2.8.1
 - !ruby/object:Gem::Dependency
   name: rubocop-rspec
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 1.42.0
+        version: 1.43.2
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 1.42.0
+        version: 1.43.2
 - !ruby/object:Gem::Dependency
   name: rubocop-rake
   requirement: !ruby/object:Gem::Requirement