Skip to content
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

Update Splash Screen with warning about openapi docs #221

Merged
merged 3 commits into from
Nov 13, 2023

Conversation

eyurtsev
Copy link
Collaborator

Update the splash screen to output a warning about OpenAPI docs missing when
using pydantic v2.

@cla-bot cla-bot bot added the cla-signed label Nov 13, 2023
@eyurtsev
Copy link
Collaborator Author

Hopefully will help with these issues:

#214
#215
#207

print(f'{orange("OpenAPI Docs:")}: ', end="")
print(
"Running with pydantic >= 2: OpenAPI docs for "
"invoke/batch/stream/stream_log` endpoints will not "
Copy link
Contributor

Choose a reason for hiding this comment

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

will not be

print(
"Running with pydantic >= 2: OpenAPI docs for "
"invoke/batch/stream/stream_log` endpoints will not "
"generated; but, API endpoints and playground will not be affected."
Copy link
Contributor

Choose a reason for hiding this comment

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

generated, but API ...

langserve/server.py Outdated Show resolved Hide resolved
@eyurtsev eyurtsev merged commit 2f4bcdd into main Nov 13, 2023
10 checks passed
@eyurtsev eyurtsev deleted the eugene/splash_screen branch November 13, 2023 19:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants