Skip to content

django 5.2 compatibility #495

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

Merged
merged 2 commits into from
Apr 4, 2025
Merged

django 5.2 compatibility #495

merged 2 commits into from
Apr 4, 2025

Conversation

nosnilmot
Copy link
Contributor

Closes #494

@nosnilmot
Copy link
Contributor Author

I don't think test failures are related to this change and are probably pre-existing problems with CI test configuration. pytest locally works with many combinations of python & django versions.

@Suor
Copy link
Owner

Suor commented Apr 3, 2025

I don't think test failures are related to this change and are probably pre-existing problems with CI test configuration. pytest locally works with many combinations of python & django versions.

Maybe not, some table is not created but I can't quickly understand why.

@nosnilmot nosnilmot force-pushed the django-5.2 branch 3 times, most recently from 9360a7a to b02ac4b Compare April 4, 2025 07:24
update tests to include Python 3.13 and use by default; and drop Python
3.7 (no longer supported, incompatible with latest pytest)
require pytest-django 4.11.1 to address DB access issues in 4.11.0
ensure same version of pytest & pytest-django used for local tests and
CI
@nosnilmot
Copy link
Contributor Author

I can't quickly understand why.

The problem was introduced by pytest-django 4.11.0, and is fixed in 4.11.1

I've fixed that, and taken the liberty to add Python 3.13 to supported versions (and removed 3.7 as unsupported and incompatible with latest pytest/pytest-django)

@Suor
Copy link
Owner

Suor commented Apr 4, 2025

Oh, thanks a lot.

@Suor Suor merged commit 3de9ff6 into Suor:master Apr 4, 2025
9 checks passed
@nosnilmot
Copy link
Contributor Author

@Suor thanks for merging this, is there a planned date for a new django-cacheops release to include support for Django 5.2?

@Suor
Copy link
Owner

Suor commented Apr 7, 2025

I usually let code to cool a little while before releasing it, but I was thinking about releasing soon.

@Suor
Copy link
Owner

Suor commented Apr 20, 2025

Ok, released 7.2

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

SubqueryConstraint was removed from Django 5.2
2 participants