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

FIX : decode #23 in NameObject #1342

Merged
merged 2 commits into from Sep 10, 2022
Merged

Conversation

pubpub-zz
Copy link
Collaborator

closes #1340

@pubpub-zz
Copy link
Collaborator Author

pending to confirmation from @sietzeberends that the bug is fixed

@codecov
Copy link

codecov bot commented Sep 10, 2022

Codecov Report

Merging #1342 (8f44458) into main (0ceaa60) will decrease coverage by 0.09%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main    #1342      +/-   ##
==========================================
- Coverage   94.72%   94.62%   -0.10%     
==========================================
  Files          30       30              
  Lines        5118     5118              
  Branches     1052     1052              
==========================================
- Hits         4848     4843       -5     
- Misses        158      164       +6     
+ Partials      112      111       -1     
Impacted Files Coverage Δ
PyPDF2/generic/_base.py 100.00% <100.00%> (ø)
PyPDF2/generic/_data_structures.py 95.19% <0.00%> (-0.83%) ⬇️

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

@pubpub-zz
Copy link
Collaborator Author

All good!

Co-authored-by: Martin Thoma <info@martin-thoma.de>
@pubpub-zz pubpub-zz closed this Sep 10, 2022
@pubpub-zz pubpub-zz reopened this Sep 10, 2022
@MartinThoma MartinThoma merged commit d3a4a99 into py-pdf:main Sep 10, 2022
@MartinThoma
Copy link
Member

Thank you @pubpub-zz 🤗

@pubpub-zz
Copy link
Collaborator Author

don't forget to propose @sietzeberends as contributor, his reporting was excellent

MartinThoma added a commit that referenced this pull request Sep 11, 2022
Bug Fixes (BUG):
-  Fix Error in transformations (#1341)
-  Decode #23 in NameObject (#1342)

Testing (TST):
-  Use pytest.warns() for warnings, and .raises() for exceptions (#1325)

Full Changelog: 2.10.6...2.10.7
@pubpub-zz pubpub-zz deleted the iss_unnumber branch June 24, 2023 08:39
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.

[PyPDF2-2.10.6] ValueError: invalid literal for int() with base 16
2 participants