search

caktus / margarita

A collection of delicious Salt states for Django project deployments.
BSD 3-Clause "New" or "Revised" License
34 stars 8 forks source link

Always migrate during deploy. #140

Closed kmtracey closed 7 years ago

kmtracey commented 7 years ago

migrate --list does not exist in Django 1.10, showmigrations does not exist in Django 1.7 so supporting a wide range of Django levels plus only running migrate when absolutely necessary is considerably harder than simply running migrate during deploy.

ironfroggy commented 7 years ago

I can think of no good reason not to do this

ironfroggy commented 7 years ago

come to think of it, now I can't remember why we ever required the check in the first place!

dpoirier commented 7 years ago

:guitar:

mlavin commented 7 years ago

The original behavior was almost certainly something I added because it seemed like a good idea at the time ("efficiency") but once again the simpler thing is here is better.