Merge pull request #63 from projecthydra/separate_migration

Remove db:migrate from the generator
samvera · Feb 23, 2017 · fb7a478 · fb7a478
2 parents 9f24341 + 9b742d3
commit fb7a478
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 6 deletions.
diff --git a/README.md b/README.md
@@ -111,7 +111,11 @@ The database-based minter stores minter state information in your application's
 $ rails generate active_fedora:noid:install
 ```
 
-This will create the necessary database tables and seed the database minter. To start minting identifiers with the new minter, override the AF::Noid configuration in e.g. `config/initializers/active_fedora-noid.rb`:
+This will create the necessary database migrations.
+
+Then run `rake db:migrate`
+
+To start minting identifiers with the new minter, override the AF::Noid configuration in e.g. `config/initializers/active_fedora-noid.rb`:
 
 ```ruby
 require 'active_fedora/noid'

diff --git a/lib/generators/active_fedora/noid/install_generator.rb b/lib/generators/active_fedora/noid/install_generator.rb
@@ -14,11 +14,6 @@ def banner
 
       def migrations
         rake 'active_fedora_noid_engine:install:migrations'
-        rake 'db:migrate'
-      end
-
-      def seed
-        generate 'active_fedora:noid:seed'
       end
     end
   end

diff --git a/spec/test_app_templates/lib/generators/test_app_generator.rb b/spec/test_app_templates/lib/generators/test_app_generator.rb
@@ -10,5 +10,6 @@ class TestAppGenerator < Rails::Generators::Base
 
   def install_engine
     generate 'active_fedora:noid:install'
+    rake 'db:migrate'
   end
 end