Skip to content

Conversation

@alexander-alderman-webb
Copy link
Contributor

@alexander-alderman-webb alexander-alderman-webb commented Nov 7, 2025

Description

See changes to pydantic_ai_slim/pydantic_ai/_tool_manager.py introduced in pydantic/pydantic-ai@930d74c.

Issues

Reminders

@alexander-alderman-webb alexander-alderman-webb changed the title [do not merge]: generate tox fix(pydantic-ai): Do not fail on new ToolManager._call_tool() parameters Nov 7, 2025
@alexander-alderman-webb alexander-alderman-webb changed the title fix(pydantic-ai): Do not fail on new ToolManager._call_tool() parameters fix(pydantic-ai): Do not fail on new ToolManager._call_tool() parameters Nov 7, 2025
@codecov
Copy link

codecov bot commented Nov 7, 2025

Codecov Report

❌ Patch coverage is 66.66667% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 83.95%. Comparing base (2e31bc6) to head (798d73e).
✅ All tests successful. No failed tests found.

Files with missing lines Patch % Lines
...ntry_sdk/integrations/pydantic_ai/patches/tools.py 66.66% 1 Missing ⚠️
Additional details and impacted files
@@               Coverage Diff                @@
##           webb/run-ci    #5084       +/-   ##
================================================
+ Coverage        69.32%   83.95%   +14.62%     
================================================
  Files              179      179               
  Lines            17951    17951               
  Branches          3194     3194               
================================================
+ Hits             12445    15070     +2625     
+ Misses            4806     1907     -2899     
- Partials           700      974      +274     
Files with missing lines Coverage Δ
...ntry_sdk/integrations/pydantic_ai/patches/tools.py 78.94% <66.66%> (+78.94%) ⬆️

... and 64 files with indirect coverage changes

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.

2 participants