Skip to content

Conversation

@Kludex
Copy link
Member

@Kludex Kludex commented Jan 6, 2026

No description provided.

Comment on lines -56 to -61
CONTENT_TYPE = "content-type"
ACCEPT = "accept"


JSON = "application/json"
SSE = "text/event-stream"
Copy link
Member Author

Choose a reason for hiding this comment

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

Completely unnecessary.

@Kludex Kludex requested a review from maxisbey January 6, 2026 22:21
Copy link
Contributor

@maxisbey maxisbey left a comment

Choose a reason for hiding this comment

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

one comment as questino, feel free to dismiss

providing support for HTTP POST requests with optional SSE streaming responses
and session management.
"""
from __future__ import annotations as _annotations
Copy link
Contributor

Choose a reason for hiding this comment

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

why as _annotations?

Copy link
Contributor

@maxisbey maxisbey left a comment

Choose a reason for hiding this comment

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

idk how to undo an approval. doing this quickly to remove it

@maxisbey
Copy link
Contributor

maxisbey commented Jan 8, 2026

Just before merging, wanting to hear your thoughts on removing this so soon? I guess for v2 we don't want any already deprecated things bought over, is that your thinking?

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.

3 participants