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

ENH: Extend PdfWriter.append() to PageObjects #1704

Merged
merged 1 commit into from
Mar 18, 2023

Conversation

pubpub-zz
Copy link
Collaborator

extend append/Merge to pass PageObject list to ease some processing

@codecov
Copy link

codecov bot commented Mar 11, 2023

Codecov Report

Patch coverage: 100.00% and no project coverage change.

Comparison is base (7f1a3f8) 92.37% compared to head (4301e84) 92.37%.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1704   +/-   ##
=======================================
  Coverage   92.37%   92.37%           
=======================================
  Files          33       33           
  Lines        6480     6482    +2     
  Branches     1281     1282    +1     
=======================================
+ Hits         5986     5988    +2     
  Misses        320      320           
  Partials      174      174           
Impacted Files Coverage Δ
pypdf/_writer.py 85.79% <100.00%> (+0.02%) ⬆️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

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

@MartinThoma MartinThoma added the is-feature A feature request label Mar 12, 2023
@MartinThoma MartinThoma changed the title enh : extend PdfWriter.append() to pageObjects ENH: extend PdfWriter.append() to pageObjects Mar 12, 2023
@MartinThoma MartinThoma changed the title ENH: extend PdfWriter.append() to pageObjects ENH: Extend PdfWriter.append() to pageObjects Mar 12, 2023
@pubpub-zz
Copy link
Collaborator Author

I don't know if you are waiting for this... but it's all yours 😉

@MartinThoma
Copy link
Member

Hahaha, I don't remember if it was this PR, but I was actually missing that comment for one PR yesterday 😄 👍

@MartinThoma MartinThoma merged commit af8d7c4 into py-pdf:main Mar 18, 2023
@MartinThoma MartinThoma changed the title ENH: Extend PdfWriter.append() to pageObjects ENH: Extend PdfWriter.append() to PageObjects Mar 18, 2023
@MartinThoma
Copy link
Member

It's merged - thank you :-)

MartinThoma added a commit that referenced this pull request Mar 18, 2023
New Features (ENH)
-  Extend PdfWriter.append() to PageObjects (#1704)
-  Support qualified names in update_page_form_field_values (#1695)

Robustness (ROB)
-  Tolerate streams without length field (#1717)
-  Accept DictionaryObject in /D of NamedDestination (#1720)
-  Widths def in cmap calls IndirectObject (#1719)
@pubpub-zz pubpub-zz deleted the enh_append branch June 24, 2023 08:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
is-feature A feature request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants