-
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
🚚 Upgrade to lamindb 1.0 #927
base: main
Are you sure you want to change the base?
Conversation
django 5 is not compatible with python 3.9 |
But what are the problems? This sounds very suspicious to me. |
I would prefer not to drop before EOL. This can definitely frustrate some users, we need to check first. |
I know that a few users might not be happy about it. But I fear that without it, there is no lamindb 1.0 (I mean it'd be super hacky and ugly not to us db_default). And without lamindb 1.0 we're practically dead-locked. So, I'd accept that small nuisance given that we'll then be in a position of operating disruption free. Do you see any other solution? |
Why is |
Here is an important approval:
To declare defaults on the database level in a meaningful way. Several new fields error otherwise on the hub. And it should be declared on the database level anyway. So if we don't do it now, we have to do it in a year or so with the next migration. |
No description provided.