Skip to content

feat: offer standard logging module usage.#31

Merged
sky-sharma merged 2 commits intoClearBlade:masterfrom
swilcox:feature/use-logging
Aug 4, 2025
Merged

feat: offer standard logging module usage.#31
sky-sharma merged 2 commits intoClearBlade:masterfrom
swilcox:feature/use-logging

Conversation

@swilcox
Copy link
Contributor

@swilcox swilcox commented Sep 5, 2024

Added a new flag to allow using python's standard library logging module.

Default is False so that existing behavior of the library is unchanged.

When set to True and the other debug options are set to True, it outputs to logging with named loggers Mqtt and CB (to stay consistent to the print statements). This allows for more precise control over logging output/format/filtering using the standard python logging configuration.

README.md is also updated to reflect the new setting.

@swilcox
Copy link
Contributor Author

swilcox commented Sep 6, 2024

I updated the README and code to use USE_LOGGING instead of USE_LOGS because #namingthings ... USE_LOGGING is more consistent with the fact that we're using the logging module of python instead of print statements.

@sky-sharma sky-sharma requested a review from WorkForPizza August 3, 2025 05:59
@sky-sharma sky-sharma merged commit a41c465 into ClearBlade:master Aug 4, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants