/diesel

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

diesel lists migrations in wrong chronological order #1500

Closed
Boscop opened this Issue Jan 20, 2018 · 0 comments

Comments

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
1 participant
@Boscop
@Boscop

Boscop commented Jan 20, 2018

Diesel lists migrations in wrong chronological order after the date format changed.
It lists 2017-11-23-064836 before 20170505070309:
http://pastebin.centos.org/504376/64198071/

sgrif added a commit that referenced this issue Jan 23, 2018

@sgrif
Display the correct migration order in `diesel migration list` 
The old code was incorrectly assuming that the migration version was
equal to the file name, which caused issues on projects that have
migrations using the old timestamp format. This continues to display the
actual file name, but sorts on migration version instead.

Fixes #1500
5923cc3

@sgrif sgrif referenced this issue Jan 23, 2018

Merged

Display the correct migration order in `diesel migration list` #1506

@sgrif sgrif closed this in #1506 Jan 29, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment