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

MAINT: Replace PyPDF2 by pypdf #307

Merged
merged 4 commits into from
Feb 7, 2023

Conversation

MartinThoma
Copy link
Contributor

@MartinThoma MartinThoma commented Jun 8, 2022

PyPDF2 changed the syntax and deprecated Python 3.5 and older. As camelot requirest 3.6 or newer, there is no reason not to upgrade.

Fixes #339

@NikilXYZ
Copy link

NikilXYZ commented Jan 2, 2023

Why has this not been merged into main (master) yet?

@MartinThoma
Copy link
Contributor Author

@vinayak-mehta

sanecz added a commit to sanecz/python-msr-doc that referenced this pull request Jan 2, 2023
camelot/handlers.py Outdated Show resolved Hide resolved
camelot/handlers.py Outdated Show resolved Hide resolved
@MartinThoma
Copy link
Contributor Author

@dimitern Can you merge this PR?

@giulioindev
Copy link

Can this PR be merged please?

@emc415
Copy link

emc415 commented Feb 6, 2023

I'm also hoping this gets merged soon!

@dimitern dimitern merged commit 4a47728 into camelot-dev:master Feb 7, 2023
This was referenced Feb 7, 2023
@MartinThoma MartinThoma deleted the pypdf2-update branch February 7, 2023 19:05
@martin-thoma
Copy link

@dimitern I've seen that this wasn't released so far https://pypi.org/project/camelot-py/#history - do you happen to know when it will be released?

@vinayak-mehta
Copy link
Member

I'm releasing it right now, sorry for the late reply here

@vinayak-mehta
Copy link
Member

@MT-Cash the release is now live: https://pypi.org/project/camelot-py/

Thank you for the PR!

@MartinThoma MartinThoma changed the title MAINT: Update PyPDF2 version MAINT: Replace PyPDF2 by pypdf Feb 26, 2023
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.

PdfFileReader is deprecated and was removed in PyPDF2 3.0.0
8 participants