search

celery / django-celery-results

Celery result back end with django
Other
684 stars 203 forks source link

Change default_auto_field to BigAutoField #426

Open browniebroke opened 6 months ago

browniebroke commented 6 months ago

Fix #307

Or at least make the issue much less frequent...

browniebroke commented 6 months ago

I don't understand the test failure, they seem unrelated with my changes... Are they broken due to an external factor or am I missing something obvious?

charlesobrien commented 6 months ago

@browniebroke They're failing because Pytest was unconstrained and breaking changes in Pytest 8 (released in Jan) are blowing up the tests.

My PR https://github.com/celery/django-celery-results/pull/425 to add Django 5 to CI also places a constraint on pytest. Hoping it might be able to get reviewed/merged soon.

browniebroke commented 6 months ago

Thanks a lot! Will wait for a bit then.