arcadia_cops 3.5.3 → 4.0.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: a307ceaf1d85dbfb6fdc61340ac7a377c1ccce6f425dce4638b430c4deea5577
-  data.tar.gz: 10596c1b66bc0a161d2d36472800919baa0eff4f3ce2125a8ff2193202cfb234
+  metadata.gz: 6651da173543ee1ffb9d8e31b0dfda75c35ef5454f9d1ff054160ae5d621041a
+  data.tar.gz: d7e2497155d064da862d66b65a1163f5abe01ef1a0afee89df5e879ac46ea281
 SHA512:
-  metadata.gz: 9c96aab67cfbd5b4200e56234bad254b63487a53520000be9f21905f83e2df33a25ce31b7c8364bdc256a2be314dc803f48b2dcb1e897367d08b20239f5a90b3
-  data.tar.gz: 2a9ec0ad89dbc322c8014cc861889fa03d1a5ed680aef2933e903d724b34c39c94bcd6b2ef7cea4c879f4500864588a32fa807eaf1526f2152e58983458f5b47
+  metadata.gz: dc99132ea7385e8d1e9cd2943b130fd2736dfef8f6a024b8b341fc4748ee519588f290c8af65c5806ef8460427d72ba21a4532a97bc9b36a54f7db12cb1cef59
+  data.tar.gz: '03087562410c086336d1d94f59431675a5b10bef7ad631acd27857e63eb15653c0bbc1fd5357a34f1d9e380f578c11ccee7963a650eb6b10f341a2a48356006f'

data/config/config.yml

@@ -1,7 +1,9 @@
 # This is the default configuration file. Enabling and disabling is configured
 # in separate files. This file adds all other parameters apart from Enabled.
 
-require: rubocop-rspec
+require:
+  - rubocop-rails
+  - rubocop-rspec
 
 # Common configuration.
 AllCops:
@@ -71,6 +73,3 @@ AllCops:
   # Else if .ruby-version exists and it contains an MRI version it is used.
   # Otherwise we fallback to the oldest officially supported Ruby version (2.1).
   # TargetRubyVersion: 2.5 # SPECIFY IN PROJECT .ruby-version file
-
-Rails:
-  Enabled: false

data/config/enabled.yml

@@ -6,18 +6,18 @@ Layout/AccessModifierIndentation:
   Description: Check indentation of private/protected visibility modifiers.
   StyleGuide: '#indent-public-private-protected'
 
-Layout/AlignArray:
+Layout/ArrayAlignment:
   Description: >-
                  Align the elements of an array literal if they span more than
                  one line.
   StyleGuide: '#align-multiline-arrays'
 
-Layout/AlignHash:
+Layout/HashAlignment:
   Description: >-
                  Align the elements of a hash literal if they span more than
                  one line.
 
-Layout/AlignParameters:
+Layout/ParameterAlignment:
   Description: Align the parameters of a method call if they span more than one line.
   StyleGuide: "#no-double-indent"
   EnforcedStyle: with_fixed_indentation
@@ -80,10 +80,6 @@ Style/CommentAnnotation:
                  (TODO, FIXME, OPTIMIZE, HACK, REVIEW).
   StyleGuide: '#annotate-keywords'
 
-Naming/ClassAndModuleCamelCase:
-  Description: 'Use CamelCase for classes and modules.'
-  StyleGuide: '#camelcase-classes'
-
 Style/ClassAndModuleChildren:
   Description: 'Checks style of children classes and modules.'
 
@@ -166,7 +162,7 @@ Style/HashSyntax:
                  { :a => 1, :b => 2 }.
   StyleGuide: '#hash-literals'
 
-Layout/IndentHash:
+Layout/FirstHashElementIndentation:
   Description: 'Checks the indentation of the first key in a hash literal.'
 
 Style/Lambda:
@@ -220,7 +216,7 @@ Layout/SpaceInsideParens:
 Layout/SpaceBeforeSemicolon:
   Description: 'No spaces before semicolons.'
 
-Layout/Tab:
+Layout/IndentationStyle:
   Description: 'No hard tabs.'
   StyleGuide: '#spaces-indentation'
 
@@ -228,7 +224,7 @@ Layout/TrailingWhitespace:
   Description: 'Avoid trailing whitespace.'
   StyleGuide: '#no-trailing-whitespace'
 
-Layout/TrailingBlankLines:
+Layout/TrailingEmptyLines:
   Description: 'Checks trailing blank lines and final newline.'
   StyleGuide: '#newline-eof'
 
@@ -249,14 +245,6 @@ Metrics/AbcSize:
   Enabled: true
   Severity: refactor
 
-######### PERFORMANCE #########
-
-Performance/Size:
-  Description: >-
-                  Use `size` instead of `count` for counting
-                  the number of elements in `Array` and `Hash`.
-  Reference: 'https://github.com/JuanitoFatas/fast-ruby#arraycount-vs-arraysize-code'
-
 ######### RAILS #########
 
 Rails/ActionFilter:
@@ -382,7 +370,7 @@ Lint/DuplicateCaseCondition:
 Lint/DuplicateMethods:
   Description: 'Check for duplicate method definitions.'
 
-Lint/DuplicatedKey:
+Lint/DuplicateHashKey:
   Description: 'Check for duplicate keys in hash literals.'
 
 Lint/EachWithObjectArgument:
@@ -406,7 +394,7 @@ Lint/EmptyWhen:
 Layout/EndAlignment:
   Description: 'Align ends correctly.'
 
-Lint/EndInMethod:
+Style/EndBlock:
   Description: 'END blocks should not be placed inside method definitions.'
 
 Lint/EnsureReturn:
@@ -421,7 +409,7 @@ Lint/FloatOutOfRange:
 Lint/FormatParameterMismatch:
   Description: 'The number of parameters to format/sprint must match the fields.'
 
-Lint/HandleExceptions:
+Lint/SuppressedException:
   Description: "Don't suppress exception."
   StyleGuide: '#dont-hide-exceptions'
 
@@ -500,7 +488,7 @@ Lint/ShadowingOuterLocalVariable:
                  Do not use the same name as outer local variable
                  for block arguments or block local variables.
 
-Lint/StringConversionInInterpolation:
+Lint/RedundantStringCoercion:
   Description: 'Checks for Object#to_s usage in string interpolation.'
   StyleGuide: '#no-to-s'
 
@@ -510,13 +498,13 @@ Lint/UnderscorePrefixedVariableName:
 Lint/UnifiedInteger:
   Description: 'Use Integer instead of Fixnum or Bignum'
 
-Lint/UnneededCopDisableDirective:
+Lint/RedundantCopDisableDirective:
   Description: >-
                  Checks for rubocop:disable comments that can be removed.
                  Note: this cop is not disabled when disabling all cops.
                  It must be explicitly disabled.
 
-Lint/UnneededSplatExpansion:
+Lint/RedundantSplatExpansion:
   Description: 'Checks for splat unnecessarily being called on literals'
 
 Lint/UnusedBlockArgument:
@@ -538,12 +526,9 @@ Lint/UselessAssignment:
   Description: 'Checks for useless assignment to a local variable.'
   StyleGuide: '#underscore-unused-vars'
 
-Lint/UselessComparison:
+Lint/BinaryOperatorWithIdenticalOperands:
   Description: 'Checks for comparison of something with itself.'
 
-Lint/UselessElseWithoutRescue:
-  Description: 'Checks for useless `else` in `begin..end` without `rescue`.'
-
 Lint/UselessSetterCall:
   Description: 'Checks for useless setter call to a local variable.'
 
@@ -567,14 +552,11 @@ Layout/FirstArrayElementLineBreak:
   Description: Checks for a line break before the first element in a multi-line array.
 
 Layout/IndentationConsistency:
-  Description: Keep indentation straight.
+  Description: This cop checks for inconsistent indentation.
   StyleGuide: "#spaces-indentation"
-  EnforcedStyle: rails
-
-Lint/Syntax:
-  Description: Checks syntax error
+  EnforcedStyle: indented_internal_methods
 
-Lint/UnneededRequireStatement:
+Lint/RedundantRequireStatement:
   Description: Checks for unnecessary `require` statement.
 
 Naming/AsciiIdentifiers:
@@ -618,7 +600,7 @@ Style/TrailingBodyOnModule:
 
 # RSpec
 
-FactoryBot/AttributeDefinedStatically:
+RSpec/FactoryBot/AttributeDefinedStatically:
   Description: Always declare attribute values as blocks.
   StyleGuide: http://www.rubydoc.info/gems/rubocop-rspec/RuboCop/Cop/RSpec/FactoryBot/AttributeDefinedStatically
 
@@ -644,10 +626,6 @@ RSpec/ImplicitExpect:
   EnforcedStyle: is_expected
   StyleGuide: http://www.rubydoc.info/gems/rubocop-rspec/RuboCop/Cop/RSpec/ImplicitExpect
 
-RSpec/InvalidPredicateMatcher:
-  Description: Checks invalid usage for predicate matcher.
-  StyleGuide: http://www.rubydoc.info/gems/rubocop-rspec/RuboCop/Cop/RSpec/InvalidPredicateMatcher
-
 RSpec/IteratedExpectation:
   Description: Check that `all` matcher is used instead of iterating over an array.
   StyleGuide: http://www.rubydoc.info/gems/rubocop-rspec/RuboCop/Cop/RSpec/IteratedExpectation
@@ -699,3 +677,13 @@ RSpec/ScatteredLet:
 
 RSpec/Focus:
   Enabled: true
+
+Style/AndOr:
+  Enabled: true
+  EnforcedStyle: always
+
+Style/Not:
+  Enabled: true
+
+Style/NegatedUnless:
+  Enabled: true

metadata

@@ -1,43 +1,57 @@
 --- !ruby/object:Gem::Specification
 name: arcadia_cops
 version: !ruby/object:Gem::Version
-  version: 3.5.3
+  version: 4.0.2
 platform: ruby
 authors:
-- justin
+- engineering
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2020-05-08 00:00:00.000000000 Z
+date: 2022-06-07 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rubocop
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - '='
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.63.1
+        version: 1.29.1
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - '='
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.63.1
+        version: 1.29.1
+- !ruby/object:Gem::Dependency
+  name: rubocop-rails
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '2.9'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '2.9'
 - !ruby/object:Gem::Dependency
   name: rubocop-rspec
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - '='
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.32'
+        version: '2.2'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - '='
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.32'
+        version: '2.2'
 - !ruby/object:Gem::Dependency
   name: bundler
   requirement: !ruby/object:Gem::Requirement
@@ -68,7 +82,7 @@ dependencies:
         version: '0'
 description: Contains enabled rubocops for arcadia power ruby repos.
 email:
-- justin@arcadiapower.com
+- engineering@arcadia.com
 executables: []
 extensions: []
 extra_rdoc_files: []
@@ -95,7 +109,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.1.2
+rubygems_version: 3.1.6
 signing_key: 
 specification_version: 4
 summary: Arcadia Power Style Cops