Skip to content

Conversation

@fg-mindee
Copy link
Contributor

This PR introduces the following modifications:

  • added the possibility to read a PDF from a byte stream
  • added corresponding unittest

Any feedback is welcome!

@fg-mindee fg-mindee added type: enhancement Improvement module: io Related to doctr.io ext: tests Related to tests folder labels Feb 11, 2021
@fg-mindee fg-mindee added this to the 0.2.0 milestone Feb 11, 2021
@fg-mindee fg-mindee self-assigned this Feb 11, 2021
@codecov
Copy link

codecov bot commented Feb 11, 2021

Codecov Report

Merging #83 (e8ea9eb) into main (e9bfd3e) will increase coverage by 0.00%.
The diff coverage is 100.00%.

Impacted file tree graph

@@           Coverage Diff           @@
##             main      #83   +/-   ##
=======================================
  Coverage   96.71%   96.72%           
=======================================
  Files          24       24           
  Lines         791      793    +2     
=======================================
+ Hits          765      767    +2     
  Misses         26       26           
Flag Coverage Δ
unittests 96.72% <100.00%> (+<0.01%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
doctr/documents/reader.py 100.00% <100.00%> (ø)

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 e9bfd3e...d0bcb59. Read the comment docs.

@fg-mindee fg-mindee merged commit 7636015 into main Feb 11, 2021
@fg-mindee fg-mindee deleted the pdf-stream branch February 11, 2021 14:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ext: tests Related to tests folder module: io Related to doctr.io type: enhancement Improvement

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants