-
Notifications
You must be signed in to change notification settings - Fork 13
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
python 3.13 #103
python 3.13 #103
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Flags with carried forward coverage won't be shown. Click here to find out more. |
8724d9b
to
ed21580
Compare
Btw, do not forget to update pyproject as well |
Btw, what is the actual project you are using? that-depends or modern-di? |
@Lancetnik for projects with sqlalchemy I'm moving to modern-di, because it's easier to manage orm sessions with scopes. For simpler projects, where scopes are not needed, that-depends is used |
Sounds like a useless complexity, isn't it? Why are you using that-depends if modern-di is better? Do you thinking about slowly deprecation of the current lib to prior to modern-di? |
I'm panning to support both libraries, but with focus on promoting modern-di.
to be fair, I don't write new projects with that-depends. I had migrated some services to test modern-di in production and it works great. I think, some day we will migrate other services to modern-di as well |
b582add
to
a078993
Compare
No description provided.