search

ovh / celery-director

Simple and rapid framework to build workflows with Celery
https://ovh.github.io/celery-director/
BSD 3-Clause "New" or "Revised" License
534 stars 58 forks source link

Bump up requirements #118

Closed fmalina closed 2 years ago

fmalina commented 2 years ago

Signed-off-by: Frantisek Malina fmalina@pm.me

fmalina commented 2 years ago

Made a separate docs PR. How about bumping up requirements to latest? ... celery==4.4.7 ... 5.2 click==7.0 ... 8.0 flower==0.9.3 ... 1.0.0 kombu==4.6.11 ... 5.1

fmalina commented 2 years ago

Okay it's not as simple as that, for my reference here's incompatibility with newer celery

f@plainjane workflows % celery --version
5.2.0b2 (dawn-chorus)
f@plainjane workflows % director celery worker
You are using `-A` as an option of the worker sub-command:
celery worker -A celeryapp <...>

The support for this usage was removed in Celery 5.0. Instead you should use `-A` as a global option:
celery -A celeryapp worker <...>
Usage: celery worker [OPTIONS]
Try 'celery worker --help' for help.

Error: No such option: -A

And this is all implemented in #115 (reason for closing)