Skip to content

fix(integrations): allow explicit op parameter in ai_track #4597

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

Merged
merged 2 commits into from
Jul 21, 2025

Conversation

mshavliuk
Copy link
Contributor

fixes #4596

@mshavliuk mshavliuk requested a review from a team as a code owner July 18, 2025 07:07
@szokeasaurusrex
Copy link
Member

Thanks @mshavliuk for raising #4596 and providing this fix!

Copy link

codecov bot commented Jul 21, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 80.64%. Comparing base (b065719) to head (d387d22).
Report is 2 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #4597      +/-   ##
==========================================
+ Coverage   80.56%   80.64%   +0.07%     
==========================================
  Files         156      156              
  Lines       16498    16498              
  Branches     2806     2806              
==========================================
+ Hits        13292    13304      +12     
+ Misses       2313     2303      -10     
+ Partials      893      891       -2     
Files with missing lines Coverage Δ
sentry_sdk/ai/monitoring.py 81.81% <100.00%> (ø)

... and 8 files with indirect coverage changes

@szokeasaurusrex szokeasaurusrex merged commit 7b028b6 into getsentry:master Jul 21, 2025
137 checks passed
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.

ai_track uses multiple values for keyword argument 'op'
2 participants