Make broadcaster dependencies optional#89
Conversation
An option is available for each backend as well as an "all" option which gets you everything.
|
Ping @orweis :-) |
Tests fail I currently don't have the CPU to look into this. |
Hi I have improved the PR by:
Unfortunately tests in I don't mind looking into why they fail and potentially fixing it but I think it is outside the scope of this PR. What do you think? Br Eskild |
|
Apparently the tests failed because the All tests pass locally with |
danyi1212
left a comment
There was a problem hiding this comment.
🚨 Breaking Changes Added
LGTM
An option is available for each backend as well as an "all" option which gets you everything.
pip install fastapi_websocket_pubsubpip install fastapi_websocket_pubsub[redis]pip install fastapi_websocket_pubsub[postgres]pip install fastapi_websocket_pubsub[kafka]pip install fastapi_websocket_pubsub[all]This PR aims to close #57. I made this PR because it will enable me to install
fastapi_websocket_pubsubonpython:3.13-alpineDocker image without first installingbuild-baseandzlib-dev.Thanks a ton for this library. It's been quite useful and I appreciate that you maintain it and offer it to the community.
Br Eskild