fix(integrations): Add partial json support to streams
Description
I actually forgotten to add the most crucial in PR where we wanted to add support for partial_json in tool calling when streaming. This PR fixes that.
cc @antonpirker apologies for this!
Codecov Report
Attention: Patch coverage is 50.00000% with 1 line in your changes missing coverage. Please review.
Project coverage is 79.55%. Comparing base (
2423299) to head (018cb05). Report is 1 commits behind head on master.
:white_check_mark: All tests successful. No failed tests found.
| Files with missing lines | Patch % | Lines |
|---|---|---|
| sentry_sdk/integrations/anthropic.py | 50.00% | 0 Missing and 1 partial :warning: |
Additional details and impacted files
@@ Coverage Diff @@
## master #3674 +/- ##
==========================================
- Coverage 79.55% 79.55% -0.01%
==========================================
Files 140 140
Lines 15515 15517 +2
Branches 2629 2630 +1
==========================================
+ Hits 12343 12344 +1
Misses 2338 2338
- Partials 834 835 +1
| Files with missing lines | Coverage Δ | |
|---|---|---|
| sentry_sdk/integrations/anthropic.py | 84.30% <50.00%> (-0.41%) |
:arrow_down: |
@kwnath could you fix the failing tests please? Thanks!
This issue has gone three weeks without activity. In another week, I will close it.
But! If you comment or otherwise update it, I will reset the clock, and if you remove the label Waiting for: Community, I will leave it alone ... forever!
"A weed is but an unloved flower." ― Ella Wheeler Wilcox 🥀
This issue has gone three weeks without activity. In another week, I will close it.
But! If you comment or otherwise update it, I will reset the clock, and if you remove the label Waiting for: Community, I will leave it alone ... forever!
"A weed is but an unloved flower." ― Ella Wheeler Wilcox 🥀