Codecov Report

Merging #913 (85dd0c0) into main (bb68a4b) will decrease coverage by 0.07%.
The diff coverage is 64.28%.

@@            Coverage Diff             @@
##             main     #913      +/-   ##
==========================================
- Coverage   77.89%   77.81%   -0.08%     
==========================================
  Files          16       16              
  Lines        4324     4327       +3     
  Branches      813      812       -1     
==========================================
- Hits         3368     3367       -1     
- Misses        784      788       +4     
  Partials      172      172              

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update bb68a4b...85dd0c0. Read the comment docs.

Thank you for the PR!

I was first a bit uncertain, because the new name is less clear than the old one. However, the documentation is clear and the name is brief + consistent with the Transformation class.

wasn't really sure what "type" to give this PR, whether it should be DEP, ENH, MAINT or some other one

I'm not going to get religious on that 😄

I try to keep this logic:

• ENH: Anything that allows the users to do something new
• MAINT: Third-party package updates, fixing typos, often things that don't quite fit in other categories
• DEP: Deprecations

If something contains deprecations + other stuff (e.g. fixing typos, adding a new function) I would still go with DEP as it requires users attention. I've set the order like this in the changelog: https://github.com/py-pdf/PyPDF2/blob/main/make_changelog.py#L66

The parts that are most important for users to notice should be on the top in the changelog message