-
Notifications
You must be signed in to change notification settings - Fork 2
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
Support migrations
#38
Comments
this can be done with other libraries keep DRY |
@rogerpadilla any particular library in mind with typescript support? |
hi @ristiisa, yeah, I have umzug in mind for this: https://github.com/sequelize/umzug. Thanks for asking. How did you reach here? are you using or interested somehow in nukak? any feedback/thoughts/comments? thanks again. |
|
@tasansga for some reason i missed your message before, apologies about that. since then i have changed my mind and i think it could be better to base it on TypeORM's migrations style here https://typeorm.io/migrations. about adding an example on how to use it, i'd prefer only to do that after we implement the support for it or it can be unclear for newcomers to nukak who would expect migrations to work if they see the example. additionally, i have been thinking about renaming 'nukak' to a potentially more SEO-friendly name, e.g. 'torm' or something around that. please let me know your thoughts, i always do appreciate them. |
I have umzug in mind for this: https://github.com/sequelize/umzug.
But we can look at different options.
The text was updated successfully, but these errors were encountered: