Centralize Ruby Version to .ruby-version (#521)

* Set .ruby-version to 3.2.2

* Remove ruby version definition in CI/CD
it will be read from .ruby-version

* Set required_ruby_version to >= 2.7.0

* Remove TargetRubyVersion in RuboCop config
reads from required_ruby_version in RuboCop 1.61+

* Autocorrect lint violations after updating to Rubocop 1.62.1

.github/workflows/analysis.yml

@@ -28,7 +28,6 @@ jobs:
       -
         uses: ruby/setup-ruby@v1
         with:
-          ruby-version: "3.2"
           bundler-cache: true
       -
         name: Ruby Linting

.rubocop.yml

@@ -10,7 +10,6 @@ inherit_gem:
   rubocop-shopify: rubocop.yml
 
 AllCops:
-  TargetRubyVersion: 2.7
   NewCops: enable
   Exclude:
     - test/allow_list_bench.rb

.ruby-version

@@ -0,0 +1 @@
+3.2.2

examples/net_http/12_bulkhead_quotes_in_forks.rb

@@ -17,7 +17,7 @@
 SEMIAN_PARAMETERS = {
   bulkhead: true,
   quota: 1, # Tickets calculated base on number of workers
-  timeout: 1,            # Timeout in seconds (1 sec) to wait to get a free ticket.
+  timeout: 1, # Timeout in seconds (1 sec) to wait to get a free ticket.
   circuit_breaker: false,
 }
 

lib/semian/mysql2.rb

@@ -136,7 +136,8 @@ def acquire_semian_resource(**)
 
     def raw_semian_options
       return query_options[:semian] if query_options.key?(:semian)
-      return query_options["semian"] if query_options.key?("semian")
+
+      query_options["semian"] if query_options.key?("semian")
     end
   end
 end

lib/semian/redis.rb

@@ -151,7 +151,8 @@ def resource_exceptions
 
     def raw_semian_options
       return options[:semian] if options.key?(:semian)
-      return options["semian"] if options.key?("semian")
+
+      options["semian"] if options.key?("semian")
     end
 
     def raise_if_out_of_memory(reply)

semian.gemspec

@@ -27,6 +27,8 @@ Gem::Specification.new do |s|
     "source_code_uri" => "https://github.com/Shopify/semian",
   }
 
+  s.required_ruby_version = ">= 2.7.0"
+
   s.files = Dir["{lib,ext}/**/**/*.{rb,h,c}"]
   s.files += ["LICENSE.md", "README.md"]
   s.extensions = ["ext/semian/extconf.rb"]

test/adapters/grpc_test.rb

@@ -18,7 +18,7 @@ class TestGRPC < Minitest::Test
   }
 
   DEFAULT_SEMIAN_CONFIGURATION = proc do |host|
-    if host == SemianConfig["toxiproxy_upstream_host"] && \
+    if host == SemianConfig["toxiproxy_upstream_host"] &&
         port == SemianConfig["toxiproxy_upstream_port"] # disable if toxiproxy
       next nil
     end

test/adapters/net_http_test.rb

@@ -11,7 +11,7 @@ class TestNetHTTP < Minitest::Test
     error_timeout: 10,
   }.freeze
   DEFAULT_SEMIAN_CONFIGURATION = proc do |host, port|
-    if host == SemianConfig["toxiproxy_upstream_host"] && \
+    if host == SemianConfig["toxiproxy_upstream_host"] &&
         port == SemianConfig["toxiproxy_upstream_port"] # disable if toxiproxy
       next nil
     end
@@ -311,7 +311,7 @@ def test_use_custom_configuration_to_combine_endpoints_into_one_resource
     sample_env = "development"
 
     semian_configuration_proc = proc do |host, port|
-      if host == SemianConfig["toxiproxy_upstream_host"] && \
+      if host == SemianConfig["toxiproxy_upstream_host"] &&
           port == SemianConfig["toxiproxy_upstream_port"] # disable if toxiproxy
         next nil
       end

test/adapters/redis_client_test.rb

@@ -405,14 +405,15 @@ def with_readonly_mode(client)
   def new_config(**options)
     options[:host] = SemianConfig["toxiproxy_upstream_host"] if options[:host].nil?
     semian_options = SEMIAN_OPTIONS.merge(options.delete(:semian) || {})
-    RedisClient.config(**{
+    RedisClient.config(
       port: SemianConfig["redis_toxiproxy_port"],
       reconnect_attempts: 0,
       db: 1,
       timeout: REDIS_TIMEOUT,
       semian: semian_options,
       driver: redis_driver,
-    }.merge(options))
+      **options,
+    )
   end
 
   def connect_to_redis!(semian_options = {})