ghost_adapter 0.4.2 → 0.6.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: d80bd988fa02baef1df1c337a4738a1b2066614b91781742c43f57083aed7b96
-  data.tar.gz: 987a7418fd4cdb664dee43d44b030bbce58ab1e632b521b9aef5b9e066a668e0
+  metadata.gz: db6c7c5238b2a991b16d8f6ee88f1235a444140c5c63cc54642f24ba0e7ab676
+  data.tar.gz: 397e5378c2ea9c23037f30ed25a855bd851ab711988a9a5aac522bc7f8b95839
 SHA512:
-  metadata.gz: fa3a445e9552d8614eebf6b999edf490e33b7bfbb2f2ce12523932788c33c57eda730fafb0856f410dc1c871266a6b13b5db48b671c92fb6df18b5e06c81ae05
-  data.tar.gz: f32b9d3949a62052cd0ba561c10b86cb064eb93f0f284915ed2e1c432e4b74e7f20322b8463e127826fcf87293b7754a4f0695ee6a063fa32da6c7cf4700eda5
+  metadata.gz: 72c3f9895504609bfb0f46bcee0545ddf2d6af5f88c07bd2fe0c678b6151c5a7198eeaa8e3f9823d4d66ad92c1b8a55d45fda648fde3a538ffca4ee62d52056c
+  data.tar.gz: a60ca3725c8ff4ffe3beef1670815991d2113b10c5a2af61b192f5586c2ee086611f98cfc605343b58eb91d829c59971426878033ca86685f5a6f2a584438573

data/.github/dependabot.yml

@@ -0,0 +1,10 @@
+version: 2
+updates:
+  - package-ecosystem: bundler
+    directory: /
+    schedule:
+      interval: weekly
+  - package-ecosystem: github-actions
+    directory: /
+    schedule:
+      interval: weekly

data/README.md

@@ -62,6 +62,16 @@ If you have used the rails generator, you can set the variable to a falsey value
 - "truthy" values: `[1, t, true, y, yes]`
 - "falsey" values: `[0, f, false, n, no]`
 
+### Running tests
+
+To run the tests for all versions, run this script:
+
+``` shell
+bin/test_all_versions
+```
+
+Make sure to use the appropriate Ruby version! ActiveRecord <6.0 is not compatible with Ruby 3, so specs for those versions will only run successfully in a Ruby 2 environment.
+
 ## Contributing
 
 Bug reports and pull requests are welcome on GitHub at https://github.com/wetransfer/ghost_adapter. This project is intended to be a safe, welcoming space for collaboration, and contributors are expected to adhere to the [code of conduct](./CODE_OF_CONDUCT.md).

data/ghost_adapter.gemspec

@@ -23,7 +23,7 @@ Gem::Specification.new do |spec|
   end
   spec.require_paths = ['lib']
 
-  spec.add_dependency 'activerecord', '>= 5', '<= 7.1'
+  spec.add_dependency 'activerecord', '>= 5', '<= 7.2'
   spec.add_dependency 'mysql2', '>= 0.4.0', '< 0.6.0'
 
   spec.add_development_dependency 'bump', '~> 0'

data/lib/active_record/connection_adapters/mysql2_ghost_adapter.rb

@@ -18,9 +18,9 @@ module ActiveRecord
       end
 
       client = Mysql2::Client.new(config)
-      if GhostAdapter::Internal.ghost_migration_enabeld?
-        dry_run = ENV['DRY_RUN'] == '1'
-        GhostAdapter::VersionChecker.validate_executable! unless ENV['SKIP_GHOST_VERSION_CHECK'] == '1'
+      if GhostAdapter::Internal.ghost_migration_enabled?
+        dry_run = ENV.fetch('DRY_RUN', nil) == '1'
+        GhostAdapter::VersionChecker.validate_executable! unless ENV.fetch('SKIP_GHOST_VERSION_CHECK', nil) == '1'
         ConnectionAdapters::Mysql2GhostAdapter.new(client, logger, nil, config, dry_run: dry_run)
       else
         ConnectionAdapters::Mysql2Adapter.new(client, logger, nil, config)
@@ -42,16 +42,29 @@ module ActiveRecord
         @dry_run = dry_run
       end
 
-      def execute(sql, name = nil)
-        # Only ALTER TABLE statements are automatically skipped by gh-ost
-        # We need to manually skip CREATE TABLE, DROP TABLE, and
-        # INSERT/DELETE (to schema migrations) for dry runs
-        return if dry_run && should_skip_for_dry_run?(sql)
-
-        if (table, query = parse_sql(sql))
-          GhostAdapter::Migrator.execute(table, query, database, dry_run)
-        else
-          super(sql, name)
+      if Gem.loaded_specs['activerecord'].version >= Gem::Version.new('7.0')
+        def execute(sql, name = nil, async: false)
+          # Only ALTER TABLE statements are automatically skipped by gh-ost
+          # We need to manually skip CREATE TABLE, DROP TABLE, and
+          # INSERT/DELETE (to schema migrations) for dry runs
+          return if dry_run && should_skip_for_dry_run?(sql)
+
+          if (table, query = parse_sql(sql))
+            GhostAdapter::Migrator.execute(table, query, database, dry_run)
+          else
+            super(sql, name, async: async)
+          end
+        end
+      else
+        def execute(sql, name = nil)
+          # See comment above -- some tables need to be skipped manually for dry runs
+          return if dry_run && should_skip_for_dry_run?(sql)
+
+          if (table, query = parse_sql(sql))
+            GhostAdapter::Migrator.execute(table, query, database, dry_run)
+          else
+            super(sql, name)
+          end
         end
       end
 
@@ -80,7 +93,7 @@ module ActiveRecord
         end
       else
         def add_index(table_name, column_name, options = {})
-          index_name, index_type, index_columns, _index_options = add_index_options(table_name, column_name, options)
+          index_name, index_type, index_columns, _index_options = add_index_options(table_name, column_name, **options)
 
           sql = build_add_index_sql(
             table_name, index_columns, index_name,

data/lib/ghost_adapter/version.rb

@@ -1,3 +1,3 @@
 module GhostAdapter
-  VERSION = '0.4.2'.freeze
+  VERSION = '0.6.0'.freeze
 end

data/lib/ghost_adapter.rb

@@ -5,7 +5,7 @@ require 'active_record/connection_adapters/mysql2_ghost_adapter'
 
 require 'ghost_adapter/config'
 
-require 'ghost_adapter/railtie' if defined? ::Rails::Railtie
+require 'ghost_adapter/railtie' if defined? Rails::Railtie
 
 module GhostAdapter
   def self.config
@@ -43,8 +43,8 @@ module GhostAdapter
       @@ghost_migration_enabled = true # rubocop:disable Style/ClassVars
     end
 
-    def self.ghost_migration_enabeld?
-      env_val = ENV['GHOST_MIGRATE']&.downcase
+    def self.ghost_migration_enabled?
+      env_val = ENV.fetch('GHOST_MIGRATE', 'false').downcase
       return false if %w[0 n no f false].include?(env_val)
 
       !!@@ghost_migration_enabled || %w[1 y yes t true].include?(env_val)

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: ghost_adapter
 version: !ruby/object:Gem::Version
-  version: 0.4.2
+  version: 0.6.0
 platform: ruby
 authors:
 - Austin C Roos
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2022-03-30 00:00:00.000000000 Z
+date: 2024-04-29 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activerecord
@@ -19,7 +19,7 @@ dependencies:
         version: '5'
     - - "<="
       - !ruby/object:Gem::Version
-        version: '7.1'
+        version: '7.2'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
@@ -29,7 +29,7 @@ dependencies:
         version: '5'
     - - "<="
       - !ruby/object:Gem::Version
-        version: '7.1'
+        version: '7.2'
 - !ruby/object:Gem::Dependency
   name: mysql2
   requirement: !ruby/object:Gem::Requirement
@@ -144,6 +144,7 @@ files:
 - ".github/ISSUE_TEMPLATE/bug_report.md"
 - ".github/ISSUE_TEMPLATE/feature_request.md"
 - ".github/PULL_REQUEST_TEMPLATE.md"
+- ".github/dependabot.yml"
 - ".github/workflows/tests.yml"
 - ".gitignore"
 - ".rubocop.yml"
@@ -197,7 +198,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.2.32
+rubygems_version: 3.4.6
 signing_key:
 specification_version: 4
 summary: Run ActiveRecord migrations through gh-ost