-
Notifications
You must be signed in to change notification settings - Fork 3k
Drop deprecated streamablehttp_client
#1836
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
base: main
Are you sure you want to change the base?
Conversation
| CONTENT_TYPE = "content-type" | ||
| ACCEPT = "accept" | ||
|
|
||
|
|
||
| JSON = "application/json" | ||
| SSE = "text/event-stream" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Completely unnecessary.
maxisbey
left a comment
There was a problem hiding this 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 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
why as _annotations?
maxisbey
left a comment
There was a problem hiding this 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
|
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? |
No description provided.