Skip to content

Print warnings when there is a skipped migration or a missing migration script #125

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.

Sign up for GitHub

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

Jump to bottom
Merged
harawata merged 1 commit into from
Aug 9, 2018
Merged

Print warnings when there is a skipped migration or a missing migration script #125

harawata merged 1 commit into from
Aug 9, 2018

Conversation

@harawata
Copy link
Member

@harawata harawata commented Aug 8, 2018

On up/down operation, Migrations will print warnings if...

  • there is a migration script that 1) is not applied to the database and 2) has a lower ID than the latest applied migration (= skipped migration).
  • there is an entry in the changelog table which has no corresponding script file in the scripts directory (= missing script).

This should fix #123 .

@harawata harawata added this to the 3.3.5 milestone Aug 8, 2018
@harawata harawata self-assigned this Aug 8, 2018
@harawata harawata merged commit 6007324 into mybatis:master Aug 9, 2018
@harawata harawata mentioned this pull request Aug 9, 2018
Closed
@harawata harawata mentioned this pull request Jun 20, 2019
Merged
harawata added a commit that referenced this pull request Jun 22, 2019
@harawata
refs #125 output warnings at the end for better visibility
81e7054
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

@harawata harawata

Labels

enhancement

Projects

None yet

Milestone

3.3.5

Development

Successfully merging this pull request may close these issues.

Skipped installation gives result as success. Can it be changed to 'Failure' or at-least a Warning message?

1 participant

@harawata