Skip to content

BUG : Named Dest in PDF1.1(#1173)#1174

Merged
MartinThoma merged 1 commit intopy-pdf:mainfrom
pubpub-zz:iss1173
Jul 27, 2022
Merged

BUG : Named Dest in PDF1.1(#1173)#1174
MartinThoma merged 1 commit intopy-pdf:mainfrom
pubpub-zz:iss1173

Conversation

@pubpub-zz
Copy link
Copy Markdown
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
Copy Markdown

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
Copy Markdown
Member

@pubpub-zz Should I wait or merge?

@pubpub-zz
Copy link
Copy Markdown
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.

2 participants