-
Notifications
You must be signed in to change notification settings - Fork 142
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
vizro_ai_integration_tests #146
Conversation
…o_ai_integration_tests
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.
@l0uden Hi Alexey, can you add one small change of the dev version. @maxschulz-COL what would be the current version if we want to bump test pacakge? thank you!
Wait I don't understand, what do you mean? |
the test pypi version we want to update with the newest merge, do we want to bump the version or it should be fine to use the same version and just replace it? |
Ah ok got it! I already published a |
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.
Lgtm, please do not forget to update the dev
version and to adjust the conditions under which this has to run
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.
LGTM, thank you!
…o_ai_integration_tests
@maxschulz-COL, what does it mean? |
Description
integration tests for checking correct response of vizro-ai
Screenshot
Checklist
Types of changes
Notice
I acknowledge and agree that, by checking this box and clicking "Submit Pull Request":