multi_ar 1.0.0.pre.rc2 → 1.0.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: d54d942aefe519b7959ea7d8207c03b304c437f2
-  data.tar.gz: abf83b48064b3e378000ff19b05642c7986919ee
+  metadata.gz: 3754772cf591a147943b8cfb7b83c85254872110
+  data.tar.gz: 620350cf731cc914a5620248dbe1d632406bd34b
 SHA512:
-  metadata.gz: 7d4b9241137a3c3bc78f0b9c0c8b04fa13701ef08250596c5fb1b54a95063049fa950c17298ecc64f92afc81bdef7fcbe9a69108b1dac8b0f7986404b500b756
-  data.tar.gz: ea38c3fb2c44b79041ff14e94988e1c7494a57091a7f5bf3fbedf74ebd9ae62dffbe975b8e5ccee222d11a301ce05df973fbfbdfe679d937dc6cd7423e0617c5
+  metadata.gz: 5af7a1f88b87c54891429d9f47cff6a4c198c5c50f85c7a47cae606417080fd1f5dc78adafd37ef3fe4515aa035657f5895c5e92fa6f59178f1986d18b327898
+  data.tar.gz: f89187f075e2ad595d6d6ea60e12c98888ef0119ef6481427d100998e5a7635ad706a29b54616f9415bee763715498ed448f6ab7053640d944c94709e2bf3d31

data/lib/multi_ar/interface.rb

@@ -77,10 +77,9 @@ module MultiAR
 
       if @migration_framework == true
         p.opt "all_rake_tasks",     "List all Rake tasks, not only commented ones",                 short: "A", type: :flag
-        # TODO: what for this is...?
-        #p.opt "common_migrations",  "Run the migrations bundled with the gem, for common databases.",           type: :flag,    default: true
         # TODO: not implemented currently, do we really need this?
         #p.opt "list_databases",     "Lists databases that contains migrations in the gem",                      type: :flag
+        # TODO: should we do migration_dirs here too instead?
         p.opt "migration_dir",      "The directory where migrations for databases are read from",               type: :string,  default: "db/migrate"
         p.opt "task",               "Rake task to execute",                                         short: "t", type: :string
         p.opt "tasks",              "List available Rake tasks",                                    short: "T", type: :flag
@@ -130,12 +129,12 @@ module MultiAR
     private
 
     def init_multi_ar
-      @multi_ar = MultiAR::MultiAR.new common_migrations: @opts["common_migrations"],
-          config: @opts["config"],
+      @multi_ar = MultiAR.new config: @opts["config"],
           databases: @opts["databases"],
           db_config: @opts["db_config"],
-          environment: @opts["environment"],
-          migration_dir: @opts["migration_dir"]
+          environment: @opts["environment"]
+
+      MultiAR.add_migration_dir @opts["migration_dir"]
     end
 
     # @note This method will always quit the application or raise another exception for errors. Catch SystemExit if that’s not good for you.

data/lib/multi_ar/version.rb

@@ -1,4 +1,4 @@
 
 module MultiAR
-  VERSION = "1.0.0-rc2"
+  VERSION = "1.0.0"
 end

data/lib/multi_ar.rb

@@ -26,10 +26,10 @@ module MultiAR
 
     # @param databases array of available databases
     # @todo config file is overriding parameters passed here... I think it should be other way around, but need more custom logic for that :/
-    def initialize databases:, environment: "development", config: "config/multi_ar.yaml", db_config: "config/database.yaml", common_migrations: true
+    def initialize databases:, environment: "development", config: "config/multi_ar.yaml", db_config: "config/database.yaml"
 
       # first load config
-      if File.exist? config
+      if not config.nil? and File.exist? config
         require "psych"
         config = Psych.load_file config
         b = binding
@@ -39,7 +39,7 @@ module MultiAR
       end
 
       # then check that we have data in format we want it to be
-      raise "#{db_config} is not valid path to a file. Try specifying --db-config <path> or configuring it in the configuration file." unless File.exist?(db_config)
+      raise "#{db_config} is not valid path to a file. Try specifying --db-config <path> or configuring it in the configuration file." if db_config.nil? or !File.exist?(db_config)
       raise "databases is not responding to :each. Try passing passing --databases <database> or configuring it in the configuration file." unless databases.respond_to? :each
 
       @databases = databases
@@ -58,7 +58,6 @@ module MultiAR
 
       Rake::Tasks.databases = databases
       Rake::Tasks.environment = environment
-      Rake::Tasks.common_migrations = common_migrations
       Rake::Tasks.define
 
       MultiAR.app = self

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: multi_ar
 version: !ruby/object:Gem::Version
-  version: 1.0.0.pre.rc2
+  version: 1.0.0
 platform: ruby
 authors:
 - Samu Voutilainen
@@ -168,9 +168,9 @@ required_ruby_version: !ruby/object:Gem::Requirement
       version: '0'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
-  - - ">"
+  - - ">="
     - !ruby/object:Gem::Version
-      version: 1.3.1
+      version: '0'
 requirements: []
 rubyforge_project: 
 rubygems_version: 2.4.8