-
Notifications
You must be signed in to change notification settings - Fork 147
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
Bump version to 0.6.1 #325
Conversation
OpenAI tests have passed previously in this PR, fixed a pre-commit formatting issue. we're good to go with this version if you could review and approve. |
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.
Approving with a question, though it's not introduced by this PR.
message = message.decode("utf-8") if isinstance(message, bytes) else message | ||
# drop the newline character | ||
if message.endswith("\n"): | ||
message = message[:-1] |
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.
This makes the log harder to read. What's the benefit?
Why are these changes needed?
Bump version to 0.6.1 for release.
Includes an update for the websocket test file to handle closed connections.
Related issue number
N/A
Checks