Point rubocop to ruby 3.1

Author: rafaelfranca

.rubocop.yml

@@ -6,7 +6,7 @@ require:
   - rubocop-md
 
 AllCops:
-  TargetRubyVersion: 2.7
+  TargetRubyVersion: 3.1
   # RuboCop has a bunch of cops enabled by default. This setting tells RuboCop
   # to ignore them, so only the ones explicitly set in this file are enabled.
   DisabledByDefault: true
@@ -107,6 +107,7 @@ Layout/EmptyLinesAroundModuleBody:
 # Use Ruby >= 1.9 syntax for hashes. Prefer { a: :b } over { :a => :b }.
 Style/HashSyntax:
   Enabled: true
+  EnforcedShorthandSyntax: either
 
 # Method definitions after `private` or `protected` isolated calls need one
 # extra level of indentation.
@@ -238,10 +239,19 @@ Lint/ErbNewArguments:
 Lint/EnsureReturn:
   Enabled: true
 
+Lint/MissingCopEnableDirective:
+  Enabled: true
+
 # Use my_method(my_arg) not my_method( my_arg ) or my_method my_arg.
 Lint/RequireParentheses:
   Enabled: true
 
+Lint/RedundantCopDisableDirective:
+  Enabled: true
+
+Lint/RedundantCopEnableDirective:
+  Enabled: true
+
 Lint/RedundantStringCoercion:
   Enabled: true
 

actionpack/lib/action_controller/metal/redirecting.rb

@@ -9,7 +9,7 @@ module Redirecting
 
     class UnsafeRedirectError < StandardError; end
 
-    ILLEGAL_HEADER_VALUE_REGEX = /[\x00-\x08\x0A-\x1F]/.freeze
+    ILLEGAL_HEADER_VALUE_REGEX = /[\x00-\x08\x0A-\x1F]/
 
     included do
       mattr_accessor :raise_on_open_redirects, default: false

actionpack/lib/action_dispatch/journey/router/utils.rb

@@ -42,11 +42,11 @@ class UriEncoder # :nodoc:
           UNRESERVED = "#{ALPHA}#{DIGIT}\\-\\._~"
           SUB_DELIMS = "!\\$&'\\(\\)\\*\\+,;="
 
-          ESCAPED  = /%[a-zA-Z0-9]{2}/.freeze
+          ESCAPED  = /%[a-zA-Z0-9]{2}/
 
-          FRAGMENT = /[^#{UNRESERVED}#{SUB_DELIMS}:@\/?]/.freeze
-          SEGMENT  = /[^#{UNRESERVED}#{SUB_DELIMS}:@]/.freeze
-          PATH     = /[^#{UNRESERVED}#{SUB_DELIMS}:@\/]/.freeze
+          FRAGMENT = /[^#{UNRESERVED}#{SUB_DELIMS}:@\/?]/
+          SEGMENT  = /[^#{UNRESERVED}#{SUB_DELIMS}:@]/
+          PATH     = /[^#{UNRESERVED}#{SUB_DELIMS}:@\/]/
 
           def escape_fragment(fragment)
             escape(fragment, FRAGMENT)

actionpack/test/dispatch/test_response_test.rb

@@ -43,16 +43,12 @@ def assert_response_code_range(range, predicate)
   test "JSON response Hash pattern matching" do
     response = ActionDispatch::TestResponse.create(200, { "Content-Type" => "application/json" }, '{ "foo": "fighters" }')
 
-    # rubocop:disable Lint/Syntax
     assert_pattern { response.parsed_body => { foo: /fighter/ } }
-    # rubocop:enable Lint/Syntax
   end
 
   test "JSON response Array pattern matching" do
     response = ActionDispatch::TestResponse.create(200, { "Content-Type" => "application/json" }, '[{ "foo": "fighters" }, { "nir": "vana" }]')
-    # rubocop:disable Lint/Syntax
     assert_pattern { response.parsed_body => [{ foo: /fighter/ }, { nir: /vana/ }] }
-    # rubocop:enable Lint/Syntax
   end
 
   test "HTML response pattern matching" do
@@ -66,17 +62,12 @@ def assert_response_code_range(range, predicate)
     HTML
     html = response.parsed_body
 
-    # rubocop:disable Lint/Syntax
     html.at("main") => {name:, content:}
-    # rubocop:enable Lint/Syntax
-
     assert_equal "main", name
     assert_equal "Some main content", content
 
-    # rubocop:disable Lint/Syntax
     assert_pattern { html.at("main") => { content: "Some main content" } }
     assert_pattern { html.at("main") => { content: /content/ } }
     assert_pattern { html.at("main") => { children: [{ name: "h1", content: /content/ }] } }
-    # rubocop:enable Lint/Syntax
   end
 end

actionview/test/template/test_case_test.rb

@@ -460,7 +460,6 @@ class PatternMatchingTestCases < ActionView::TestCase
 
       render "developers/developer_with_h1", developer: developer
 
-      # rubocop:disable Lint/Syntax
       assert_pattern { document_root_element.at("h1") => { content: "Eloy", attributes: [{ name: "id", value: "name" }] } }
       refute_pattern { document_root_element.at("h1") => { content: "Not Eloy" } }
     end
@@ -470,21 +469,17 @@ class PatternMatchingTestCases < ActionView::TestCase
 
       render "developers/developer", developer: developer
 
-      # rubocop:disable Lint/Syntax
       assert_pattern { rendered.html => { content: "Eloy" } }
       refute_pattern { rendered.html => { content: "Not Eloy" } }
-      # rubocop:enable Lint/Syntax
     end
 
     test "rendered.json integrates with pattern matching" do
       developer = DeveloperStruct.new("Eloy")
 
       render formats: :json, partial: "developers/developer", locals: { developer: developer }
 
-      # rubocop:disable Lint/Syntax
       assert_pattern { rendered.json => { name: "Eloy" } }
       refute_pattern { rendered.json => { name: "Not Eloy" } }
-      # rubocop:enable Lint/Syntax
     end
   end
 end

activesupport/lib/active_support/core_ext/object/instance_variables.rb

@@ -29,5 +29,4 @@ def instance_values
   def instance_variable_names
     instance_variables.map(&:name)
   end
-  end
 end

activesupport/lib/active_support/notifications/instrumenter.rb

@@ -210,7 +210,7 @@ def now_cpu
             Process.clock_gettime(Process::CLOCK_THREAD_CPUTIME_ID, :float_millisecond)
           end
         rescue
-          def now_cpu # rubocop:disable Lint/DuplicateMethods
+          def now_cpu
             0.0
           end
         end

activesupport/test/array_inquirer_test.rb

@@ -55,7 +55,7 @@ def respond_to_missing?(name, include_private = false)
   ensure
     Array.class_eval do
       undef_method :respond_to_missing?
-      def respond_to_missing?(name, include_private = false) # rubocop:disable Lint/DuplicateMethods
+      def respond_to_missing?(name, include_private = false)
         super
       end
     end

activesupport/test/string_inquirer_test.rb

@@ -38,7 +38,7 @@ def respond_to_missing?(name, include_private = false)
   ensure
     String.class_eval do
       undef_method :respond_to_missing?
-      def respond_to_missing?(name, include_private = false) # rubocop:disable Lint/DuplicateMethods
+      def respond_to_missing?(name, include_private = false)
         super
       end
     end

railties/lib/rails/all.rb

@@ -1,7 +1,5 @@
 # frozen_string_literal: true
 
-# rubocop:disable Style/RedundantBegin
-
 require "rails"
 
 %w(
@@ -16,7 +14,7 @@
   action_text/engine
   rails/test_unit/railtie
 ).each do |railtie|
-  begin
+  begin # rubocop:disable Style/RedundantBegin
     require railtie
   rescue LoadError
   end