iggy-rs / iggy-python-client

Official Python client SDK for Iggy.rs message streaming.
https://pypi.org/project/iggy-py/
MIT License
18 stars 7 forks source link

Adding logging for step by step clarity of proccess #6

Closed bartoszkobylinski closed 1 month ago

bartoszkobylinski commented 10 months ago

Since the SDK is still in its early stages, I have added the Loguru library for better logging and tracking.

izabelakowal commented 10 months ago

@bartoszkobylinski here are a few considerations:

I'm keen to hear the perspectives of @darioalessandro and @spetz on these matters.

izabelakowal commented 10 months ago

@bartoszkobylinski also I feel that using Python's standard logging module might be the best way forward. It's straightforward and avoids extra dependencies, keeping things simpler for anyone trying out the examples.

darioalessandro commented 10 months ago

As long as clear instructions are provided and ideally we maintain dependencies via a requirements file I am ok with this change.

I do not have strong opinions regarding this topic and overall I do feel that it is a good idea to add some degree of logging.