rubomatic 1.0.0.pre.rc.2 → 1.0.0.pre.rc.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: a3ada84f77d72e9c013db6deb061bcda7442a403ec51444ad06f0e2a052b05be
-  data.tar.gz: 49c1269ecaaf1b1beaed4c285f0b2e607e7147e055aa9fa12d7febd1a21807e8
+  metadata.gz: a9262e25d628119d818abdc947c6de36fa78e28acd989c346e799fd16a49e70c
+  data.tar.gz: e63f62b52fb4b7feb81febe7d1bb965cf60413474fd7734ad37c0a19c3ca9c53
 SHA512:
-  metadata.gz: 49fec9a0244cd5f1c73e3ad14ea33acfab8a42a32f345acb58ea027b98ac30e5474b7b5292116615aa24db4e08451eda5eda39c504b01f14245ed4f4ca31cb18
-  data.tar.gz: eecfb7001751186cb06b73ee9be773db2fe6802090bafcc5fda8de05e985810456e9eb0c952d68bfaf9c775eebff05e6c3a63db522b4ab9ac0614fdff50d9a4b
+  metadata.gz: 8e88be2f9b8880c5b530f9b0a93c120fc92e328d0361eb015954b14be893aea8d351db1193a30f73bf31120e75dbb824e9bf63999f90564bfe2b255687985957
+  data.tar.gz: cddeb0f472f82cc66edd79e6cfcc98274bbbfdeb3840be65fe372454d9415821862c854e00d3d8f91c0633700782e13f19b2e75cda58aa1c6ce96358aee6d479

data/CHANGELOG.adoc

@@ -2,6 +2,13 @@
 
 * Initial public release
 
+=== Dependencies
+
+* `rubocop@1.45.0`
+* `rubocop-brands_insurance@1.1.0`
+* `rubocop-performance@1.16.0`
+* `rubocop-rails@2.18`
+
 == 0.0.1
 
 * Initial release

data/configs/lint.yml

@@ -956,10 +956,12 @@ Lint/UselessMethodDefinition:
   VersionChanged: '0.91'
   Safe: false
 
+  <% if RuboCop::Version.version.to_f >= 1.43 %>
 Lint/UselessRescue:
   Description: 'Checks for useless `rescue`s, which only reraise rescued exceptions.'
   Enabled: true
   VersionAdded: '1.43'
+  <% end %>
 
 Lint/UselessRuby2Keywords:
   Description: 'Finds unnecessary uses of `ruby2_keywords`.'

data/configs/performance.yml

@@ -2,127 +2,263 @@ 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}`.
+  SafeAutoCorrect: false
   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
+  SafeAutoCorrect: false
   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
+  SafeAutoCorrect: false
   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
+  SafeAutoCorrect: false
+  VersionAdded: '1.11'
 
 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'
 
 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
 
 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
+  SafeAutoCorrect: false
   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
+  SafeAutoCorrect: false
+  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.
+  SafeAutoCorrect: false
+  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
+  SafeAutoCorrect: false
+  VersionAdded: '0.36'
+  VersionChanged: '1.13'

data/configs/style.yml

@@ -62,10 +62,12 @@ Style/ArrayCoercion:
   Enabled: true
   VersionAdded: '0.88'
 
+  <% if RuboCop::Version.document_version.to_f >= 1.40 %>
 Style/ArrayIntersect:
   Description: 'Use `array1.intersect?(array2)` instead of `(array1 & array2).any?`.'
   Enabled: true
   VersionAdded: '1.40'
+  <% end %>
 
 Style/ArrayJoin:
   Description: 'Use Array#join instead of Array#*.'
@@ -445,16 +447,20 @@ Style/CommentedKeyword:
   VersionAdded: '0.51'
   VersionChanged: '1.19'
 
+  <% if RuboCop::Version.document_version.to_f >= 1.44 %>
 Style/ComparableClamp:
   Description: 'Enforces the use of `Comparable#clamp` instead of comparison by minimum and maximum.'
   Enabled: true
   VersionAdded: '1.44'
+  <% end %>
 
+  <% if RuboCop::Version.document_version.to_f >= 1.41 %>
 Style/ConcatArrayLiterals:
   Description: 'Enforces the use of `Array#push(item)` instead of `Array#concat([item])` to avoid redundant array literals.'
   Enabled: true
   Safe: false
   VersionAdded: '1.41'
+  <% end %>
 
 Style/ConditionalAssignment:
   Description: >-
@@ -534,11 +540,13 @@ Style/Dir:
   Enabled: true
   VersionAdded: '0.50'
 
-#Style/DirEmpty:
-#  Description: >-
-#    Prefer to use `Dir.empty?('path/to/dir')` when checking if a directory is empty.
-#  Enabled: true
-#  VersionAdded: '1.48'
+  <% if RuboCop::Version.document_version.to_f >= 1.48 %>
+Style/DirEmpty:
+  Description: >-
+    Prefer to use `Dir.empty?('path/to/dir')` when checking if a directory is empty.
+  Enabled: true
+  VersionAdded: '1.48'
+  <% end %>
 
 Style/DisableCopsWithinSourceCodeDirective:
   Severity: info # Or warning
@@ -737,11 +745,13 @@ Style/FetchEnvVar:
   # Environment variables to be excluded from the inspection.
   AllowedVars: [ ]
 
-#Style/FileEmpty:
-#  Description: >-
-#    Prefer to use `File.empty?('path/to/file')` when checking if a file is empty.
-#  Enabled: true
-#  VersionAdded: '1.48'
+  <% if RuboCop::Version.document_version.to_f >= 1.48 %>
+Style/FileEmpty:
+  Description: >-
+    Prefer to use `File.empty?('path/to/file')` when checking if a file is empty.
+  Enabled: true
+  VersionAdded: '1.48'
+  <% end %>
 
 Style/FileRead:
   Description: 'Favor `File.(bin)read` convenience methods.'
@@ -1065,6 +1075,7 @@ Style/InverseMethods:
     :select: :reject
     :select!: :reject!
 
+  <% if RuboCop::Version.document_version.to_f >= 1.44 %>
 Style/InvertibleUnlessCondition:
   Description: 'Favor `if` with inverted condition over `unless`.'
   Enabled: true
@@ -1091,6 +1102,7 @@ Style/InvertibleUnlessCondition:
     :exclude?: :include?
     # :one?: :many?
     # :many?: :one?
+  <% end %>
 
 Style/IpAddresses:
   Description: "Don't include literal IP addresses in code."
@@ -1763,6 +1775,7 @@ Style/RandomWithOffset:
   Enabled: true
   VersionAdded: '0.52'
 
+  <% if RuboCop::Version.document_version.to_f >= 1.40 %>
 Style/RedundantArgument:
   Description: 'Check for a redundant argument passed to certain methods.'
   Enabled: true
@@ -1776,6 +1789,7 @@ Style/RedundantArgument:
     split:
     chomp:
     chomp!:
+  <% end %>
 
 Style/RedundantAssignment:
   Description: 'Checks for redundant assignment before returning.'
@@ -1804,15 +1818,19 @@ Style/RedundantConditional:
   Enabled: true
   VersionAdded: '0.50'
 
+  <% if RuboCop::Version.document_version.to_f >= 1.40 %>
 Style/RedundantConstantBase:
   Description: Avoid redundant `::` prefix on constant.
   Enabled: true
   VersionAdded: '1.40'
+  <% end %>
 
+  <% if RuboCop::Version.document_version.to_f >= 1.41 %>
 Style/RedundantDoubleSplatHashBraces:
   Description: 'Checks for redundant uses of double splat hash braces.'
   Enabled: true
   VersionAdded: '1.41'
+  <% end %>
 
 Style/RedundantEach:
   Description: 'Checks for redundant `each`.'
@@ -1854,10 +1872,12 @@ Style/RedundantFreeze:
   VersionAdded: '0.34'
   VersionChanged: '0.66'
 
+  <% if RuboCop::Version.document_version.to_f >= 1.45 %>
 Style/RedundantHeredocDelimiterQuotes:
   Description: 'Checks for redundant heredoc delimiter quotes.'
   Enabled: true
   VersionAdded: '1.45'
+  <% end %>
 
 Style/RedundantInitialize:
   Description: 'Checks for redundant `initialize` methods.'
@@ -1959,11 +1979,13 @@ Style/RegexpLiteral:
   # are found in the regexp string.
   AllowInnerSlashes: false
 
+  <% if RuboCop::Version.document_version.to_f >= 1.40 %>
 Style/RequireOrder:
   Description: Sort `require` and `require_relative` in alphabetical order.
   Enabled: true
   SafeAutoCorrect: false
   VersionAdded: '1.40'
+  <% end %>
 
 Style/RescueModifier:
   Description: 'Avoid using rescue in its modifier form.'

data/lib/rubomatic/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Rubomatic
-  VERSION = '1.0.0-rc.2'
+  VERSION = '1.0.0-rc.3'
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: rubomatic
 version: !ruby/object:Gem::Version
-  version: 1.0.0.pre.rc.2
+  version: 1.0.0.pre.rc.3
 platform: ruby
 authors:
 - Brands Insurance