Skip to content

fix: misc httpx instrumentation fixes#22

Merged
sohankshirsagar merged 11 commits intomainfrom
sohan/fix-httpx-instrumentation
Jan 10, 2026
Merged

fix: misc httpx instrumentation fixes#22
sohankshirsagar merged 11 commits intomainfrom
sohan/fix-httpx-instrumentation

Conversation

@sohankshirsagar
Copy link
Contributor

No description provided.

Copy link

@cubic-dev-ai cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

1 issue found across 4 files

Prompt for AI agents (all issues)

Check if these issues are valid — if so, understand the root cause of each and fix them.


<file name="drift/instrumentation/httpx/instrumentation.py">

<violation number="1" location="drift/instrumentation/httpx/instrumentation.py:129">
P2: Missing import for `SpanInfo`. The return type annotation uses `SpanInfo` but it's not imported from `...core.tracing.span_utils`.</violation>
</file>

Reply with feedback, questions, or to request a fix. Tag @cubic-dev-ai to re-run a review.

@sohankshirsagar sohankshirsagar merged commit 9de29ad into main Jan 10, 2026
13 checks passed
@sohankshirsagar sohankshirsagar deleted the sohan/fix-httpx-instrumentation branch January 10, 2026 03:44
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.

1 participant