Skip to content

Conversation

j-t-1
Copy link
Contributor

@j-t-1 j-t-1 commented Jul 29, 2025

No description provided.

Copy link

codecov bot commented Jul 29, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 96.95%. Comparing base (01c98a5) to head (4c8dc18).
⚠️ Report is 5 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #3406      +/-   ##
==========================================
- Coverage   96.96%   96.95%   -0.01%     
==========================================
  Files          55       55              
  Lines        9343     9340       -3     
  Branches     1708     1708              
==========================================
- Hits         9059     9056       -3     
  Misses        169      169              
  Partials      115      115              

☔ 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
Copy link
Collaborator

We cannot remove a deprecation without issuing a major release. I plan to do a major release and open an issue for it - until then I will put this PR on hold.

@stefan6419846 stefan6419846 added on-hold PR requests that need clarification before they can be merged.A comment must give details and removed on-hold PR requests that need clarification before they can be merged.A comment must give details labels Jul 30, 2025
@j-t-1 j-t-1 changed the title MAINT: Remove deprecation MAINT: Remove erroneous comment Aug 1, 2025
@stefan6419846 stefan6419846 merged commit ad85a22 into py-pdf:main Aug 4, 2025
17 checks passed
@j-t-1 j-t-1 deleted the deprecation branch August 5, 2025 07:43
j-t-1 added a commit to j-t-1/pypdf that referenced this pull request Aug 5, 2025
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.

2 participants