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

Big scary warning when there are actually no migrations to execute. #306

Closed
tchalvak opened this issue Jul 8, 2015 · 2 comments · Fixed by #480
Closed

Big scary warning when there are actually no migrations to execute. #306

tchalvak opened this issue Jul 8, 2015 · 2 comments · Fixed by #480
Milestone

Comments

@tchalvak
Copy link

tchalvak commented Jul 8, 2015

Often when I run:

php app/console doctrine:migrations:migrate

I get this:

WARNING! You are about to execute a database migration that could result in schema changes and data lost. Are you sure you wish to continue? (y/n)y

No migrations to execute.

I think it would aid usability if the command did a pre-check for 0 migrations to run and just reported that first, so that the WARNING would be a special case certainly requiring extra scrutiny.

php app/console doctrine:migrations:migrate

No migrations to execute.
@mikeSimonson
Copy link
Contributor

It would be nicer indeed, but that functionality in mixed up in the migrate function now.
You could try to refactor it out though.

@mikeSimonson mikeSimonson added this to the 1.1 milestone Jul 8, 2015
@tchalvak
Copy link
Author

Hmmm, let me see if I can figure out where it is and how it'd be refactored out.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants