Setup Ruby linter and correct mistakes

Author: Edouard-chin

.rubocop.yml

@@ -0,0 +1,8 @@
+inherit_gem:
+  rubocop-shopify: rubocop.yml
+
+AllCops:
+  NewCops: disable
+  SuggestExtensions: false
+  Exclude:
+    - test/fixtures/date/**/*

Gemfile

@@ -7,3 +7,4 @@ gemspec
 
 gem "rake", "~> 13.0"
 gem "minitest", "~> 5.16"
+gem "rubocop-shopify"

Gemfile.lock

@@ -9,12 +9,43 @@ PATH
 GEM
   remote: https://rubygems.org/
   specs:
+    ast (2.4.3)
+    json (2.16.0)
+    language_server-protocol (3.17.0.5)
+    lint_roller (1.1.0)
     minitest (5.25.5)
+    parallel (1.27.0)
+    parser (3.3.10.0)
+      ast (~> 2.4.1)
+      racc
     prism (1.6.0)
+    racc (1.8.1)
+    rainbow (3.1.1)
     rake (13.3.0)
     rake-compiler (1.3.0)
       rake
+    regexp_parser (2.11.3)
+    rubocop (1.81.7)
+      json (~> 2.3)
+      language_server-protocol (~> 3.17.0.2)
+      lint_roller (~> 1.1.0)
+      parallel (~> 1.10)
+      parser (>= 3.3.0.2)
+      rainbow (>= 2.2.2, < 4.0)
+      regexp_parser (>= 2.9.3, < 3.0)
+      rubocop-ast (>= 1.47.1, < 2.0)
+      ruby-progressbar (~> 1.7)
+      unicode-display_width (>= 2.4.0, < 4.0)
+    rubocop-ast (1.48.0)
+      parser (>= 3.3.7.2)
+      prism (~> 1.4)
+    rubocop-shopify (2.18.0)
+      rubocop (~> 1.62)
+    ruby-progressbar (1.13.0)
     thor (1.4.0)
+    unicode-display_width (3.2.0)
+      unicode-emoji (~> 4.1)
+    unicode-emoji (4.1.0)
 
 PLATFORMS
   arm64-darwin-23
@@ -24,6 +55,7 @@ DEPENDENCIES
   cibuildgem!
   minitest (~> 5.16)
   rake (~> 13.0)
+  rubocop-shopify
 
 BUNDLED WITH
    2.7.2

Rakefile

@@ -13,4 +13,4 @@ rescue LoadError
   # it the GitHub action clones the repo and runs `rake install`, without running a `bundle install` first.
 end
 
-task default: :test
+task(default: :test)

cibuildgem.gemspec

@@ -30,7 +30,7 @@ Gem::Specification.new do |spec|
   spec.executables = spec.files.grep(%r{\Aexe/}) { |f| File.basename(f) }
   spec.require_paths = ["lib"]
 
-  spec.add_dependency "rake-compiler"
-  spec.add_dependency "thor"
-  spec.add_dependency "prism"
+  spec.add_dependency("prism")
+  spec.add_dependency("rake-compiler")
+  spec.add_dependency("thor")
 end

lib/cibuildgem/cli.rb

@@ -9,8 +9,10 @@ class CLI < Thor
 
     source_root(File.expand_path("templates", __dir__))
 
-    def self.exit_on_failure?
-      true
+    class << self
+      def exit_on_failure?
+        true
+      end
     end
 
     desc "compile", "Compile a gem's native extension."
@@ -92,13 +94,12 @@ def clobber
     method_option "working-directory", type: "string", required: false, desc: "If your gem lives outside of the repository root, specify where."
     method_option "test-command", type: "string", required: false, desc: "The test command to run. Defaults to running `bundle exec rake test` and `bundle exec rake spec`."
     def ci_template
-      # os = ["macos-latest", "macos-15-intel", "ubuntu-latest", "windows-latest"]
-      os = ["macos-latest", "ubuntu-22.04"] # Just this for now because the CI takes too long otherwise.
-
       ruby_requirements = compilation_task.gemspec.required_ruby_version
-      latest_supported_ruby_version = RubySeries.latest_version_for_requirements(ruby_requirements)
-      runtime_version_for_compilation = RubySeries.runtime_version_for_compilation(ruby_requirements)
-      ruby_versions_for_testing = RubySeries.versions_to_test_against(ruby_requirements)
+      # os = ["macos-latest", "macos-15-intel", "ubuntu-latest", "windows-latest"]
+      @os = ["macos-latest", "ubuntu-22.04"] # Just this for now because the CI takes too long otherwise.
+      @latest_supported_ruby_version = RubySeries.latest_version_for_requirements(ruby_requirements)
+      @runtime_version_for_compilation = RubySeries.runtime_version_for_compilation(ruby_requirements)
+      @ruby_versions_for_testing = RubySeries.versions_to_test_against(ruby_requirements)
 
       directory(".github", context: instance_eval("binding"))
     end
@@ -117,12 +118,12 @@ def release
     desc "print_ruby_cc_version", "Output the cross compile ruby version needed for the gem. For internal usage", hide: true
     method_option "gemspec", type: "string", required: false, desc: "The gemspec to use. If the option is not passed, a gemspec file from the current working directory will be used."
     def print_ruby_cc_version
-      print compilation_task.ruby_cc_version
+      print(compilation_task.ruby_cc_version)
     end
 
     desc "normalized_platform", "The platform name for compilation purposes", hide: true
     def print_normalized_platform
-      print compilation_task.normalized_platform
+      print(compilation_task.normalized_platform)
     end
 
     private
@@ -143,7 +144,7 @@ def run_rake_tasks!(*tasks)
     def compilation_task
       @compilation_task ||= CompilationTasks.new(false)
     rescue GemspecError => e
-      print e.message
+      print(e.message)
 
       Kernel.exit(false)
     end

lib/cibuildgem/compilation_tasks.rb

@@ -10,7 +10,7 @@ class CompilationTasks
     attr_reader :gemspec, :native, :create_packaging_task, :extension_task
 
     def initialize(create_packaging_task = false, gemspec = nil)
-      @gemspec  = Bundler.load_gemspec(gemspec || find_gemspec)
+      @gemspec = Bundler.load_gemspec(gemspec || find_gemspec)
       verify_gemspec!
 
       @create_packaging_task = create_packaging_task
@@ -85,7 +85,7 @@ def disable_shared
           makefile_content.match(/LIBRUBYARG_SHARED = (.*)/) do |match|
             shared_flags = match[1].split(" ")
             shared_flags.reject! { |flag| flag == "-l$(RUBY_SO_NAME)" }
-            makefile_content.gsub!(/(LIBRUBYARG_SHARED = ).*/, "\\1#{shared_flags.join(' ')}")
+            makefile_content.gsub!(/(LIBRUBYARG_SHARED = ).*/, "\\1#{shared_flags.join(" ")}")
 
             File.write(task.name, makefile_content)
           end

lib/cibuildgem/ruby_series.rb

@@ -31,7 +31,7 @@ def versions_to_test_against(requirements)
       end.reverse
 
       selected_rubies.map do |version|
-        version.segments.tap(&:pop).join('.')
+        version.segments.tap(&:pop).join(".")
       end
     end
 

lib/cibuildgem/tasks/wrapper.rake

@@ -22,7 +22,7 @@ end
 
 unless Rake::Task.task_defined?(:test)
   task(:test) do
-    raise(RuntimeError, "Don't know how to build task 'test'") unless Rake::Task.task_defined?(:spec)
+    raise("Don't know how to build task 'test'") unless Rake::Task.task_defined?(:spec)
 
     Rake::Task[:spec].invoke
   end

lib/cibuildgem/templates/.github/workflows/cibuildgem.yaml.tt

@@ -13,15 +13,15 @@ jobs:
     name: "Cross compile the gem on different ruby versions"
     strategy:
       matrix:
-        os: <%= os %>
+        os: <%= @os %>
     runs-on: "${{ matrix.os }}"
     steps:
       - name: "Checkout code"
         uses: "actions/checkout@v5"
       - name: "Setup Ruby"
         uses: "ruby/setup-ruby@v1"
         with:
-          ruby-version: "<%= runtime_version_for_compilation %>"
+          ruby-version: "<%= @runtime_version_for_compilation %>"
           bundler-cache: true
           <%- if options['working-directory'] -%>
           working-directory: "<%= options['working-directory'] %>"
@@ -39,8 +39,8 @@ jobs:
     needs: compile
     strategy:
       matrix:
-        os: <%= os %>
-        rubies: <%= ruby_versions_for_testing %>
+        os: <%= @os %>
+        rubies: <%= @ruby_versions_for_testing %>
         type: ["cross", "native"]
     runs-on: "${{ matrix.os }}"
     steps:
@@ -70,13 +70,13 @@ jobs:
     needs: test
     strategy:
       matrix:
-        os: <%= os %>
+        os: <%= @os %>
     runs-on: "${{ matrix.os }}"
     steps:
       - name: "Setup Ruby"
         uses: "ruby/setup-ruby@v1"
         with:
-          ruby-version: "<%= latest_supported_ruby_version %>"
+          ruby-version: "<%= @latest_supported_ruby_version %>"
       - name: "Run cibuildgem"
         uses: "shopify/cibuildgem/.github/actions/cibuildgem@main"
         with: