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 : Named Dest in PDF1.1(#1173) #1174

Merged
merged 1 commit into from Jul 27, 2022
Merged

Conversation

pubpub-zz
Copy link
Collaborator

Named dest are stored in a dictionnary in PDF 1.1
TODO : a test for named destination stored in name tree should be added to improve coverage
[#1173)

 Named dest are stored in a dictionnary in PDF 1.1
TODO : a test for named destination stored in name tree should be added to improve coverage
@codecov
Copy link

codecov bot commented Jul 27, 2022

Codecov Report

Merging #1174 (94b1859) into main (d8bd12f) will decrease coverage by 0.01%.
The diff coverage is 83.33%.

@@            Coverage Diff             @@
##             main    #1174      +/-   ##
==========================================
- Coverage   92.29%   92.27%   -0.02%     
==========================================
  Files          24       24              
  Lines        4813     4818       +5     
  Branches      991      993       +2     
==========================================
+ Hits         4442     4446       +4     
  Misses        230      230              
- Partials      141      142       +1     
Impacted Files Coverage Δ
PyPDF2/_reader.py 90.55% <83.33%> (-0.07%) ⬇️

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 d8bd12f...94b1859. Read the comment docs.

@MartinThoma
Copy link
Member

@pubpub-zz Should I wait or merge?

@pubpub-zz
Copy link
Collaborator Author

ready to be merged. Can not find a test file for the TODO

@MartinThoma MartinThoma merged commit 9c8252d into py-pdf:main Jul 27, 2022
MartinThoma added a commit that referenced this pull request Jul 31, 2022
New Features (ENH):
-  Add ability to add hex encoded colors to outline items (#1186)
-  Add support for pathlib.Path in PdfMerger.merge (#1190)
-  Add link annotation (#1189)
-  Add capability to filter text extraction by orientation  (#1175)

Bug Fixes (BUG):
-  Named Dest in PDF1.1 (#1174)
-  Incomplete Graphic State save/restore (#1172)

Documentation (DOC):
-  Update changelog url in package metadata (#1180)
-  Table extraction (#1179)
-  Mention pyHanko for signing PDF documents (#1178)
-  We now have CMAP support (#1177)

Maintenance (MAINT):
-  Consistant usage of warnings / log messages (#1164)
-  Consistent terminology for outline items (#1156)

Code Style (STY):
-  Apply pre-commit (#1188)

Full Changelog: 2.8.1...2.9.0
@pubpub-zz pubpub-zz deleted the iss1173 branch August 8, 2022 06:55
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