-
Notifications
You must be signed in to change notification settings - Fork 5
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
aerich
cannot generate migrations
#133
Comments
I think there are 2 main options here if we decide to move away from
I am more familiar with SQLAlchemy personally, but piccolo also looks good (with the added benefit of built in migrations). My only concern with SQLAlchemy is I am very unfamiliar with the async capabilities of it. |
The following works for me:
|
@easybe was two minutes faster than I was... The following Aerich issue hinted for this approach: tortoise/aerich#270 Annoying - but not sure if other frameworks are that much better. I'd propose to gain some more experience with this first. |
To me this is completely natural 🤷♂️ |
I also really like Tortoise ORM, but the apparent lack of maintenance or fixes on Aerich makes it a bit annoying to do the DB migrations. At least this works, maybe should add a bit to the documentation about it. |
Re-opening this to bring discussion to a couple other options I've come across recently. It is still relatively early with this project, so swapping ORM's now is much easier than later. Option 1: SQLModel
Option 2: ormar
Overall I think both of these might be better options than tortoiseORM, and I lean towards SQLModel. If we decide we want to look into either of these, I could create a branch illustrating the differences. |
I only had a quick glance, but, |
Attempting to generate DB migrations for #132 fails to find any difference. I am starting to wonder if
aerich
is even worth using, or if there are any better options. Looking at the github page, it is not in a good state, the top issue currently is users wondering if the project is even being maintained anymore.Would like to start discussion here about other possible options or workarounds for this.
The text was updated successfully, but these errors were encountered: