Update rubocop to 1.28.1 (#313)

- General bundle update.
- Update the docs too.

Author: BrianHawley

Gemfile

@@ -6,7 +6,7 @@ gem "activesupport", require: false
 gem "mry", require: false
 gem "parser"
 gem "pry", require: false
-gem "rubocop", "1.25.1", require: false
+gem "rubocop", "1.28.1", require: false
 gem "rubocop-i18n", require: false
 gem "rubocop-graphql", require: false
 gem "rubocop-minitest", require: false

Gemfile.lock

@@ -1,84 +1,84 @@
 GEM
   remote: https://rubygems.org/
   specs:
-    activesupport (7.0.1)
+    activesupport (7.0.2.3)
       concurrent-ruby (~> 1.0, >= 1.0.2)
       i18n (>= 1.6, < 2)
       minitest (>= 5.1)
       tzinfo (~> 2.0)
     ast (2.4.2)
     coderay (1.1.3)
-    concurrent-ruby (1.1.9)
+    concurrent-ruby (1.1.10)
     diff-lcs (1.5.0)
-    i18n (1.9.1)
+    i18n (1.10.0)
       concurrent-ruby (~> 1.0)
     method_source (1.0.0)
     minitest (5.15.0)
     mry (0.78.0.0)
       rubocop (>= 0.41.0)
-    parallel (1.21.0)
-    parser (3.1.0.0)
+    parallel (1.22.1)
+    parser (3.1.2.0)
       ast (~> 2.4.1)
     pry (0.14.1)
       coderay (~> 1.1)
       method_source (~> 1.0)
     rack (2.2.3)
     rainbow (3.1.1)
     rake (13.0.6)
-    regexp_parser (2.2.0)
+    regexp_parser (2.3.0)
     rexml (3.2.5)
-    rspec (3.10.0)
-      rspec-core (~> 3.10.0)
-      rspec-expectations (~> 3.10.0)
-      rspec-mocks (~> 3.10.0)
-    rspec-core (3.10.2)
-      rspec-support (~> 3.10.0)
-    rspec-expectations (3.10.2)
+    rspec (3.11.0)
+      rspec-core (~> 3.11.0)
+      rspec-expectations (~> 3.11.0)
+      rspec-mocks (~> 3.11.0)
+    rspec-core (3.11.0)
+      rspec-support (~> 3.11.0)
+    rspec-expectations (3.11.0)
       diff-lcs (>= 1.2.0, < 2.0)
-      rspec-support (~> 3.10.0)
-    rspec-mocks (3.10.3)
+      rspec-support (~> 3.11.0)
+    rspec-mocks (3.11.1)
       diff-lcs (>= 1.2.0, < 2.0)
-      rspec-support (~> 3.10.0)
-    rspec-support (3.10.3)
-    rubocop (1.25.1)
+      rspec-support (~> 3.11.0)
+    rspec-support (3.11.0)
+    rubocop (1.28.1)
       parallel (~> 1.10)
       parser (>= 3.1.0.0)
       rainbow (>= 2.2.2, < 4.0)
       regexp_parser (>= 1.8, < 3.0)
       rexml
-      rubocop-ast (>= 1.15.1, < 2.0)
+      rubocop-ast (>= 1.17.0, < 2.0)
       ruby-progressbar (~> 1.7)
       unicode-display_width (>= 1.4.0, < 3.0)
-    rubocop-ast (1.15.1)
-      parser (>= 3.0.1.1)
-    rubocop-graphql (0.12.3)
+    rubocop-ast (1.17.0)
+      parser (>= 3.1.1.0)
+    rubocop-graphql (0.14.2)
       rubocop (>= 0.87, < 2)
     rubocop-i18n (3.0.0)
       rubocop (~> 1.0)
-    rubocop-minitest (0.17.1)
+    rubocop-minitest (0.19.1)
       rubocop (>= 0.90, < 2.0)
-    rubocop-performance (1.13.2)
+    rubocop-performance (1.13.3)
       rubocop (>= 1.7.0, < 2.0)
       rubocop-ast (>= 0.4.0)
-    rubocop-rails (2.13.2)
+    rubocop-rails (2.14.2)
       activesupport (>= 4.2.0)
       rack (>= 1.1)
       rubocop (>= 1.7.0, < 2.0)
     rubocop-rake (0.6.0)
       rubocop (~> 1.0)
-    rubocop-rspec (2.8.0)
+    rubocop-rspec (2.10.0)
       rubocop (~> 1.19)
     rubocop-sequel (0.3.3)
       rubocop (~> 1.0)
-    rubocop-shopify (2.4.0)
-      rubocop (~> 1.24)
-    rubocop-sorbet (0.6.5)
+    rubocop-shopify (2.5.0)
+      rubocop (~> 1.25)
+    rubocop-sorbet (0.6.7)
       rubocop (>= 0.90.0)
     rubocop-thread_safety (0.4.4)
       rubocop (>= 0.53.0)
     ruby-progressbar (1.11.0)
     safe_yaml (1.0.5)
-    test-prof (1.0.7)
+    test-prof (1.0.8)
     tzinfo (2.0.4)
       concurrent-ruby (~> 1.0)
     unicode-display_width (2.1.0)
@@ -93,7 +93,7 @@ DEPENDENCIES
   pry
   rake
   rspec
-  rubocop (= 1.25.1)
+  rubocop (= 1.28.1)
   rubocop-graphql
   rubocop-i18n
   rubocop-minitest

config/contents/gemspec/date_assignment.md

@@ -5,11 +5,11 @@ It is set automatically when the gem is packaged.
 
     # bad
     Gem::Specification.new do |spec|
-      s.name = 'your_cool_gem_name'
+      spec.name = 'your_cool_gem_name'
       spec.date = Time.now.strftime('%Y-%m-%d')
     end
 
     # good
     Gem::Specification.new do |spec|
-      s.name = 'your_cool_gem_name'
+      spec.name = 'your_cool_gem_name'
     end

config/contents/gemspec/require_mfa.md

@@ -1,8 +1,9 @@
 Requires a gemspec to have `rubygems_mfa_required` metadata set.
 
-This setting tells RubyGems that MFA is required for accounts to
-be able perform privileged operations, such as (see
-RubyGems' documentation for the full list of privileged operations):
+This setting tells RubyGems that MFA (Multi-Factor Authentication) is
+required for accounts to be able perform privileged operations, such as
+(see RubyGems' documentation for the full list of privileged
+operations):
 
 * `gem push`
 * `gem yank`

config/contents/layout/case_indentation.md

@@ -1,4 +1,4 @@
-This cop checks how the `when` and `in`s of a `case` expression
+This cop checks how the `when` and ``in``s of a `case` expression
 are indented in relation to its `case` or `end` keyword.
 
 It will register a separate offense for each misaligned `when` and `in`.

config/contents/layout/indentation_width.md

@@ -19,7 +19,7 @@ one.
       end
     end
 
-### Example: IgnoredPatterns: ['^\s*module']
+### Example: AllowedPatterns: ['^\s*module']
     # bad
     module A
     class B

config/contents/layout/multiline_operation_indentation.md

@@ -1,24 +1,23 @@
-This cop checks the indentation of the right hand side operand in
-binary operations that span more than one line.
+This cop checks the indentation of the right hand side operand in binary operations that
+span more than one line.
 
-The `aligned` style checks that operators are aligned if they are part
-of an `if` or `while` condition, a `return` statement, etc. In other
-contexts, the second operand should be indented regardless of enforced
-style.
+The `aligned` style checks that operators are aligned if they are part of an `if` or `while`
+condition, an explicit `return` statement, etc. In other contexts, the second operand should
+be indented regardless of enforced style.
 
 ### Example: EnforcedStyle: aligned (default)
     # bad
     if a +
         b
       something &&
-        something_else
+      something_else
     end
 
     # good
     if a +
        b
       something &&
-      something_else
+        something_else
     end
 
 ### Example: EnforcedStyle: indented

config/contents/lint/duplicate_require.md

@@ -1,5 +1,10 @@
 This cop checks for duplicate `require`s and `require_relative`s.
 
+### Safety:
+
+This cop's autocorrection is unsafe because it may break the dependency order
+of `require`.
+
 ### Example:
     # bad
     require 'foo'

config/contents/lint/incompatible_io_select_with_fiber_scheduler.md

@@ -1,6 +1,16 @@
 
 This cop checks for `IO.select` that is incompatible with Fiber Scheduler since Ruby 3.0.
 
+NOTE: When the method is successful the return value of `IO.select` is `[[IO]]`,
+and the return value of `io.wait_readable` and `io.wait_writable` are `self`.
+They are not auto-corrected when assigning a return value because these types are different.
+It's up to user how to handle the return value.
+
+### Safety:
+
+This cop's autocorrection is unsafe because `NoMethodError` occurs
+if `require 'io/wait'` is not called.
+
 ### Example:
 
     # bad
@@ -14,8 +24,3 @@ This cop checks for `IO.select` that is incompatible with Fiber Scheduler since
 
     # good
     io.wait_writable(timeout)
-
-### Safety:
-
-This cop's autocorrection is unsafe because `NoMethodError` occurs
-if `require 'io/wait'` is not called.

config/contents/lint/inherit_exception.md

@@ -1,9 +1,14 @@
-This cop looks for error classes inheriting from `Exception`
-and its standard library subclasses, excluding subclasses of
-`StandardError`. It is configurable to suggest using either
-`RuntimeError` (default) or `StandardError` instead.
+This cop looks for error classes inheriting from `Exception`.
+It is configurable to suggest using either `StandardError` (default) or
+`RuntimeError` instead.
 
-### Example: EnforcedStyle: runtime_error (default)
+### Safety:
+
+This cop's autocorrection is unsafe because `rescue` that omit
+exception class handle `StandardError` and its subclasses,
+but not `Exception` and its subclasses.
+
+### Example: EnforcedStyle: standard_error (default)
     # bad
 
     class C < Exception; end
@@ -12,11 +17,11 @@ and its standard library subclasses, excluding subclasses of
 
     # good
 
-    class C < RuntimeError; end
+    class C < StandardError; end
 
-    C = Class.new(RuntimeError)
+    C = Class.new(StandardError)
 
-### Example: EnforcedStyle: standard_error
+### Example: EnforcedStyle: runtime_error
     # bad
 
     class C < Exception; end
@@ -25,6 +30,6 @@ and its standard library subclasses, excluding subclasses of
 
     # good
 
-    class C < StandardError; end
+    class C < RuntimeError; end
 
-    C = Class.new(StandardError)
+    C = Class.new(RuntimeError)