fix rubocop

.github/workflows/checkstyle.yml

@@ -24,8 +24,7 @@ jobs:
       - name: Set up Ruby ${{ matrix.ruby-version }}
         uses: ruby/setup-ruby@v1
         with:
-          ruby-version: 2.7
-          bundler: 1
+          ruby-version: 3.2
           bundler-cache: true
 
       - name: Run rubocop

.rubocop.yml

@@ -3,7 +3,10 @@ AllCops:
     - 'vendor/**/*'
     - 'gemfiles/**/*'
 
-Layout/TrailingBlankLines:
+Gemspec/RequiredRubyVersion:
+  Enabled: false
+
+Layout/TrailingEmptyLines:
   Enabled: false
 
 Metrics/BlockLength:
@@ -12,6 +15,10 @@ Metrics/BlockLength:
 Metrics/LineLength:
   Max: 120
 
+Naming/FileName:
+  Exclude:
+    - lib/rails-data-migrations.rb
+
 Style/Documentation:
   Enabled: false
 

Appraisals

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 ruby_version = Gem::Version.new(RUBY_VERSION)
 
 if ruby_version < Gem::Version.new('2.7.0')
@@ -6,6 +8,7 @@ if ruby_version < Gem::Version.new('2.7.0')
       gem 'rails', "~> #{rails_version}"
       gem 'bigdecimal', '1.3.5'
       gem 'sqlite3', '~> 1.3.6'
+      gem 'rubocop', '0.52.1'
     end
   end
 end

Gemfile

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 source 'https://rubygems.org'
 
 # Specify your gem's dependencies in activerecord-data-migrations.gemspec

Rakefile

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'bundler/gem_tasks'
 require 'rspec/core/rake_task'
 

lib/active_record/data_migration.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'rails/version'
 
 module ActiveRecord

lib/generators/data_migration_generator.rb

@@ -1,8 +1,10 @@
+# frozen_string_literal: true
+
 require 'rails/generators'
 require 'rails-data-migrations'
 
 class DataMigrationGenerator < Rails::Generators::NamedBase
-  source_root File.expand_path('../templates', __FILE__)
+  source_root File.expand_path('templates', __dir__)
 
   def create_migration_file
     migration_file_name =

lib/generators/templates/data_migration_generator.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 class ClassName < ActiveRecord::DataMigration
   def up
     # put your code here

lib/rails-data-migrations.rb

@@ -1,9 +1,9 @@
-# rubocop:disable Naming/FileName
+# frozen_string_literal: true
+
 require 'rails'
 require 'active_record'
 require 'active_record/data_migration'
 require 'rails_data_migrations/version'
 require 'rails_data_migrations/log_entry'
 require 'rails_data_migrations/migrator'
 require 'rails_data_migrations/railtie'
-# rubocop:enable Naming/FileName

lib/rails_data_migrations/log_entry.rb

@@ -1,8 +1,10 @@
+# frozen_string_literal: true
+
 module RailsDataMigrations
   class LogEntry < ::ActiveRecord::SchemaMigration
     class << self
       def table_name
-        ActiveRecord::Base.table_name_prefix + 'data_migrations' + ActiveRecord::Base.table_name_suffix
+        "#{ActiveRecord::Base.table_name_prefix}data_migrations#{ActiveRecord::Base.table_name_suffix}"
       end
 
       def index_name

lib/rails_data_migrations/migrator.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module RailsDataMigrations
   class Migrator < ::ActiveRecord::Migrator
     MIGRATOR_SALT = 2053462855
@@ -61,7 +63,7 @@ def list_pending_migrations
           already_migrated = get_all_versions
           list_migrations.reject { |m| already_migrated.include?(m.version) }
         else
-          open(migrations_path).pending_migrations
+          open(migrations_path).pending_migrations # rubocop:disable Security/Open
         end
       end
 

lib/rails_data_migrations/railtie.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'rails'
 
 module RailsDataMigrations

lib/rails_data_migrations/version.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module RailsDataMigrations
-  VERSION = '1.2.0'.freeze
+  VERSION = '1.2.0'
 end

lib/tasks/data_migrations.rake

@@ -1,8 +1,11 @@
+# frozen_string_literal: true
+
 require 'rake'
 
 namespace :data do
   def apply_single_migration(direction, version)
     raise 'VERSION is required' unless version
+
     RailsDataMigrations::Migrator.run_migration(
       direction,
       RailsDataMigrations::Migrator.migrations_path,
@@ -42,7 +45,8 @@ namespace :data do
     desc 'Skip single data migration using VERSION'
     task skip: :init_migration do
       version = ENV['VERSION'].to_i
-      raise 'VERSION is required' unless version > 0
+      raise 'VERSION is required' unless version.positive?
+
       if RailsDataMigrations::LogEntry.where(version: version).any?
         puts "data migration #{version} was already applied."
       else

rails-data-migrations.gemspec

@@ -1,4 +1,6 @@
-lib = File.expand_path('../lib', __FILE__)
+# frozen_string_literal: true
+
+lib = File.expand_path('lib', __dir__)
 $LOAD_PATH.unshift(lib) unless $LOAD_PATH.include?(lib)
 require 'rails_data_migrations/version'
 
@@ -23,6 +25,10 @@ Gem::Specification.new do |spec|
   spec.add_development_dependency 'appraisal', '~> 2.1'
   spec.add_development_dependency 'rake', '>= 12.3.3'
   spec.add_development_dependency 'rspec', '3.5.0'
-  spec.add_development_dependency 'rubocop', '0.52.1'
-  spec.add_development_dependency 'sqlite3', '~> 1.3.6'
+  spec.add_development_dependency 'rubocop', '1.63.5'
+
+  spec.add_development_dependency(
+    'sqlite3',
+    Gem::Version.new(RUBY_VERSION) >= Gem::Version.new('3.0.0') ? '~> 1.3' : '~> 1.3.6'
+  )
 end

spec/data_migrations_spec.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'spec_helper'
 
 describe RailsDataMigrations do
@@ -44,7 +46,7 @@
     end
 
     def load_rake_rasks
-      load File.expand_path('../../lib/tasks/data_migrations.rake', __FILE__)
+      load File.expand_path('../lib/tasks/data_migrations.rake', __dir__)
       Rake::Task.define_task(:environment)
     end
 

spec/spec_helper.rb

@@ -1,7 +1,9 @@
+# frozen_string_literal: true
+
 require 'rails/generators'
 require 'rake'
 
-$LOAD_PATH.unshift File.expand_path('../../lib', __FILE__)
+$LOAD_PATH.unshift File.expand_path('../lib', __dir__)
 require 'rails-data-migrations'
 
 ActiveRecord::Base.establish_connection adapter: 'sqlite3', database: ':memory:'

vendor/bundle/ruby/2.7.0/bin/appraisal

@@ -0,0 +1,27 @@
+#!/usr/bin/env ruby_executable_hooks
+#
+# This file was generated by RubyGems.
+#
+# The application 'appraisal' is installed as part of a gem, and
+# this file is here to facilitate running it.
+#
+
+require 'rubygems'
+
+version = ">= 0.a"
+
+str = ARGV.first
+if str
+  str = str.b[/\A_(.*)_\z/, 1]
+  if str and Gem::Version.correct?(str)
+    version = str
+    ARGV.shift
+  end
+end
+
+if Gem.respond_to?(:activate_bin_path)
+load Gem.activate_bin_path('appraisal', 'appraisal', version)
+else
+gem "appraisal", version
+load Gem.bin_path("appraisal", "appraisal", version)
+end

vendor/bundle/ruby/2.7.0/bin/htmldiff

@@ -0,0 +1,27 @@
+#!/usr/bin/env ruby_executable_hooks
+#
+# This file was generated by RubyGems.
+#
+# The application 'diff-lcs' is installed as part of a gem, and
+# this file is here to facilitate running it.
+#
+
+require 'rubygems'
+
+version = ">= 0.a"
+
+str = ARGV.first
+if str
+  str = str.b[/\A_(.*)_\z/, 1]
+  if str and Gem::Version.correct?(str)
+    version = str
+    ARGV.shift
+  end
+end
+
+if Gem.respond_to?(:activate_bin_path)
+load Gem.activate_bin_path('diff-lcs', 'htmldiff', version)
+else
+gem "diff-lcs", version
+load Gem.bin_path("diff-lcs", "htmldiff", version)
+end

vendor/bundle/ruby/2.7.0/bin/irb

@@ -0,0 +1,27 @@
+#!/usr/bin/env ruby_executable_hooks
+#
+# This file was generated by RubyGems.
+#
+# The application 'irb' is installed as part of a gem, and
+# this file is here to facilitate running it.
+#
+
+require 'rubygems'
+
+version = ">= 0.a"
+
+str = ARGV.first
+if str
+  str = str.b[/\A_(.*)_\z/, 1]
+  if str and Gem::Version.correct?(str)
+    version = str
+    ARGV.shift
+  end
+end
+
+if Gem.respond_to?(:activate_bin_path)
+load Gem.activate_bin_path('irb', 'irb', version)
+else
+gem "irb", version
+load Gem.bin_path("irb", "irb", version)
+end

vendor/bundle/ruby/2.7.0/bin/ldiff

@@ -0,0 +1,27 @@
+#!/usr/bin/env ruby_executable_hooks
+#
+# This file was generated by RubyGems.
+#
+# The application 'diff-lcs' is installed as part of a gem, and
+# this file is here to facilitate running it.
+#
+
+require 'rubygems'
+
+version = ">= 0.a"
+
+str = ARGV.first
+if str
+  str = str.b[/\A_(.*)_\z/, 1]
+  if str and Gem::Version.correct?(str)
+    version = str
+    ARGV.shift
+  end
+end
+
+if Gem.respond_to?(:activate_bin_path)
+load Gem.activate_bin_path('diff-lcs', 'ldiff', version)
+else
+gem "diff-lcs", version
+load Gem.bin_path("diff-lcs", "ldiff", version)
+end

vendor/bundle/ruby/2.7.0/bin/nokogiri

@@ -0,0 +1,27 @@
+#!/usr/bin/env ruby_executable_hooks
+#
+# This file was generated by RubyGems.
+#
+# The application 'nokogiri' is installed as part of a gem, and
+# this file is here to facilitate running it.
+#
+
+require 'rubygems'
+
+version = ">= 0.a"
+
+str = ARGV.first
+if str
+  str = str.b[/\A_(.*)_\z/, 1]
+  if str and Gem::Version.correct?(str)
+    version = str
+    ARGV.shift
+  end
+end
+
+if Gem.respond_to?(:activate_bin_path)
+load Gem.activate_bin_path('nokogiri', 'nokogiri', version)
+else
+gem "nokogiri", version
+load Gem.bin_path("nokogiri", "nokogiri", version)
+end

vendor/bundle/ruby/2.7.0/bin/racc

@@ -0,0 +1,27 @@
+#!/usr/bin/env ruby_executable_hooks
+#
+# This file was generated by RubyGems.
+#
+# The application 'racc' is installed as part of a gem, and
+# this file is here to facilitate running it.
+#
+
+require 'rubygems'
+
+version = ">= 0.a"
+
+str = ARGV.first
+if str
+  str = str.b[/\A_(.*)_\z/, 1]
+  if str and Gem::Version.correct?(str)
+    version = str
+    ARGV.shift
+  end
+end
+
+if Gem.respond_to?(:activate_bin_path)
+load Gem.activate_bin_path('racc', 'racc', version)
+else
+gem "racc", version
+load Gem.bin_path("racc", "racc", version)
+end

vendor/bundle/ruby/2.7.0/bin/rackup

@@ -0,0 +1,27 @@
+#!/usr/bin/env ruby_executable_hooks
+#
+# This file was generated by RubyGems.
+#
+# The application 'rackup' is installed as part of a gem, and
+# this file is here to facilitate running it.
+#
+
+require 'rubygems'
+
+version = ">= 0.a"
+
+str = ARGV.first
+if str
+  str = str.b[/\A_(.*)_\z/, 1]
+  if str and Gem::Version.correct?(str)
+    version = str
+    ARGV.shift
+  end
+end
+
+if Gem.respond_to?(:activate_bin_path)
+load Gem.activate_bin_path('rackup', 'rackup', version)
+else
+gem "rackup", version
+load Gem.bin_path("rackup", "rackup", version)
+end