-
Notifications
You must be signed in to change notification settings - Fork 57
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
refactor: summerize all migration files #4148
base: main
Are you sure you want to change the base?
Conversation
🐙 OctomindTest Report: 0/14 successful.
commit sha: b116466 |
The deployment to the dev cluster was successful. You can find the deployment here: https://4148.development.scrumlr.fra.ics.inovex.io Deployed Images
|
🐙 OctomindTest Report: 14/14 successful.
commit sha: b116466 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
is there a particular reason the migration files aren't needed anymore? isn't this nice to have in the case one needs to revert to the previous/different state of the db, e.g. in case of another branch?
Maybe because it were so many that now was a nice point to summarize them before it gets to chaotic. |
I thought it's quite hard finding out the current blueprint of our database with so many migration script files. But if there's a good reason why you shouldn't combine them into one (maybe because the project is still ongoing?), we can of course keep them. |
@brandstetterm @mateo-ivc well personally, I don't mind since I usually flatten the DB anyways before switching to a new feature branch 😂 But concerning maintainability I would like to hear @Lennart01's opinion on this |
Description
Summarized all migrations into the initial sql file
Changelog
Checklist
(Optional) Visual Changes