-
Notifications
You must be signed in to change notification settings - Fork 344
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
OPIK-814 [SDK] Implement opik healthcheck
CLI command / python function to validate the current...
#1341
Conversation
…tion to validate the current...
@japdubengsub some feedback to your draft :)
You probably haven't got to backend/httpx errors results parsing, so I'm not commenting this part yet |
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.
Please don't forget about the general SDK guidelines.
Also, whenever you're adding a new code, pay extra attention to the questions:
- do we actually need an extra function for that?
- if we need it, is there a better place for it?
Looks way better, thanks! One last UX improvement, we currently have a message:
We know Opik URL, so it's better to:
After that we can merge |
Details
Output examples:
