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.

Sign up for GitHub

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

Jump to bottom

Testing - Add test coverage for assistants streaming and ability to u… #444

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Testing - Add test coverage for assistants streaming and ability to u… #444

wants to merge 2 commits into from

Conversation

knash94
Copy link
Contributor

@knash94 knash94 commented Jul 4, 2024

What:

  • Bug Fix
  • New Feature

Description:

This PR adds some new tests for assistants streaming and adds the ability to mock the responses.
@knash94
Copy link
Contributor Author

knash94 commented Jul 4, 2024

@gehrisandro Noticed the tests are failing because the GitHub actions workflow runs a composer update which updates the rules of the linter.

I could either update the action, run a composer update + lint or just ignore for now?
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
1 participant
@knash94