Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

add support for named db:migrate:undo #387

Merged
merged 1 commit into from
Mar 25, 2017

Conversation

gotrevor
Copy link

@gotrevor gotrevor commented Nov 5, 2016

@mention-bot
Copy link

@gotrevor, thanks for your PR! By analyzing the history of the files in this pull request, we identified @sdepold, @Americas and @cusspvz to be potential reviewers.

@gotrevor
Copy link
Author

Can I get some feedback on this? How active is this repo, and how quickly should I be expecting feedback? @sdepold @Americas @cusspvz

@cusspvz
Copy link

cusspvz commented Nov 14, 2016

cc @mickhansen @janmeier

@mickhansen
Copy link

Neither me nor @janmeier have any experience with the CLI.
You likely need @sdepold to take a look.

@cusspvz
Copy link

cusspvz commented Nov 14, 2016

@gotrevor for what I remember from the cli, it seems ok, but as @mickhansen said, you'll gonna need @sdepold opinion.

@gotrevor gotrevor mentioned this pull request Dec 6, 2016
@gotrevor
Copy link
Author

gotrevor commented Jan 6, 2017

@sdepold @Americas Any chance to get movement on this?

@gotrevor
Copy link
Author

@sushantdhiman @sdepold Any chance we could get this merged?

@sushantdhiman
Copy link
Contributor

@gotrevor I am not familiar with CLI code base that much, I will review this PR on weekend and if everything looks good I will merge and release it

@sushantdhiman sushantdhiman merged commit 37cca8b into sequelize:master Mar 25, 2017
@sushantdhiman
Copy link
Contributor

Released in v2.7.0

@vidur149
Copy link

@sushantdhiman how to use that undo migration by name

@gotrevor
Copy link
Author

@vidur149
Use the --name option to give the name of the migration.
This is listed in the last line of the help:

$ ./node_modules/.bin/sequelize help:db:migrate:undo

Sequelize [Node: 6.10.1, CLI: 2.7.0, ORM: 3.30.4]

COMMANDS
    sequelize db:migrate:undo -- Reverts a migration.
DESCRIPTION
    Reverts a migration.
OPTIONS
    --env              The environment to run the command in. Default: development
    --coffee           Enables coffee script support. Default: false
    --config           The path to the config file. Default: config/config.json
    --options-path     The path to a JSON file with additional options. Default: none
    --migrations-path  The path to the migrations folder. Default: migrations
    --seeders-path     The path to the seeders folder. Default: seeders
    --models-path      The path to the models folder.Default: models
    --url              The database connection string to use. Alternative to using --config files. Default: none
    --name             Name of the migration to undo.

@gotrevor gotrevor deleted the named-db-migrate-undo branch July 31, 2017 13:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants