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

Make rollback more informative #104

Merged

Conversation

m-darbinyan
Copy link
Collaborator

Closes #45

Description

This PR improves the rollback process by making it more informative. The key improvements include:

  • Adds a message before each rollback indicating the gem is rolling back a phantom migration.
  • Displays the error message if something goes wrong during the rollback.
  • Prints a success message when all rollbacks are completed successfully.
  • Updates the error summary to list the failed migrations with their filename and branch, and adds some steps on how to fix them. Error messages have been removed from the summary as they are now displayed during the rollback process.

Screenshots

Success

Before After
Before Screenshot Screenshot 2024-12-09 at 01 14 06

Error

Before After
Screenshot 2024-12-09 at 01 17 10 Screenshot 2024-12-09 at 01 14 57

@ka8725
Copy link
Member

ka8725 commented Dec 15, 2024

Thanks for your contribution!

@m-darbinyan m-darbinyan force-pushed the issue-45-make-rollback-more-informative branch from 64c91fd to 18081be Compare December 16, 2024 10:38
@ka8725 ka8725 merged commit ff0040d into widefix:main Dec 16, 2024
8 checks passed
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.

Make rollback more informative
2 participants