refactor(logging): replace package print statements with module-level…

[d-lazenby]

Use module-level logging instead of print in package modules

Closes #61

Summary

• Replaces package print statements with module-level logger calls for consistent logging behavior and formatting.
• Updates discovery warning assertions to use log capture.
• Keeps CLI fallback error prints unchanged.

[Copilot]

Pull request overview

Refactors package modules to use module-level logging instead of print(), aligning output with the CLI’s logging configuration and enabling tests to assert warnings via log capture.

Changes:

• Replaced print() calls in package modules with logger.info(...), logger.warning(...), logger.error(...), and logger.exception(...).
• Updated discovery warning test to assert via caplog instead of capsys.
• Added module-level logger = logging.getLogger(__name__) to affected modules.

Reviewed changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated 1 comment.

Show a summary per file

File	Description
tests/test_discovery.py	Switches unmanaged-module warning assertions from stdout capture to log capture.
src/update_tf_modules/discovery.py	Uses logger.warning(...) for unmanaged-module warnings instead of print().
src/update_tf_modules/clients/github_api.py	Replaces GitHub client error print() calls with structured logger calls.
src/update_tf_modules/clients/registry_api.py	Replaces Registry client error print() calls with structured logger calls.
src/update_tf_modules/updaters/github_source.py	Logs GitHub module update messages via logger.info(...) instead of print().
src/update_tf_modules/updaters/registry_source.py	Logs registry module update messages via logger.info(...) instead of print().

Suppressed comments (1)

src/update_tf_modules/clients/registry_api.py:61

• There is trailing whitespace on the blank line after the exception handler (line 59). This can cause lint failures and noisy diffs.

        return None
    

def semver_key(version: str) -> tuple[tuple[int, int | str], ...]:

---

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.