Update rubocop to 1.39.0 (#337)

Author: BrianHawley

Gemfile

@@ -5,7 +5,7 @@ source 'https://rubygems.org'
 gem "activesupport", require: false
 gem "parser"
 gem "pry", require: false
-gem "rubocop", "1.36.0", require: false
+gem "rubocop", "1.39.0", require: false
 gem "rubocop-i18n", require: false
 gem "rubocop-graphql", require: false
 gem "rubocop-minitest", require: false

Gemfile.lock

@@ -24,49 +24,49 @@ GEM
     rack (3.0.0)
     rainbow (3.1.1)
     rake (13.0.6)
-    regexp_parser (2.5.0)
+    regexp_parser (2.6.0)
     rexml (3.2.5)
-    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.1)
+    rspec (3.12.0)
+      rspec-core (~> 3.12.0)
+      rspec-expectations (~> 3.12.0)
+      rspec-mocks (~> 3.12.0)
+    rspec-core (3.12.0)
+      rspec-support (~> 3.12.0)
+    rspec-expectations (3.12.0)
       diff-lcs (>= 1.2.0, < 2.0)
-      rspec-support (~> 3.11.0)
-    rspec-mocks (3.11.1)
+      rspec-support (~> 3.12.0)
+    rspec-mocks (3.12.0)
       diff-lcs (>= 1.2.0, < 2.0)
-      rspec-support (~> 3.11.0)
-    rspec-support (3.11.1)
-    rubocop (1.36.0)
+      rspec-support (~> 3.12.0)
+    rspec-support (3.12.0)
+    rubocop (1.39.0)
       json (~> 2.3)
       parallel (~> 1.10)
       parser (>= 3.1.2.1)
       rainbow (>= 2.2.2, < 4.0)
       regexp_parser (>= 1.8, < 3.0)
       rexml (>= 3.2.5, < 4.0)
-      rubocop-ast (>= 1.20.1, < 2.0)
+      rubocop-ast (>= 1.23.0, < 2.0)
       ruby-progressbar (~> 1.7)
       unicode-display_width (>= 1.4.0, < 3.0)
-    rubocop-ast (1.21.0)
+    rubocop-ast (1.23.0)
       parser (>= 3.1.1.0)
-    rubocop-graphql (0.14.6)
+    rubocop-graphql (0.18.0)
       rubocop (>= 0.87, < 2)
     rubocop-i18n (3.0.0)
       rubocop (~> 1.0)
-    rubocop-minitest (0.22.1)
+    rubocop-minitest (0.23.2)
       rubocop (>= 0.90, < 2.0)
     rubocop-performance (1.15.0)
       rubocop (>= 1.7.0, < 2.0)
       rubocop-ast (>= 0.4.0)
-    rubocop-rails (2.16.1)
+    rubocop-rails (2.17.2)
       activesupport (>= 4.2.0)
       rack (>= 1.1)
       rubocop (>= 1.33.0, < 2.0)
     rubocop-rake (0.6.0)
       rubocop (~> 1.0)
-    rubocop-rspec (2.13.2)
+    rubocop-rspec (2.15.0)
       rubocop (~> 1.33)
     rubocop-sequel (0.3.4)
       rubocop (~> 1.0)
@@ -77,7 +77,7 @@ GEM
     rubocop-thread_safety (0.4.4)
       rubocop (>= 0.53.0)
     ruby-progressbar (1.11.0)
-    test-prof (1.0.10)
+    test-prof (1.0.11)
     tzinfo (2.0.5)
       concurrent-ruby (~> 1.0)
     unicode-display_width (2.3.0)
@@ -91,7 +91,7 @@ DEPENDENCIES
   pry
   rake
   rspec
-  rubocop (= 1.36.0)
+  rubocop (= 1.39.0)
   rubocop-graphql
   rubocop-i18n
   rubocop-minitest

config/contents/layout/space_inside_array_percent_literal.md

@@ -1,6 +1,9 @@
 Checks for unnecessary additional spaces inside array percent literals
 (i.e. %i/%w).
 
+Note that blank percent literals (e.g. `%i( )`) are checked by
+`Layout/SpaceInsidePercentLiteralDelimiters`.
+
 ### Example:
 
     # bad

config/contents/layout/space_inside_hash_literal_braces.md

@@ -41,10 +41,13 @@ surrounding space depending on configuration.
     # bad
     foo = { }
     bar = {    }
+    baz = {
+    }
 
     # good
     foo = {}
     bar = {}
+    baz = {}
 
 ### Example: EnforcedStyleForEmptyBraces: space
     # The `space` EnforcedStyleForEmptyBraces style enforces that
@@ -55,5 +58,6 @@ surrounding space depending on configuration.
 
     # good
     foo = { }
-    foo = {  }
-    foo = {     }
+    foo = {    }
+    foo = {
+    }

config/contents/layout/space_inside_percent_literal_delimiters.md

@@ -3,11 +3,28 @@ Checks for unnecessary additional spaces inside the delimiters of
 
 ### Example:
 
+    # bad
+    %i( foo bar baz )
+
     # good
     %i(foo bar baz)
 
     # bad
     %w( foo bar baz )
 
+    # good
+    %w(foo bar baz)
+
+    # bad
+    %x(  ls -l )
+
+    # good
+    %x(ls -l)
+
     # bad
-    %x(  ls -l )
+    %w( )
+    %w(
+    )
+
+    # good
+    %w()

config/contents/layout/space_inside_reference_brackets.md

@@ -33,6 +33,8 @@ surrounding space depending on configuration.
     # bad
     foo[ ]
     foo[     ]
+    foo[
+    ]
 
     # good
     foo[]
@@ -44,6 +46,8 @@ surrounding space depending on configuration.
     # bad
     foo[]
     foo[    ]
+    foo[
+    ]
 
     # good
     foo[ ]

config/contents/lint/ambiguous_block_association.md

@@ -40,7 +40,7 @@ By default, there are no methods to allowed.
     # bad
     expect { do_something }.to change { object.attribute }
 
-### Example: AllowedPatterns: [/change/]
+### Example: AllowedPatterns: ['change']
 
     # good
     expect { do_something }.to change { object.attribute }

config/contents/lint/duplicate_magic_comment.md

@@ -0,0 +1,21 @@
+Checks for duplicated magic comments.
+
+### Example:
+
+    # bad
+
+    # encoding: ascii
+    # encoding: ascii
+
+    # good
+
+    # encoding: ascii
+
+    # bad
+
+    # frozen_string_literal: true
+    # frozen_string_literal: true
+
+    # good
+
+    # frozen_string_literal: true

config/contents/lint/nested_method_definition.md

@@ -25,6 +25,9 @@ Checks for nested method definitions.
 
     # good
 
+    # `class_eval`, `instance_eval`, `module_eval`, `class_exec`, `instance_exec`, and
+    # `module_exec` blocks are allowed by default.
+
     def foo
       self.class.class_eval do
         def bar
@@ -48,4 +51,40 @@ Checks for nested method definitions.
         def bar
         end
       end
-    end
+    end
+
+### Example: AllowedMethods: [] (default)
+    # bad
+    def do_something
+      has_many :articles do
+        def find_or_create_by_name(name)
+        end
+      end
+    end
+
+### Example: AllowedMethods: ['has_many']
+    # bad
+    def do_something
+      has_many :articles do
+        def find_or_create_by_name(name)
+        end
+      end
+    end
+
+### Example: AllowedPatterns: [] (default)
+    # bad
+    def foo(obj)
+      obj.do_baz do
+        def bar
+        end
+      end
+    end
+
+### Example: AllowedPatterns: ['baz']
+    # good
+    def foo(obj)
+      obj.do_baz do
+        def bar
+        end
+      end
+    end

config/contents/lint/number_conversion.md

@@ -57,7 +57,7 @@ input if it is not a standard class.
     # bad
     10.minutes.to_i
 
-### Example: AllowedPatterns: [/min*/]
+### Example: AllowedPatterns: ['min*']
 
     # good
     10.minutes.to_i