data-migration 1.0.0 → 1.2.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: c310b238ffce7232c8d7e043b3bb05c22c7fc9feb1a202653172152e0eafe54f
-  data.tar.gz: c177a8be4fb4805a0cdac0683ea2b1711f8e78df14176f271d834869c9d7eb0f
+  metadata.gz: 9a8f110139404aa9a93001f7d7d62df44dfa39ed17c6467c3042b71071f0f5a9
+  data.tar.gz: 17ea84c1d597d6ba5907f0841bee35cf231225f9ac6febfa0dfa4b6c697105d3
 SHA512:
-  metadata.gz: 5c3653642986c3de1ac483875541c433f44fc604fe648dcf3b22a9065b16d0370b83c6a55ad20c7a41ed78dee646c14f69eacd9cf2f3f3fa49f7e9b8e944fb44
-  data.tar.gz: f604aa6ac2c1993c04b7f993eabf4f56f81c0db9b1c0a892e46f390a75fe4c11a0f8d237b2e80de190b497a07b68d0f7e3d08b162682245234120e1e40a6e58f
+  metadata.gz: 2cb6a04608ff5fa007f3bc3e5e6fd819413d73b9041ee8ec542de029719d6c089706528e3ec72e9d0442d75596709cb2646f004c00bce6a9100a063be2e91eab
+  data.tar.gz: 53b024625cc45959ac7cc94c5ccb5bf22f1ff463a04cfea4f847a43292b688092aaf9f3e757193fd15c17c47f2a9539b208be3153e043cca313f3b78682ff36c

data/CHANGELOG.md

@@ -1,3 +1,13 @@
-# 1.0.0
+# CHANGELOG
+
+## 1.2.0
+
+- Wrap Task.status enum with ActiveRecord version check
+
+## 1.1.0
+
+- Fix gem main file name to be `data-migration.rb` instead of `data_migration.rb`
+
+## 1.0.0
 
 - Initial version

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    data-migration (1.0.0)
+    data-migration (1.2.0)
       activejob (> 5)
       activerecord (> 5)
       activesupport (> 5)

data/README.md

@@ -129,7 +129,7 @@ Contribution policy:
 Prefer using script `usr/bin/release.sh`, it will ensure that repository is synced and after publishing gem will create a tag.
 
 ```sh
-GEM_VERSION=$(grep -Eo "VERSION\s*=\s*\".+\"" lib/data_migration.rb  | grep -Eo "[0-9.]{5,}")
+GEM_VERSION=$(grep -Eo "VERSION\s*=\s*\".+\"" lib/data-migration.rb  | grep -Eo "[0-9.]{5,}")
 rm data-migration-*.gem
 gem build data-migration.gemspec
 gem push data-migration-$GEM_VERSION.gem

data/data-migration.gemspec

@@ -1,6 +1,6 @@
 Gem::Specification.new do |gem|
   gem.name = "data-migration"
-  gem.version = File.read(File.expand_path("../lib/data_migration.rb", __FILE__)).match(/VERSION\s*=\s*"(.*?)"/)[1]
+  gem.version = File.read(File.expand_path("../lib/data-migration.rb", __FILE__)).match(/VERSION\s*=\s*"(.*?)"/)[1]
 
   repository_url = "https://github.com/amkisko/data-migration.rb"
   root_files = %w[CHANGELOG.md LICENSE.md README.md]

data/lib/{data_migration.rb → data-migration.rb}

@@ -3,7 +3,7 @@ require "data_migration/job"
 require "data_migration/task"
 
 module DataMigration
-  VERSION = "1.0.0".freeze
+  VERSION = "1.2.0".freeze
 
   module_function
 
@@ -17,7 +17,7 @@ module DataMigration
 
   def notify(message, context: {})
     if Object.const_defined?(:ActionReporter)
-      ActionReporter.notify(message, context:)
+      ActionReporter.notify(message, context: context.presence || {})
     elsif Object.const_defined?(:Rails)
       Rails.logger.info("#{message} #{context.inspect}")
     end

data/lib/data_migration/config.rb

@@ -35,7 +35,7 @@ module DataMigration
 
     attr_writer :operator_resolver
     def operator_resolver(&block)
-      if block_given?
+      if block.present?
         @operator_resolver = block
       else
         @operator_resolver ||= -> do
@@ -50,7 +50,7 @@ module DataMigration
 
     attr_writer :monitoring_context
     def monitoring_context(&block)
-      if block_given?
+      if block.present?
         @monitoring_context = block
       else
         @monitoring_context ||= ->(migration) do

data/lib/data_migration/job.rb

@@ -18,7 +18,7 @@ module DataMigration
         return
       end
 
-      task.job_check_in!(job_id, job_args:, job_kwargs:)
+      task.job_check_in!(job_id, job_args: job_args, job_kwargs: job_kwargs)
 
       require migration_path
       klass_name = migration_name.gsub(/^[0-9_]+/, "").camelize

data/lib/data_migration/task.rb

@@ -2,6 +2,7 @@ require "active_record"
 
 module DataMigration
   class JobConcurrencyLimitError < StandardError; end
+
   class JobConflictError < StandardError; end
 
   def self.tasks_table_name
@@ -18,16 +19,21 @@ module DataMigration
       self.jobs_limit ||= DataMigration.config.default_jobs_limit
     end
 
-    enum :status, {
+    STATUS_OPTIONS = {
       started: "started",
       performing: "performing",
       paused: "paused",
       completed: "completed"
     }
+    if ActiveRecord::VERSION::MAJOR >= 7
+      enum :status, STATUS_OPTIONS
+    else
+      enum status: STATUS_OPTIONS
+    end
 
     validates :name, presence: true
-    validates :pause_minutes, numericality: { greater_than_or_equal_to: 0, only_integer: true }, if: -> { pause_minutes.present? }
-    validates :jobs_limit, numericality: { greater_than_or_equal_to: 0, only_integer: true }, if: -> { jobs_limit.present? }
+    validates :pause_minutes, numericality: {greater_than_or_equal_to: 0, only_integer: true}, if: -> { pause_minutes.present? }
+    validates :jobs_limit, numericality: {greater_than_or_equal_to: 0, only_integer: true}, if: -> { jobs_limit.present? }
     validate :file_should_exist
 
     after_save do
@@ -50,7 +56,7 @@ module DataMigration
     end
 
     def self.perform_now(name, **kwargs)
-      create!(name:).perform_now(**kwargs)
+      create!(name: name).perform_now(**kwargs)
     end
 
     def perform_now(**perform_args)
@@ -59,7 +65,7 @@ module DataMigration
     end
 
     def self.perform_later(name, **kwargs)
-      create!(name:).perform_later(**kwargs)
+      create!(name: name).perform_later(**kwargs)
     end
 
     def perform_later(**perform_args)
@@ -68,7 +74,7 @@ module DataMigration
     end
 
     def self.prepare(name, pause_minutes: nil, jobs_limit: nil)
-      create!(name:, pause_minutes:, jobs_limit:)
+      create!(name: name, pause_minutes: pause_minutes, jobs_limit: jobs_limit)
     end
 
     def self.root_path

data/spec/fixtures/data_migrations/20241206200114_create_batch_users.rb

@@ -4,6 +4,6 @@ class CreateBatchUsers
 
     User.find_or_create_by(email: "test_#{index}@example.com")
 
-    enqueue(index: index + 1, background:)
+    enqueue(index: index + 1, background: background)
   end
 end

data/spec/rails_helper.rb

@@ -1,6 +1,6 @@
 require "active_record"
 
-require "data_migration"
+require "data-migration"
 
 require "support/rails_helpers"
 

data/spec/spec_helper.rb

@@ -9,7 +9,7 @@ SimpleCov.formatter = SimpleCov::Formatter::CoberturaFormatter
 
 require "active_record"
 
-require "data_migration"
+require "data-migration"
 
 Dir[File.expand_path("support/**/*.rb", __dir__)].each { |f| require_relative f }
 

data/usr/bin/release.sh

@@ -18,7 +18,7 @@ if [[ $(git diff --shortstat 2>/dev/null | tail -n1) != "" ]]; then
 fi
 
 GEM_NAME="data-migration"
-VERSION=$(grep -Eo "VERSION\s*=\s*\".+\"" lib/data_migration.rb | grep -Eo "[0-9.]{5,}")
+VERSION=$(grep -Eo "VERSION\s*=\s*\".+\"" lib/data-migration.rb | grep -Eo "[0-9.]{5,}")
 GEM_FILE="$GEM_NAME-$VERSION.gem"
 
 e "gem build $GEM_NAME.gemspec"

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: data-migration
 version: !ruby/object:Gem::Version
-  version: 1.0.0
+  version: 1.2.0
 platform: ruby
 authors:
 - Andrei Makarov
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2024-12-08 00:00:00.000000000 Z
+date: 2024-12-10 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails
@@ -176,7 +176,7 @@ files:
 - LICENSE.md
 - README.md
 - data-migration.gemspec
-- lib/data_migration.rb
+- lib/data-migration.rb
 - lib/data_migration/config.rb
 - lib/data_migration/job.rb
 - lib/data_migration/task.rb