Skip to content

Conversation

@JustusRijke
Copy link
Owner

Summary

  • Merged comparators.py into __init__.py to consolidate the main comparison logic
  • Removed the compare_pdfs function and integrated its logic directly into the diffpdf function
  • Updated CLI to call diffpdf directly instead of through a separate comparators module
  • Consolidated test files: moved API tests to test_api.py and CLI-specific tests to test_cli.py
  • Deleted obsolete files: comparators.py and test_comparators.py

Test plan

  • All 10 tests pass
  • Code formatted with ruff
  • No breaking changes to public API

@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 e2ba819 into main Jan 1, 2026
8 checks passed
@JustusRijke JustusRijke deleted the refactor_entrypoint branch January 1, 2026 13:21
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