Skip to content
This repository has been archived by the owner on Mar 15, 2018. It is now read-only.

fix for bug 997294 #1950

Closed
wants to merge 1 commit into from
Closed

fix for bug 997294 #1950

wants to merge 1 commit into from

Conversation

dethe
Copy link
Contributor

@dethe dethe commented Apr 16, 2014

Fix duplicate migration numbers breaking migration.

@andymckay
Copy link
Contributor

but that will mean the last migration will get bumped up by one and hence run again? will it do something bad if gets run again?

@andymckay
Copy link
Contributor

did you mean to change migration #70 ?

@dethe
Copy link
Contributor Author

dethe commented Apr 16, 2014

No, thanks for catching that!

@andymckay
Copy link
Contributor

759 looks a like a migration that can be run multiple times, would suggest moving it to the highest number, letting it run again

@andymckay
Copy link
Contributor

759-apps-edit that is

@dethe
Copy link
Contributor Author

dethe commented Apr 16, 2014

On my system none of these migrations were getting run, but I guess folks who ran before the conflict could have migrations that run more than once.

But your suggestion sounds better. I will undo this change and just move 759-apps-edit

@dethe
Copy link
Contributor Author

dethe commented Apr 16, 2014

Abandoning this PR, will make a new one.

@dethe dethe closed this Apr 16, 2014
@dethe dethe deleted the bump_migration_versions branch April 16, 2014 17:57
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants