Skip to content

TST: Add test for #1738#1744

Merged
MartinThoma merged 2 commits intopy-pdf:mainfrom
pubpub-zz:tst1737
Mar 26, 2023
Merged

TST: Add test for #1738#1744
MartinThoma merged 2 commits intopy-pdf:mainfrom
pubpub-zz:tst1737

Conversation

@pubpub-zz
Copy link
Copy Markdown
Collaborator

@pubpub-zz pubpub-zz commented Mar 25, 2023

completes #1737

@codecov
Copy link
Copy Markdown

codecov bot commented Mar 25, 2023

Codecov Report

Patch and project coverage have no change.

Comparison is base (20ebcfd) 92.44% compared to head (19fe2f3) 92.44%.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1744   +/-   ##
=======================================
  Coverage   92.44%   92.44%           
=======================================
  Files          34       34           
  Lines        6538     6538           
  Branches     1296     1296           
=======================================
  Hits         6044     6044           
  Misses        323      323           
  Partials      171      171           

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.

@pubpub-zz
Copy link
Copy Markdown
Collaborator Author

@MartinThoma

up,to you to merge it or close it

@MartinThoma MartinThoma changed the title [TST] : add test for #1738 TST: Add test for #1738 Mar 25, 2023
@MartinThoma
Copy link
Copy Markdown
Member

Why would I want to close it?

@pubpub-zz
Copy link
Copy Markdown
Collaborator Author

test coverage was not improve. does this PR worth it?

@MartinThoma MartinThoma merged commit 3ecdedb into py-pdf:main Mar 26, 2023
@pubpub-zz pubpub-zz deleted the tst1737 branch June 24, 2023 08:42
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