Skip to content

Conversation

@JustusRijke
Copy link
Owner

Summary

  • Removed debug logging support (simplified to WARNING and INFO levels only)
  • Changed verbosity: int parameter to verbose: bool for simplicity
  • Removed colorlog dependency, using standard library logging instead
  • Removed save_log option
  • Refactored check functions to get logger internally instead of passing as parameter

Test plan

  • All 9 tests pass
  • Code formatted with ruff
  • Removed 1 dependency (colorlog)
  • Simplified API with bool instead of int verbosity

@codecov-commenter
Copy link

⚠️ Please install the 'codecov app svg image' to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@JustusRijke JustusRijke merged commit 30d9803 into main Jan 1, 2026
8 checks passed
@JustusRijke JustusRijke deleted the refactor_logging branch January 1, 2026 13:35
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