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: File expansion when updating with Page Contents #1906

Merged
merged 8 commits into from
Jun 23, 2023

Conversation

pubpub-zz
Copy link
Collaborator

closes #1897

@MartinThoma
Copy link
Member

You have to install fpdf2. I can do that later and rebase your PR :-)

@MartinThoma
Copy link
Member

Thank you for the quick fix 🙏

pypdf/_page.py Outdated Show resolved Hide resolved
pypdf/_page.py Outdated Show resolved Hide resolved
pypdf/_page.py Outdated Show resolved Hide resolved
pubpub-zz and others added 4 commits June 22, 2023 18:49
Co-authored-by: Martin Thoma <[email protected]>
Co-authored-by: Martin Thoma <[email protected]>
Co-authored-by: Martin Thoma <[email protected]>
@codecov
Copy link

codecov bot commented Jun 22, 2023

Codecov Report

Patch coverage: 100.00% and project coverage change: +0.01 🎉

Comparison is base (449b8e5) 93.81% compared to head (b0c497a) 93.83%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1906      +/-   ##
==========================================
+ Coverage   93.81%   93.83%   +0.01%     
==========================================
  Files          34       34              
  Lines        6922     6943      +21     
  Branches     1365     1370       +5     
==========================================
+ Hits         6494     6515      +21     
- Misses        280      282       +2     
+ Partials      148      146       -2     
Impacted Files Coverage Δ
pypdf/_page.py 93.58% <100.00%> (+0.65%) ⬆️
pypdf/_writer.py 88.52% <100.00%> (-0.33%) ⬇️

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

@pubpub-zz
Copy link
Collaborator Author

Now coverage to be improve. give me a coupe of hours for dinner 😋

@pubpub-zz
Copy link
Collaborator Author

looks good now!
@MartinThoma
all your 😁

@MartinThoma MartinThoma changed the title BUG : file expansion when updating with Page Contents BUG: File expansion when updating with Page Contents Jun 23, 2023
@MartinThoma MartinThoma merged commit fca29c7 into py-pdf:main Jun 23, 2023
@MartinThoma
Copy link
Member

Very nice! Thank you :-)

MartinThoma added a commit that referenced this pull request Jun 23, 2023
New Features (ENH):
-  Add page_number property (#1856)

Bug Fixes (BUG):
- File expansion when updating with Page Contents (#1906)
- Missing Alternate in indexed/ICCbased colorspaces (#1896)

[Full Changelog](3.10.0...3.11.0)
@pubpub-zz pubpub-zz deleted the upd_contents branch September 2, 2023 09:51
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.

File Size Increase from pypdf==3.8.1 to pypdf==3.9.0 when watermarking
2 participants