armitage-rubocop 0.88.0 → 0.89.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 140748b670a47306e4446b79a9c61665e5dcef9f9264af29e3d353d6acdbb0a3
-  data.tar.gz: 3f4d1be7db4045eef7b5315ffbe33666be78226d65b3a88f847b54aa3c6e7f2c
+  metadata.gz: c28154435f7b4b3dce611b7eedca8497f6e5d6e53360459b7e9a9c8ac140bb99
+  data.tar.gz: e0080568e776946dc84708b57ca704bdfbeb1fcde5d7a5d7777a41b701ed491e
 SHA512:
-  metadata.gz: 25f30f612097f816d8e3c47fee8fc6d28bd59c9cd07255d4294582a7b86328ea28b0e2e3520b0587469bbe55cef357de12393ad0139931736e5d50e3f4b6fcad
-  data.tar.gz: c241318e983d388bbe248be699b8968ffd1cbe487d637074ce93e4be3fdc3688a057be58ce8f4671eb5dea6d9ec45d507bf0aafebda4699d6d4178d3bfaa7037
+  metadata.gz: 4bff4699b1f8920f5d72db839673d0b5e167d5607f929a782e6d9945182fe18f3b557bb6beb4136afff858bb10d181bab9006be715fe1c21e6bd7f38fbebafc3
+  data.tar.gz: 4772b4ee3cf65a8f3abe93bd33dda81bb00f4ffddefc8f4bc281f0826c5d194d4132646b865a721792f16783dd08ef09e39dd728721ce867bd9ba68048b0b8d3

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.88.0'
+  spec.version     = '0.89.1'
 
   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.88.0'
+  spec.add_dependency 'rubocop',             '= 0.89.1'
   spec.add_dependency 'rubocop-performance', '= 1.7.1'
-  spec.add_dependency 'rubocop-rails',       '= 2.6.0'
-  spec.add_dependency 'rubocop-rspec',       '= 1.42.0'
+  spec.add_dependency 'rubocop-rails',       '= 2.7.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

@@ -367,6 +367,9 @@ Layout/IndentationStyle:
   EnforcedStyle: spaces
   IndentationWidth: 2
 
+Layout/SpaceAroundMethodCallOperator:
+  Enabled: true
+
 Layout/TrailingEmptyLines:
   Enabled: true
   EnforcedStyle: final_newline

data/lib/common/lint.yml

@@ -14,6 +14,9 @@ Lint/AssignmentInCondition:
 Lint/BigDecimalNew:
   Enabled: true
 
+Lint/BinaryOperatorWithIdenticalOperands:
+  Enabled: true
+
 Lint/BooleanSymbol:
   Enabled: false
 
@@ -53,9 +56,16 @@ Lint/DuplicateElsifCondition:
 Lint/EachWithObjectArgument:
   Enabled: true
 
+Lint/DuplicateRescueException:
+  Enabled: true
+
 Lint/ElseLayout:
   Enabled: true
 
+Lint/EmptyConditionalBody:
+  Enabled: true
+  AllowComments: true
+
 Lint/EmptyEnsure:
   Enabled: true
   AutoCorrect: true
@@ -72,7 +82,6 @@ Lint/EmptyWhen:
 
 Lint/EnsureReturn:
   Enabled: true
-  AutoCorrect: false
 
 Lint/ErbNewArguments:
   Enabled: true
@@ -84,10 +93,13 @@ Lint/FlipFlop:
 Lint/FloatOutOfRange:
   Enabled: true
 
+Lint/FloatComparison:
+  Enabled: false
+
 Lint/FormatParameterMismatch:
   Enabled: true
 
-Layout/SpaceAroundMethodCallOperator:
+Lint/SelfAssignment:
   Enabled: true
 
 Lint/StructNewOverride:
@@ -121,6 +133,7 @@ Lint/LiteralInInterpolation:
 
 Lint/Loop:
   Enabled: true
+  AutoCorrect: true
 
 Lint/MissingCopEnableDirective:
   Enabled: true
@@ -129,6 +142,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
 
@@ -153,6 +169,9 @@ Lint/NumberConversion:
 Lint/OrderedMagicComments:
   Enabled: true
 
+Lint/OutOfRangeRegexpRef:
+  Enabled: true
+
 Lint/ParenthesesAsGroupedExpression:
   Enabled: true
   AutoCorrect: true
@@ -229,12 +248,18 @@ Lint/Syntax:
 Lint/ToJSON:
   Enabled: true
 
+Lint/TopLevelReturnWithArgument:
+  Enabled: true
+
 Lint/UnderscorePrefixedVariableName:
   Enabled: false
 
 Lint/UnifiedInteger:
   Enabled: true
 
+Lint/UnreachableLoop:
+  Enabled: true
+
 Lint/RedundantCopDisableDirective:
   Enabled: true
 
@@ -279,9 +304,6 @@ Lint/UselessAccessModifier:
 Lint/UselessAssignment:
   Enabled: true
 
-Lint/UselessComparison:
-  Enabled: true
-
 Lint/UselessElseWithoutRescue:
   Enabled: true
 

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:
@@ -52,6 +57,9 @@ Rails/Date:
   Enabled: true
   EnforcedStyle: flexible
 
+Rails/DefaultScope:
+  Enabled: false
+
 Rails/Delegate:
   Enabled: false
 
@@ -88,6 +96,9 @@ Rails/FilePath:
 Rails/FindBy:
   Enabled: true
 
+Rails/FindById:
+  Enabled: true
+
 Rails/FindEach:
   Enabled: true
 
@@ -112,6 +123,9 @@ Rails/IndexBy:
 Rails/IndexWith:
   Enabled: true
 
+Rails/Inquiry:
+  Enabled: false
+
 Rails/InverseOf:
   Enabled: true
 
@@ -121,6 +135,16 @@ 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
 
@@ -133,6 +157,15 @@ Rails/OutputSafety:
 Rails/Pick:
   Enabled: true
 
+Rails/Pluck:
+  Enabled: true
+
+Rails/PluckId:
+  Enabled: false
+
+Rails/PluckInWhere:
+  Enabled: true
+
 Rails/PluralizationGrammar:
   Enabled: true
 
@@ -170,6 +203,13 @@ Rails/RelativeDateConstant:
   Enabled: true
   AutoCorrect: false
 
+Rails/RenderInline:
+  Enabled: false
+
+Rails/RenderPlainText:
+  Enabled: true
+  ContentTypeCompatibility: true
+
 Rails/RequestReferer:
   Enabled: true
   EnforcedStyle: referer
@@ -190,6 +230,10 @@ Rails/SaveBang:
 Rails/ScopeArgs:
   Enabled: true
 
+Rails/ShortI18n:
+  Enabled: true
+  EnforcedStyle: conservative
+
 Rails/SkipsModelValidations:
   Enabled: false
 
@@ -216,3 +260,6 @@ Rails/UnknownEnv:
 
 Rails/Validation:
   Enabled: true
+
+Rails/WhereExists:
+  Enabled: false

data/lib/common/style.yml

@@ -52,6 +52,8 @@ Style/BlockDelimiters:
 
 Style/CaseEquality:
   Enabled: true
+  AllowOnConstant: false
+  AutoCorrect: true
 
 Style/CaseLikeIf:
   Enabled: true
@@ -169,6 +171,9 @@ Style/ExponentialNotation:
   Enabled: true
   EnforcedStyle: scientific
 
+Style/ExplicitBlockArgument:
+  Enabled: true
+
 Style/FloatDivision:
   Enabled: false
 
@@ -193,6 +198,9 @@ Style/GlobalVars:
   Enabled: true
   AllowedVariables: []
 
+Style/GlobalStdStream:
+  Enabled: false
+
 Style/GuardClause:
   Enabled: false
   MinBodyLength: 1
@@ -278,9 +286,6 @@ Style/MethodDefParentheses:
   Enabled: true
   EnforcedStyle: require_parentheses
 
-Style/MethodMissingSuper:
-  Enabled: false
-
 Style/MinMax:
   Enabled: true
 
@@ -357,7 +362,7 @@ Style/Next:
 
 Style/NilComparison:
   Enabled: true
-  EnforcedStyle: predicate
+  EnforcedStyle: comparison
 
 Style/NonNilCheck:
   Enabled: true
@@ -374,9 +379,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:
@@ -388,6 +393,9 @@ Style/OptionHash:
 Style/OptionalArguments:
   Enabled: true
 
+Style/OptionalBooleanParameter:
+  Enabled: false
+
 Style/ParallelAssignment:
   Enabled: false
 
@@ -492,6 +500,9 @@ Style/SignalException:
   Enabled: true
   EnforcedStyle: only_raise
 
+Style/SingleArgumentDig:
+  Enabled: true
+
 Style/SingleLineBlockParams:
   Enabled: false
 
@@ -514,6 +525,9 @@ Style/StabbyLambdaParentheses:
 Style/StderrPuts:
   Enabled: true
 
+Style/StringConcatenation:
+  Enabled: true
+
 Style/StringHashKeys:
   Enabled: false
 

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.88.0
+  version: 0.89.1
 platform: ruby
 authors:
 - Rustam Ibragimov
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2020-07-19 00:00:00.000000000 Z
+date: 2020-08-25 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rubocop
@@ -16,14 +16,14 @@ dependencies:
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 0.88.0
+        version: 0.89.1
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 0.88.0
+        version: 0.89.1
 - !ruby/object:Gem::Dependency
   name: rubocop-performance
   requirement: !ruby/object:Gem::Requirement
@@ -44,28 +44,28 @@ dependencies:
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 2.6.0
+        version: 2.7.1
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 2.6.0
+        version: 2.7.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