Skip to content

BUG: Use remove_orphans in compress_identical_objects#3310

Open
j-t-1 wants to merge 10 commits intopy-pdf:mainfrom
j-t-1:compress
Open

BUG: Use remove_orphans in compress_identical_objects#3310
j-t-1 wants to merge 10 commits intopy-pdf:mainfrom
j-t-1:compress

Conversation

@j-t-1
Copy link
Contributor

@j-t-1 j-t-1 commented Jun 11, 2025

Issue #3306: PdfWriter.compress_identical_objects ignored remove_orphans. Correct for this. Also deprecate_with_replacement remove_orphans to remove_unreferenced.

j-t-1 added 4 commits June 10, 2025 18:39
Issue py-pdf#3306: PdfWriter.compress_identical_objects ignored remove_orphans. Correct for this. Also deprecate_with_replacement remove_orphans to remove_unreferenced.
@codecov
Copy link

codecov bot commented Jun 11, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 97.35%. Comparing base (4670513) to head (e507b42).

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #3310   +/-   ##
=======================================
  Coverage   97.35%   97.35%           
=======================================
  Files          55       55           
  Lines        9916     9920    +4     
  Branches     1814     1816    +2     
=======================================
+ Hits         9654     9658    +4     
  Misses        152      152           
  Partials      110      110           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@stefan6419846 stefan6419846 added the needs-change The PR/issue cannot be handled as issue and needs to be improved label Jul 31, 2025
@j-t-1
Copy link
Contributor Author

j-t-1 commented Feb 26, 2026

Fixed deprecate_with_replacement. Ready for code review.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

needs-change The PR/issue cannot be handled as issue and needs to be improved

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants