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

Fix database migrations #246

Merged
merged 4 commits into from
May 6, 2019
Merged

Conversation

moseb
Copy link
Contributor

@moseb moseb commented May 2, 2019

Meant to address issue #217

@auvipy
Copy link
Member

auvipy commented May 2, 2019

Looking forward to it

@moseb moseb force-pushed the issue-217-fix-migrations branch 11 times, most recently from e5c4d03 to 5fdf91a Compare May 3, 2019 15:49
@@ -1,95 +0,0 @@
# Generated by Django 2.1.2 on 2018-10-12 14:18
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

So you deleted the squashed migration here, but that shipped in 1.4.0. I'm surprised that it didn't complain about it being in the migrations table later on upgrade, but I guess if it works it works /shrug

@moseb moseb force-pushed the issue-217-fix-migrations branch from 5fdf91a to 8032046 Compare May 3, 2019 16:31
@moseb moseb changed the title [W.I.P.] Fix migrations Fix migrations May 3, 2019
@liquidpele
Copy link
Contributor

@moseb looks like flake8 is failing now due to the comment... but I'm not sure what BEGIN ex and END ex mean.

Maybe say something like

# depends on higher numbers due to a squashed migration
# that was later removed due to migration issues it caused

@auvipy auvipy changed the title Fix migrations Fixed migrations May 3, 2019
@moseb moseb force-pushed the issue-217-fix-migrations branch from 8032046 to 1d4c465 Compare May 6, 2019 09:23
@moseb moseb changed the title Fixed migrations Fix database migrations May 6, 2019
@auvipy auvipy merged commit 786290c into celery:master May 6, 2019
@moseb
Copy link
Contributor Author

moseb commented May 6, 2019

Thanks!

@moseb moseb mentioned this pull request May 6, 2019
@liquidpele
Copy link
Contributor

Awesome, thanks @moseb for the help!

@auvipy auvipy added this to the 1.5.0 milestone May 8, 2019
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.

4 participants