celery / django-celery-results

Celery result back end with django
Other
668 stars 206 forks source link

add django 5.0 to CI/test matrix and project classifiers #425

Closed charlesobrien closed 3 months ago

charlesobrien commented 4 months ago

Adding Django 5.0 to CI/test matrix and bumping some other Django minor versions

charlesobrien commented 4 months ago

please check the build failures

@auvipy Build should be passing now. The primary issue was pytest 8 was failing (all prior builds were running against pytest 7 and the pytest 8 release in Jan 2024 had breaking changes). I had to apply a <8 version constraint and the build would pass.

browniebroke commented 3 months ago

I'm not a maintainer but may I also suggest to update the classifiers:

https://github.com/celery/django-celery-results/blob/cb9448976bc51865922319c91b6c2218859ba80d/setup.py#L44-L48

Opened a PR against your fork to add it https://github.com/charlesobrien/django-celery-results/pull/1

charlesobrien commented 3 months ago

I'm not a maintainer but may I also suggest to update the classifiers: https://github.com/celery/django-celery-results/blob/cb9448976bc51865922319c91b6c2218859ba80d/setup.py#L44-L48

Opened a PR against your fork to add it charlesobrien#1

Thanks; I've merged that!

charlesobrien commented 3 months ago

@auvipy Following up. Anything else needed for merge?