-
Notifications
You must be signed in to change notification settings - Fork 12
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
add django test matrix #40
add django test matrix #40
Conversation
to be dj5 ready
@alexandernst @NachE ready to merge. good of you? |
This reverts commit 0d08592.
are these changes enough to consider django 5 supported? /cc @alexandernst |
I believe so. At least tests pass. Haven't run it in PROD env yet. |
Up to Django 4.2 it wasn't possible to have multiple storages (not in an easy way). Starting with 4.2 django encourages users to actually use multiple storages, that's why we shouldn't asume the |
see 4f7df0c. good of for you? |
@NachE @alexandernst good to merge? any improvements reasonable? |
I think we're good! Feel free to merge 🙏 |
due to related Django settings deprecation in v4.2 let's ensure all versions keep working
addresses #32