Summary
Several modules in the package use print() with manual [ERROR]/[WARN] prefixes instead of the standard logging module. This bypasses the logging configuration established in cli.py (level filtering, formatting, handler routing) and makes the output uncontrollable for downstream consumers.
Affected files:
clients/github_api.py — 4 error prints in get_latest_github_tag and get_commit_hash_for_tagclients/registry_api.py — 2 error printsupdaters/github_source.py — 1 info-level printupdaters/registry_source.py — 1 error printdiscovery.py — 1 warning print + loop
Proposed fix:
Add a module-level logger to each affected file (as already done in app.py):
logger = logging.getLogger(__name__)
Then replace each print(...) with the appropriate logger.error(...), logger.warning(...) or logger.info(...) call, removing the manual [ERROR]/[WARN] prefixes.
Out of scope: The print calls in cli.py's exception handlers — these are at the entry point and are acceptable as is.
Summary
Several modules in the package use
print()with manual[ERROR]/[WARN]prefixes instead of the standardloggingmodule. This bypasses the logging configuration established in cli.py (level filtering, formatting, handler routing) and makes the output uncontrollable for downstream consumers.Affected files:
clients/github_api.py— 4 error prints inget_latest_github_tagandget_commit_hash_for_tagclients/registry_api.py— 2 error printsupdaters/github_source.py— 1 info-level printupdaters/registry_source.py— 1 error printdiscovery.py— 1 warning print + loopProposed fix:
Add a module-level logger to each affected file (as already done in
app.py):Then replace each
print(...)with the appropriatelogger.error(...),logger.warning(...)orlogger.info(...)call, removing the manual[ERROR]/[WARN]prefixes.Out of scope: The
printcalls incli.py's exception handlers — these are at the entry point and are acceptable as is.