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

Missing migrations #73

Closed
Stepych opened this issue Oct 4, 2017 · 1 comment
Closed

Missing migrations #73

Stepych opened this issue Oct 4, 2017 · 1 comment

Comments

@Stepych
Copy link

Stepych commented Oct 4, 2017

The same issue as mattes/migrate#237 - applying "missing" migrations: now if it's less or equal to than the version in the database, it is ignored.

@VojtechVitek
Copy link
Collaborator

We explicitly prefer to error out if there's any gap in the migrations. If goose skips those migrations right now, it's probably a bug and we should fix that and make it an error.

We don't want to apply any migrations that are out of the sequence number order.

There shouldn't be any "skipped" migration if you version your migrations properly. See https://github.com/pressly/goose#hybrid-versioning for further explanation.

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 a pull request may close this issue.

2 participants