-
Notifications
You must be signed in to change notification settings - Fork 465
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
[API] update api for multi file and pdf support #1522
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #1522 +/- ##
=======================================
Coverage 95.81% 95.81%
=======================================
Files 166 166
Lines 7648 7648
=======================================
Hits 7328 7328
Misses 320 320
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
6086d74
to
73e3faa
Compare
Extended with more configuration options: (which was planned for a follow up PR 😅) Also improved tests with a small tolerance for model updates (should not change much in the future so response mocks are fine) |
Fixed CI (which was green before but hasn't run any test 😅) |
CC @eudoxos |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Awesome works here, thank you Felix !
This PR:
Any feedback is welcome 🤗