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

Annotate tests appropriately #1551

Merged
merged 4 commits into from
Jan 14, 2023
Merged

Annotate tests appropriately #1551

merged 4 commits into from
Jan 14, 2023

Conversation

dkg
Copy link
Contributor

@dkg dkg commented Jan 13, 2023

By annotating these tests, we can use pytest markers to skip external tests and tests that depend on sample-files.

dkg and others added 3 commits January 13, 2023 17:37
By annotating these tests, we can use pytest markers to skip external
tests and tests that depend on sample-files.
@codecov
Copy link

codecov bot commented Jan 14, 2023

Codecov Report

Base: 91.79% // Head: 91.79% // No change to project coverage 👍

Coverage data is based on head (eabeb15) compared to base (f1cd7ae).
Patch has no changes to coverable lines.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1551   +/-   ##
=======================================
  Coverage   91.79%   91.79%           
=======================================
  Files          33       33           
  Lines        6093     6093           
  Branches     1200     1200           
=======================================
  Hits         5593     5593           
  Misses        323      323           
  Partials      177      177           

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 at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

@MartinThoma
Copy link
Member

@dkg Thank you for fixing the annotations ❤️

@MasterOdin Thank you for fixing the formatting so that the CI runs through 🙏

@MartinThoma MartinThoma merged commit 767047b into py-pdf:main Jan 14, 2023
MartinThoma added a commit that referenced this pull request Jan 22, 2023
New Features (ENH):
-  Add page label support to PdfWriter (#1558)
-  Accept inline images with space before EI (#1552)
-  Add circle annotation support (#1556)
-  Add polygon annotation support (#1557)
-  Make merging pages produce a deterministic PDF (#1542, #1543)

Bug Fixes (BUG):
-  Fix error in cmap extraction (#1544)
-  Remove erroneous assertion check (#1564)
-  Fix dictionary access of optional page label keys (#1562)

Robustness (ROB):
-  Set ignore_eof=True for read_until_regex (#1521)

Documentation (DOC):
-  Paper size (#1550)

Developer Experience (DEV):
-  Fix broken combination of dependencies of docs.txt
-  Annotate tests appropriately (#1551)

[Full Changelog](3.2.1...3.3.0)
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

3 participants