-
Notifications
You must be signed in to change notification settings - Fork 49
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
Update requirements, in particular Django 4.0 #442
Comments
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as outdated.
This comment was marked as outdated.
are you gonna use this tool https://github.com/adamchainz/django-upgrade? ;) |
Yes, I will give it try. But thanks for pointing it out. The author also points out https://django-codemod.readthedocs.io. I will upgrade these with priority: django and drf. Do you know, if all the other requirements are stil needed? Some of the dependencies have not received updates in a long time, e.g. https://github.com/chibisov/drf-extensions. I will research the dependencies and report in the corresponding PR. |
Could we also broaden some of the version constraints? For example I would suggest using this form instead |
@MyPyDavid, you can assign this issue to me. |
This comment was marked as outdated.
This comment was marked as outdated.
Thanks for picking this up. I would not use a tool though. I did the migration once for a different project and it was pretty easy. I would just run the tests and fix the problems. Are there particular problems with mptt and jsonfield and 4.2? |
I will test the tool just to get to know it. But yes, the way you propose with fixing breaking tests makes sense. No, there are no problems so far to my knowledge. It was just an observation, that there is still an open issue regarding mptt. What to do with it? |
Rationale / Begründung
Django 4.0 is out, we need to upgrade eventually. We should use this to review all other entries in
requirements.txt
.The text was updated successfully, but these errors were encountered: