rubocop-rubomatic 1.3.0 → 1.4.0.pre.rc.2

data/config/metrics.yml

@@ -1,13 +1,5 @@
 Metrics/AbcSize:
-  Description: >-
-    A calculated magnitude based on number of assignments,
-    branches, and conditions.
-  Reference:
-    - http://c2.com/cgi/wiki?AbcMetric
-    - https://en.wikipedia.org/wiki/ABC_Software_Metric
   Enabled: true
-  VersionAdded: '0.27'
-  VersionChanged: '1.5'
   # The ABC size is a calculated magnitude, so this number can be an Integer or
   # a Float.
   AllowedMethods: [ ]
@@ -16,10 +8,7 @@ Metrics/AbcSize:
   Max: 250
 
 Metrics/BlockLength:
-  Description: 'Avoid long blocks with many lines.'
   Enabled: true
-  VersionAdded: '0.44'
-  VersionChanged: '1.5'
   CountComments: false  # count full line comments?
   Max: 500
   CountAsOne:
@@ -37,28 +26,19 @@ Metrics/BlockLength:
     - 'config/routes/*'
 
 Metrics/BlockNesting:
-  Description: 'Avoid excessive block nesting.'
-  StyleGuide: '#three-is-the-number-thou-shalt-count'
   Enabled: true
-  VersionAdded: '0.25'
-  VersionChanged: '0.47'
   CountBlocks: false
   Max: 10
 
 Metrics/ClassLength:
-  Description: 'Avoid classes longer than 100 lines of code.'
   Enabled: false
-  VersionAdded: '0.25'
-  VersionChanged: '0.87'
   CountComments: false  # count full line comments?
   Max: 100
   CountAsOne: [ ]
 
 <% if RuboCop::Version.document_version.to_f >= 1.47 %>
 Metrics/CollectionLiteralLength:
-  Description: Checks for `Array` or `Hash` literals with many entries.
   Enabled: true
-  VersionAdded: '1.47'
   LengthThreshold: 250
   Exclude:
     - lib/tasks/seeds
@@ -66,22 +46,13 @@ Metrics/CollectionLiteralLength:
 
 # Avoid complex methods.
 Metrics/CyclomaticComplexity:
-  Description: >-
-    A complexity metric that is strongly correlated to the number
-    of test cases needed to validate a method.
   Enabled: true
-  VersionAdded: '0.25'
-  VersionChanged: '0.81'
   AllowedMethods: [ ]
   AllowedPatterns: [ ]
   Max: 150
 
 Metrics/MethodLength:
-  Description: 'Avoid methods longer than 10 lines of code.'
-  StyleGuide: '#short-methods'
   Enabled: true
-  VersionAdded: '0.25'
-  VersionChanged: '1.5'
   CountComments: false  # count full line comments?
   Max: 350
   CountAsOne:
@@ -92,31 +63,19 @@ Metrics/MethodLength:
   AllowedPatterns: [ ]
 
 Metrics/ModuleLength:
-  Description: 'Avoid modules longer than 100 lines of code.'
   Enabled: false
-  VersionAdded: '0.31'
-  VersionChanged: '0.87'
   CountComments: false  # count full line comments?
   Max: 100
   CountAsOne: [ ]
 
 Metrics/ParameterLists:
-  Description: 'Avoid parameter lists longer than three or four parameters.'
-  StyleGuide: '#too-many-params'
   Enabled: true
-  VersionAdded: '0.25'
-  VersionChanged: '1.5'
   Max: 5
   CountKeywordArgs: false
   MaxOptionalParameters: 3
 
 Metrics/PerceivedComplexity:
-  Description: >-
-    A complexity metric geared towards measuring complexity for a
-    human reader.
   Enabled: true
-  VersionAdded: '0.25'
-  VersionChanged: '0.81'
   AllowedMethods: [ ]
   AllowedPatterns: [ ]
   Max: 75

data/config/migration.yml

@@ -1,6 +1,2 @@
 Migration/DepartmentName:
-  Description: >-
-    Check that cop names in rubocop:disable (etc) comments are
-    given with department name.
   Enabled: true
-  VersionAdded: '0.75'

data/config/naming.yml

@@ -1,29 +1,15 @@
 Naming/AccessorMethodName:
-  Description: Check the naming of accessor methods for get_/set_.
-  StyleGuide: '#accessor_mutator_method_names'
   Enabled: true
-  VersionAdded: '0.50'
 
 Naming/AsciiIdentifiers:
-  Description: 'Use only ascii symbols in identifiers and constants.'
-  StyleGuide: '#english-identifiers'
   Enabled: true
-  VersionAdded: '0.50'
-  VersionChanged: '0.87'
   AsciiConstants: true
 
 Naming/BinaryOperatorParameterName:
-  Description: 'When defining binary operators, name the argument other.'
-  StyleGuide: '#other-arg'
   Enabled: true
-  VersionAdded: '0.50'
-  VersionChanged: '1.2'
 
 Naming/BlockForwarding:
-  Description: 'Use anonymous block forwarding.'
-  StyleGuide: '#block-forwarding'
   Enabled: <%= ENV['RUBY_MAJOR'].to_f >= 3.1 %>
-  VersionAdded: '1.24'
   EnforcedStyle: anonymous
   SupportedStyles:
     - anonymous
@@ -31,12 +17,7 @@ Naming/BlockForwarding:
   BlockForwardingName: block
 
 Naming/BlockParameterName:
-  Description: >-
-    Checks for block parameter names that contain capital letters,
-    end in numbers, or do not meet a minimal length.
   Enabled: true
-  VersionAdded: '0.53'
-  VersionChanged: '0.77'
   # Parameter names may be equal to or greater than this value
   MinNameLength: 1
   AllowNamesEndingInNumbers: true
@@ -46,28 +27,17 @@ Naming/BlockParameterName:
   ForbiddenNames: [ ]
 
 Naming/ClassAndModuleCamelCase:
-  Description: 'Use CamelCase for classes and modules.'
-  StyleGuide: '#camelcase-classes'
   Enabled: true
-  VersionAdded: '0.50'
-  VersionChanged: '0.85'
   # Allowed class/module names can be specified here.
   # These can be full or part of the name.
   AllowedNames:
     - module_parent
 
 Naming/ConstantName:
-  Description: 'Constants should use SCREAMING_SNAKE_CASE.'
-  StyleGuide: '#screaming-snake-case'
   Enabled: true
-  VersionAdded: '0.50'
 
 Naming/FileName:
-  Description: 'Use snake_case for source file names.'
-  StyleGuide: '#snake-case-files'
   Enabled: true
-  VersionAdded: '0.50'
-  VersionChanged: '1.23'
   # Camel case file names listed in `AllCops:Include` and all file names listed
   # in `AllCops:Exclude` are excluded by default. Add extra excludes here.
   Exclude: [ ]
@@ -139,29 +109,19 @@ Naming/FileName:
     - XSS
 
 Naming/HeredocDelimiterCase:
-  Description: 'Use configured case for heredoc delimiters.'
-  StyleGuide: '#heredoc-delimiters'
   Enabled: true
-  VersionAdded: '0.50'
-  VersionChanged: '1.2'
   EnforcedStyle: uppercase
   SupportedStyles:
     - lowercase
     - uppercase
 
 Naming/HeredocDelimiterNaming:
-  Description: 'Use descriptive heredoc delimiters.'
-  StyleGuide: '#heredoc-delimiters'
   Enabled: true
-  VersionAdded: '0.50'
   ForbiddenDelimiters:
     - !ruby/regexp '/(^|\s)(EO[A-Z]{1}|END)(\s|$)/'
 
 Naming/InclusiveLanguage:
-  Description: 'Recommend the use of inclusive language instead of problematic terms.'
   Enabled: false
-  VersionAdded: '1.18'
-  VersionChanged: '1.21'
   CheckIdentifiers: true
   CheckConstants: true
   CheckVariables: true
@@ -185,23 +145,15 @@ Naming/InclusiveLanguage:
       Suggestions: [ 'replica', 'secondary', 'follower' ]
 
 Naming/MemoizedInstanceVariableName:
-  Description: >-
-    Memoized method name should match memo instance variable name.
   Enabled: true
-  VersionAdded: '0.53'
-  VersionChanged: '1.2'
   EnforcedStyleForLeadingUnderscores: disallowed
   SupportedStylesForLeadingUnderscores:
     - disallowed
     - required
     - optional
-  Safe: false
 
 Naming/MethodName:
-  Description: 'Use the configured style when naming methods.'
-  StyleGuide: '#snake-case-symbols-methods-vars'
   Enabled: true
-  VersionAdded: '0.50'
   EnforcedStyle: snake_case
   SupportedStyles:
     - snake_case
@@ -215,12 +167,7 @@ Naming/MethodName:
   AllowedPatterns: [ ]
 
 Naming/MethodParameterName:
-  Description: >-
-    Checks for method parameter names that contain capital letters,
-    end in numbers, or do not meet a minimal length.
   Enabled: true
-  VersionAdded: '0.53'
-  VersionChanged: '0.77'
   # Parameter names may be equal to or greater than this value
   MinNameLength: 1
   AllowNamesEndingInNumbers: true
@@ -245,11 +192,7 @@ Naming/MethodParameterName:
   ForbiddenNames: [ ]
 
 Naming/PredicateName:
-  Description: 'Check the names of predicate methods.'
-  StyleGuide: '#bool-methods-qmark'
   Enabled: true
-  VersionAdded: '0.50'
-  VersionChanged: '0.77'
   # Predicate name prefixes.
   NamePrefix:
     - is_
@@ -274,18 +217,11 @@ Naming/PredicateName:
     - 'spec/**/*'
 
 Naming/RescuedExceptionsVariableName:
-  Description: 'Use consistent rescued exceptions variables naming.'
   Enabled: true
-  VersionAdded: '0.67'
-  VersionChanged: '0.68'
   PreferredName: e
 
 Naming/VariableName:
-  Description: 'Use the configured style when naming variables.'
-  StyleGuide: '#snake-case-symbols-methods-vars'
   Enabled: true
-  VersionAdded: '0.50'
-  VersionChanged: '1.8'
   EnforcedStyle: snake_case
   SupportedStyles:
     - snake_case
@@ -294,11 +230,7 @@ Naming/VariableName:
   AllowedPatterns: [ ]
 
 Naming/VariableNumber:
-  Description: 'Use the configured style when numbering symbols, methods and variables.'
-  StyleGuide: '#snake-case-symbols-methods-vars-with-numbers'
   Enabled: true
-  VersionAdded: '0.50'
-  VersionChanged: '1.4'
   EnforcedStyle: normalcase
   SupportedStyles:
     - snake_case

data/config/performance.yml

@@ -4,258 +4,133 @@ Performance:
   Enabled: false
 
 Performance/BigDecimalWithNumericArgument:
-  Description: Convert numeric literal to string and pass it to `BigDecimal`.
   Enabled: true
-  VersionAdded: '1.7'
 
 Performance/ChainArrayAllocation:
-  Description: Instead of chaining array methods that allocate new arrays, mutate an
-    existing array.
-  Reference: https://twitter.com/schneems/status/1034123879978029057
   Enabled: true
-  VersionAdded: '0.59'
 
 Performance/CollectionLiteralInLoop:
-  Description: Extract Array and Hash literals outside of loops into local variables
-    or constants.
   Enabled: true
-  VersionAdded: '1.8'
   MinSize: 1
 
 Performance/CompareWithBlock:
-  Description: Use `sort_by(&:foo)` instead of `sort { |a, b| a.foo <=> b.foo }`.
   Enabled: true
-  VersionAdded: '0.46'
 
 Performance/ConstantRegexp:
-  Description: Finds regular expressions with dynamic components that are all constants.
   Enabled: true
-  VersionAdded: '1.9'
-  VersionChanged: '1.10'
 
 Performance/Count:
-  Description: Use `count` instead of `{select,find_all,filter,reject}...{size,count,length}`.
   Enabled: true
-  VersionAdded: '0.31'
-  VersionChanged: '1.8'
 
 Performance/DeletePrefix:
-  Description: Use `delete_prefix` instead of `gsub`.
   Enabled: true
-  Safe: false
   SafeMultiline: false
-  VersionAdded: '1.6'
-  VersionChanged: '1.11'
 
 Performance/DeleteSuffix:
-  Description: Use `delete_suffix` instead of `gsub`.
   Enabled: true
-  Safe: false
   SafeMultiline: false
-  VersionAdded: '1.6'
-  VersionChanged: '1.11'
 
 Performance/Detect:
-  Description: Use `detect` instead of `select.first`, `find_all.first`, `filter.first`,
-    `select.last`, `find_all.last`, and `filter.last`.
-  Reference: https://github.com/JuanitoFatas/fast-ruby#enumerabledetect-vs-enumerableselectfirst-code
   Enabled: true
-  VersionAdded: '0.30'
-  VersionChanged: '1.8'
 
 Performance/DoubleStartEndWith:
-  Description: Use `str.{start,end}_with?(x, ..., y, ...)` instead of `str.{start,end}_with?(x,
-    ...) || str.{start,end}_with?(y, ...)`.
   Enabled: true
-  VersionAdded: '0.36'
-  VersionChanged: '0.48'
   IncludeActiveSupportAliases: true
 
 Performance/EndWith:
-  Description: Use `end_with?` instead of a regex match anchored to the end of a string.
-  Reference: https://github.com/JuanitoFatas/fast-ruby#stringmatch-vs-stringstart_withstringend_with-code-start-code-end
   Enabled: true
   SafeMultiline: false
-  VersionAdded: '0.36'
-  VersionChanged: '1.10'
 
 Performance/FixedSize:
-  Description: Do not compute the size of statically sized objects except in constants.
   Enabled: true
-  VersionAdded: '0.35'
 
 Performance/FlatMap:
-  Description: Use `Enumerable#flat_map` instead of `Enumerable#map...Array#flatten(1)`
-    or `Enumerable#collect..Array#flatten(1)`.
-  Reference: https://github.com/JuanitoFatas/fast-ruby#enumerablemaparrayflatten-vs-enumerableflat_map-code
   Enabled: true
-  VersionAdded: '0.30'
   EnabledForFlattenWithoutParams: true
 
 Performance/InefficientHashSearch:
-  Description: Use `key?` or `value?` instead of `keys.include?` or `values.include?`.
-  Reference: https://github.com/JuanitoFatas/fast-ruby#hashkey-instead-of-hashkeysinclude-code
   Enabled: true
-  VersionAdded: '0.56'
-  Safe: false
 
 Performance/IoReadlines:
-  Description: Use `IO.each_line` (`IO#each_line`) instead of `IO.readlines` (`IO#readlines`).
-  Reference: https://docs.gitlab.com/ee/development/performance.html#reading-from-files-and-other-data-sources
   Enabled: true
-  VersionAdded: '1.7'
 
 Performance/MapCompact:
-  Description: Use `filter_map` instead of `collection.map(&:do_something).compact`.
   Enabled: true
-  VersionAdded: '1.11'
+
+Performance/MapMethodChain:
+  Enabled: true
 
 Performance/MethodObjectAsBlock:
-  Description: Use block explicitly instead of block-passing a method object.
-  Reference: https://github.com/JuanitoFatas/fast-ruby#normal-way-to-apply-method-vs-method-code
   Enabled: true
-  VersionAdded: '1.9'
 
 Performance/OpenStruct:
-  Description: Use `Struct` instead of `OpenStruct`.
   Enabled: true
-  VersionAdded: '0.61'
-  Safe: false
 
 Performance/RangeInclude:
-  Description: Use `Range#cover?` instead of `Range#include?` (or `Range#member?`).
-  Reference: https://github.com/JuanitoFatas/fast-ruby#cover-vs-include-code
   Enabled: true
-  VersionAdded: '0.36'
-  VersionChanged: '1.7'
-  Safe: false
 
 Performance/RedundantBlockCall:
-  Description: Use `yield` instead of `block.call`.
-  Reference: https://github.com/JuanitoFatas/fast-ruby#proccall-and-block-arguments-vs-yieldcode
   Enabled: true
-  VersionAdded: '0.36'
 
 Performance/RedundantEqualityComparisonBlock:
-  Description: Checks for uses `Enumerable#all?`, `Enumerable#any?`, `Enumerable#one?`,
-    or `Enumerable#none?` are compared with `===` or similar methods in block.
-  Reference: https://github.com/rails/rails/pull/41363
   Enabled: true
-  Safe: false
-  VersionAdded: '1.10'
   AllowRegexpMatch: true
 
 Performance/RedundantMatch:
-  Description: Use `=~` instead of `String#match` or `Regexp#match` in a context where
-    the returned `MatchData` is not needed.
   Enabled: true
-  VersionAdded: '0.36'
 
 Performance/RedundantMerge:
-  Description: Use Hash#[]=, rather than Hash#merge! with a single key-value pair.
-  Reference: https://github.com/JuanitoFatas/fast-ruby#hashmerge-vs-hash-code
   Enabled: true
-  Safe: false
-  VersionAdded: '0.36'
-  VersionChanged: '1.11'
   MaxKeyValuePairs: 2
   Exclude:
     - './**/*.json.jbuilder'
 
 Performance/RedundantSortBlock:
-  Description: Use `sort` instead of `sort { |a, b| a <=> b }`.
   Enabled: true
-  VersionAdded: '1.7'
 
 Performance/RedundantSplitRegexpArgument:
-  Description: Identifies places where `split` argument can be replaced from a deterministic
-    regexp to a string.
   Enabled: true
-  VersionAdded: '1.10'
 
 Performance/RedundantStringChars:
-  Description: Checks for redundant `String#chars`.
   Enabled: true
-  VersionAdded: '1.7'
 
 Performance/RegexpMatch:
-  Description: Use `match?` instead of `Regexp#match`, `String#match`, `Symbol#match`,
-    `Regexp#===`, or `=~` when `MatchData` is not used.
-  Reference: https://github.com/JuanitoFatas/fast-ruby#regexp-vs-stringmatch-vs-string-vs-stringmatch-code-
   Enabled: true
-  VersionAdded: '0.47'
 
 Performance/ReverseEach:
-  Description: Use `reverse_each` instead of `reverse.each`.
-  Reference: https://github.com/JuanitoFatas/fast-ruby#enumerablereverseeach-vs-enumerablereverse_each-code
   Enabled: true
-  VersionAdded: '0.30'
 
 Performance/ReverseFirst:
-  Description: Use `last(n).reverse` instead of `reverse.first(n)`.
   Enabled: true
-  VersionAdded: '1.7'
 
 Performance/SelectMap:
-  Description: Use `filter_map` instead of `ary.select(&:foo).map(&:bar)`.
   Enabled: true
-  VersionAdded: '1.11'
 
 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#arraylength-vs-arraysize-vs-arraycount-code
   Enabled: true
-  VersionAdded: '0.30'
 
 Performance/SortReverse:
-  Description: Use `sort.reverse` instead of `sort { |a, b| b <=> a }`.
   Enabled: true
-  VersionAdded: '1.7'
 
 Performance/Squeeze:
-  Description: Use `squeeze('a')` instead of `gsub(/a+/, 'a')`.
-  Reference: https://github.com/JuanitoFatas/fast-ruby#remove-extra-spaces-or-other-contiguous-characters-code
   Enabled: true
-  VersionAdded: '1.7'
 
 Performance/StartWith:
-  Description: Use `start_with?` instead of a regex match anchored to the beginning
-    of a string.
-  Reference: https://github.com/JuanitoFatas/fast-ruby#stringmatch-vs-stringstart_withstringend_with-code-start-code-end
   Enabled: true
   SafeMultiline: false
-  VersionAdded: '0.36'
-  VersionChanged: '1.10'
 
 Performance/StringIdentifierArgument:
-  Description: Use symbol identifier argument instead of string identifier argument.
   Enabled: true
-  VersionAdded: '1.13'
 
 Performance/StringInclude:
-  Description: Use `String#include?` instead of a regex match with literal-only pattern.
   Enabled: true
-  VersionAdded: '1.7'
-  VersionChanged: '1.12'
 
 Performance/StringReplacement:
-  Description: Use `tr` instead of `gsub` when you are replacing the same number of
-    characters. Use `delete` instead of `gsub` when you are deleting characters.
-  Reference: https://github.com/JuanitoFatas/fast-ruby#stringgsub-vs-stringtr-code
   Enabled: true
-  VersionAdded: '0.33'
 
 Performance/Sum:
-  Description: Use `sum` instead of a custom array summation.
-  Reference: https://blog.bigbinary.com/2016/11/02/ruby-2-4-introduces-enumerable-sum.html
   Enabled: true
-  VersionAdded: '1.8'
-  VersionChanged: '1.13'
   OnlySumOrWithInitialValue: false
 
 Performance/TimesMap:
-  Description: Checks for .times.map calls.
   Enabled: true
-  VersionAdded: '0.36'
-  VersionChanged: '1.13'

data/config/security.yml

@@ -1,51 +1,20 @@
 Security/CompoundHash:
-  Description: 'When overwriting Object#hash to combine values, prefer delegating to Array#hash over writing a custom implementation.'
   Enabled: true
-  VersionAdded: '1.28'
 
 Security/Eval:
-  Description: 'The use of eval represents a serious security risk.'
   Enabled: true
-  VersionAdded: '0.47'
 
 Security/IoMethods:
-  Description: >-
-    Checks for the first argument to `IO.read`, `IO.binread`, `IO.write`, `IO.binwrite`,
-    `IO.foreach`, and `IO.readlines`.
   Enabled: true
-  Safe: false
-  VersionAdded: '1.22'
 
 Security/JSONLoad:
-  Description: >-
-    Prefer usage of `JSON.parse` over `JSON.load` due to potential
-    security issues. See reference for more information.
-  Reference: 'https://ruby-doc.org/stdlib-2.7.0/libdoc/json/rdoc/JSON.html#method-i-load'
   Enabled: true
-  VersionAdded: '0.43'
-  VersionChanged: '1.22'
-  # Autocorrect here will change to a method that may cause crashes depending
-  # on the value of the argument.
 
 Security/MarshalLoad:
-  Description: >-
-    Avoid using of `Marshal.load` or `Marshal.restore` due to potential
-    security issues. See reference for more information.
-  Reference: 'https://ruby-doc.org/core-2.7.0/Marshal.html#module-Marshal-label-Security+considerations'
   Enabled: true
-  VersionAdded: '0.47'
 
 Security/Open:
-  Description: 'The use of `Kernel#open` and `URI.open` represent a serious security risk.'
   Enabled: true
-  VersionAdded: '0.53'
-  VersionChanged: '1.0'
-  Safe: false
 
 Security/YAMLLoad:
-  Description: >-
-    Prefer usage of `YAML.safe_load` over `YAML.load` due to potential
-    security issues. See reference for more information.
-  Reference: 'https://ruby-doc.org/stdlib-2.7.0/libdoc/yaml/rdoc/YAML.html#module-YAML-label-Security'
   Enabled: true
-  VersionAdded: '0.47'