refactor(transition): remove transitioner entirely

since it is not needed anymore. BREAKING CHANGE: the transitioner will disappear from the API entirely. The need for it disappeared since it was there to help with the migration from very old migration schemas to the new ones that did not support very old globals and async the library provided by db-migrate itself. Users that for some reason need that can get it from the v0.11.x versions and then migrate to the newest version afterwards. Refers to #627 Signed-off-by: Tobias Gurtzick <[email protected]>
db-migrate · Apr 16, 2020 · a0432f1 · a0432f1
1 parent 8874fa6
commit a0432f1
Show file tree

Hide file tree

Showing 12 changed files with 49 additions and 479 deletions.
diff --git a/api.js b/api.js
@@ -320,13 +320,6 @@ dbmigrate.prototype = {
     return log.silence(isSilent);
   },
 
-  /**
-   * Transition migrations to the latest defined protocol.
-   */
-  transition: function () {
-    load('transition')(this.internals);
-  },
-
   /**
    * Creates a correctly formatted migration
    */

diff --git a/lib/commands/run.js b/lib/commands/run.js
@@ -5,16 +5,12 @@ var yargs = require('yargs');
 
 function run (internals, config) {
   const { load } = internals;
-  const transition = load('transition');
   var action = internals.argv._.shift();
   var folder = action.split(':');
 
   action = folder[0];
 
   switch (action) {
-    case 'transition':
-      transition(internals);
-      break;
     case 'create':
       if (folder[1]) {
         internals.matching = folder[1];
@@ -114,8 +110,7 @@ function run (internals, config) {
         );
       } else {
         log.error(
-          'Invalid Action: Must be [up|down|check|create|reset|sync|' +
-            'db|transition].'
+          'Invalid Action: Must be [up|down|check|create|reset|sync|db].'
         );
         yargs.showHelp();
         process.exit(1);

diff --git a/lib/commands/set-default-argv.js b/lib/commands/set-default-argv.js
@@ -23,7 +23,7 @@ module.exports = function (internals, isModule) {
     internals.argv = yargs
       .default(defaultConfig)
       .usage(
-        'Usage: db-migrate [up|down|check|reset|sync|create|db|transition] ' +
+        'Usage: db-migrate [up|down|check|reset|sync|create|db] ' +
           '[[dbname/]migrationName|all] [options]'
       )
       .describe(

diff --git a/lib/commands/transition.js b/lib/commands/transition.js
diff --git a/lib/transitions/1.js b/lib/transitions/1.js
diff --git a/lib/transitions/ask.js b/lib/transitions/ask.js
diff --git a/lib/transitions/snippets/setup.sjs b/lib/transitions/snippets/setup.sjs
diff --git a/lib/transitions/transitioner.js b/lib/transitions/transitioner.js
diff --git a/lib/transitions/try-require.js b/lib/transitions/try-require.js
diff --git a/lib/transitions/update-version.js b/lib/transitions/update-version.js