James Sullivan

Results 3 issues of James Sullivan

`DataMigrator::Migrator.needs_migration?` should be `DataMigrator::DataMigrator.needs_migration?`

In the blob: https://github.com/ilyakatz/data-migrate/blob/1db042bfd1f29602dffe27b233e3261fa7eb7bb4/tasks/databases.rake#L13-L46 The following variables are currently assigned to the same value: https://github.com/ilyakatz/data-migrate/blob/1db042bfd1f29602dffe27b233e3261fa7eb7bb4/tasks/databases.rake#L16 https://github.com/ilyakatz/data-migrate/blob/1db042bfd1f29602dffe27b233e3261fa7eb7bb4/tasks/databases.rake#L25 I think Line 25 is instead supposed to read ``` current_data_version = DataMigrate::DataMigrator.current_version ```...

I think I found a small typo in `DataMigrate::Migration.check_pending!` (located in `/lib/data_migrate/migration.rb`) `DataMigrator::Migrator` should be `DataMigrator::DataMigrator` So this: https://github.com/ilyakatz/data-migrate/blob/1db042bfd1f29602dffe27b233e3261fa7eb7bb4/lib/data_migrate/migration.rb#L5-L7 should be: ``` def check_pending!(connection = ::ActiveRecord::Base.connection) raise ActiveRecord::PendingMigrationError if DataMigrator::Migrator.needs_migration?(connection)...