crocs-muni / sec-certs

Tool for analysis of security certificates and their security targets (Common Criteria, NIST FIPS140-2...).
https://sec-certs.org
MIT License
9 stars 7 forks source link

feat: Added pytesseract wrapper #357

Closed GeorgeFI closed 9 months ago

codecov[bot] commented 9 months ago

Codecov Report

Attention: 5 lines in your changes are missing coverage. Please review.

Comparison is base (30f7475) 76.82% compared to head (779c19e) 75.30%.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #357 +/- ## ========================================== - Coverage 76.82% 75.30% -1.52% ========================================== Files 54 54 Lines 6694 6698 +4 ========================================== - Hits 5142 5043 -99 - Misses 1552 1655 +103 ``` | [Files](https://app.codecov.io/gh/crocs-muni/sec-certs/pull/357?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=crocs-muni) | Coverage Δ | | |---|---|---| | [src/sec\_certs/sample/cc\_scheme.py](https://app.codecov.io/gh/crocs-muni/sec-certs/pull/357?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=crocs-muni#diff-c3JjL3NlY19jZXJ0cy9zYW1wbGUvY2Nfc2NoZW1lLnB5) | `82.81% <100.00%> (-9.04%)` | :arrow_down: | | [src/sec\_certs/utils/pdf.py](https://app.codecov.io/gh/crocs-muni/sec-certs/pull/357?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=crocs-muni#diff-c3JjL3NlY19jZXJ0cy91dGlscy9wZGYucHk=) | `56.07% <28.58%> (-0.14%)` | :arrow_down: | ... and [3 files with indirect coverage changes](https://app.codecov.io/gh/crocs-muni/sec-certs/pull/357/indirect-changes?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=crocs-muni)

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

GeorgeFI commented 9 months ago

@adamjanovsky Hi, the Pytesseract wrapper should be fixed now. The PR is ready for review.

GeorgeFI commented 9 months ago

@adamjanovsky pipeline passed successfully. The PR is ready for review now.