zero_downtime_migrations 0.0.3 → 0.0.4

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: b7c11ffb1f189aa72d6e3f293ed8b002a237fd0a
-  data.tar.gz: 9c21a8f52fa53bcc6e37968e98934f1eed21cae5
+  metadata.gz: c54176ffa5402b379ae752cfb6a16a47213f31f6
+  data.tar.gz: 61c5e351d8d8540725c9c296df00cdc4ff70efb9
 SHA512:
-  metadata.gz: a0f57e9b8d06259aaec58f60145ec790188b847e3c78e824b65be954e16a9da44f053d439718cf8edda9e1c55d375258435d72ffa5b0054404f7c50389fa7ae2
-  data.tar.gz: acddd8d588ffdb007be328a1b932a96583b6762be95859cfd7860afd4d7db3191c561989676360db8ed3641ee8ce64ff9249a7874c4c7782e216e4fb31cef76b
+  metadata.gz: a79a860e93a831230f19886aefd0cc53e965175f18ce2e2701b60c19fe716e0f2837531cb8015dd180c75ab139feab0f71555f2e2893578afd0c8566b781a553
+  data.tar.gz: fe960c837d456b92dafa3236bd0051d910bf0e5b2b66527aa0c1ab5dcc6d8bae14ea655d30c12f1f3efa4a2af441934474334c823cdeedf32e9ca861ccaa4790

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    zero_downtime_migrations (0.0.3)
+    zero_downtime_migrations (0.0.4)
       activerecord
 
 GEM

data/lib/zero_downtime_migrations/dsl.rb

@@ -1,6 +1,6 @@
 module ZeroDowntimeMigrations
   module DSL
-    attr_accessor :data, :ddl, :index, :migrating, :safe
+    attr_accessor :current, :data, :ddl, :index, :safe
 
     def data?
       !!@data
@@ -15,7 +15,7 @@ module ZeroDowntimeMigrations
     end
 
     def migrating?
-      !!@migrating
+      !!@current
     end
 
     def mixed?

data/lib/zero_downtime_migrations/migration.rb

@@ -11,24 +11,24 @@ module ZeroDowntimeMigrations
     end
 
     def define(*)
-      Migration.migrating = true
+      Migration.current = self
       Migration.safe = true
-      super
+      super.tap { Migration.current = nil }
     end
 
     def migrate(direction)
       @direction = direction
 
+      Migration.current = self
       Migration.data = false
       Migration.ddl = false
       Migration.index = false
-      Migration.migrating = true
       Migration.safe ||= reverse_migration? || rollup_migration?
 
       super.tap do
         validate(:ddl_migration)
         validate(:mixed_migration)
-        Migration.migrating = false
+        Migration.current = nil
         Migration.safe = false
       end
     end
@@ -94,7 +94,7 @@ module ZeroDowntimeMigrations
     end
 
     def validate(type, *args)
-      Validation.validate!(type, self, *args)
+      Validation.validate!(type, *args)
     rescue UndefinedValidationError
       nil
     end

data/lib/zero_downtime_migrations/validation/find_each.rb

@@ -19,10 +19,10 @@ module ZeroDowntimeMigrations
           If you're 100% positive that this migration is already safe, then wrap the
           call to `each` in a `safety_assured` block.
 
-            class YourMigration < ActiveRecord::Migration
+            class #{migration_name} < ActiveRecord::Migration
               def change
                 safety_assured do
-                  # use .each in this block
+                  # use ActiveRecord::Relation.each in this block
                 end
               end
             end

data/lib/zero_downtime_migrations/validation.rb

@@ -1,11 +1,11 @@
 module ZeroDowntimeMigrations
   class Validation
-    def self.validate!(type, migration = nil, *args)
+    def self.validate!(type, *args)
       return unless Migration.migrating? && Migration.unsafe?
 
       begin
         validator = type.to_s.classify
-        const_get(validator).new(migration, *args).validate!
+        const_get(validator).new(Migration.current, *args).validate!
       rescue NameError
         raise UndefinedValidationError.new(validator)
       end
@@ -19,7 +19,10 @@ module ZeroDowntimeMigrations
     end
 
     def error!(message)
-      raise UnsafeMigrationError.new(message)
+      error = UnsafeMigrationError
+      debug = "#{error}: #{migration_name} is unsafe!"
+      message = [message, debug, nil].join("\n")
+      raise error.new(message)
     end
 
     def migration_name

data/zero_downtime_migrations.gemspec

@@ -11,7 +11,7 @@ Gem::Specification.new do |s|
   s.required_ruby_version = ">= 2.0.0"
   s.summary               = "Zero downtime migrations with ActiveRecord and PostgreSQL"
   s.test_files            = `git ls-files -- spec/*`.split("\n")
-  s.version               = "0.0.3"
+  s.version               = "0.0.4"
 
   s.add_dependency "activerecord"
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: zero_downtime_migrations
 version: !ruby/object:Gem::Version
-  version: 0.0.3
+  version: 0.0.4
 platform: ruby
 authors:
 - LendingHome