Skip to content

Implement logging using built-in logging library#4

Merged
ecarrenolozano merged 3 commits into
mainfrom
feat/migration-logger
Mar 9, 2026
Merged

Implement logging using built-in logging library#4
ecarrenolozano merged 3 commits into
mainfrom
feat/migration-logger

Conversation

@ecarrenolozano

Copy link
Copy Markdown
Contributor

This pull request refactors logging throughout the cache_manager package to use Python's standard logging module instead of the custom _log function. The changes improve consistency, flexibility, and integration with external logging systems. The main updates include replacing all _log calls with logger.debug, introducing module-level loggers, and removing _log imports.

Logging refactor in cache_manager/_cache.py:

Logging refactor in cache_manager/_freshness.py:

  • Replaced _log with logger.debug in the _check_etag function and removed the _log import. [1] [2]
  • Added a module-level logger to cache_manager/_freshness.py.

API cleanup:

  • Removed the log and _log exports from cache_manager/__init__.py to reflect the removal of the custom logging function.

@ecarrenolozano ecarrenolozano merged commit 48a83eb into main Mar 9, 2026
8 checks passed
@ecarrenolozano ecarrenolozano deleted the feat/migration-logger branch March 9, 2026 13:32
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.

1 participant