Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

BUG : Merger to be initialize with strict=False default #871

Merged
merged 1 commit into from
May 13, 2022

Conversation

pubpub-zz
Copy link
Collaborator

tracked in
mvicenzi/pdf_tools#2

as said in title

@MartinThoma MartinThoma merged commit a9c31a4 into py-pdf:deprecations May 13, 2022
@MartinThoma
Copy link
Member

Thank you!

@MartinThoma
Copy link
Member

... damn, that merge was actually wrong. The "deprecations" branch will be merged into main. No breaking changes should happen there.

But I do want to change the behavior for PyPDF2 2.0.0

@pubpub-zz
Copy link
Collaborator Author

But I do want to change the behavior for PyPDF2 2.0.0

@MartinThoma can you please clarify your thoughts?

@MartinThoma
Copy link
Member

I want to say that changing the default is not acceptable in PyPDF2 1.x, but I definitely want to have this change with PyPDF2 2.0.0 (I actually thought I had added it there already)

@MartinThoma
Copy link
Member

I have a nice idea! We leave the new strict=False default in the PdfMerger, but for users of the PdfFileMerger the default stays at strict=True. This way I don't need to make a major version bump while still being able to move into this direction :-) (with a notice in the changelog, of course :-) )

@MartinThoma
Copy link
Member

Here is what I meant: ba6a427

@pubpub-zz pubpub-zz deleted the FixMergeStrict branch August 8, 2022 06:56
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.

None yet

2 participants