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

[Issue]: test in contrib-tests.yml should also use --skip-openai to skip the openai test #1112

Closed
LinxinS97 opened this issue Jan 1, 2024 · 2 comments · Fixed by #1132
Closed

[Issue]: test in contrib-tests.yml should also use --skip-openai to skip the openai test #1112

LinxinS97 opened this issue Jan 1, 2024 · 2 comments · Fixed by #1132
Labels
dev development experience/productivity

Comments

@LinxinS97
Copy link
Collaborator

Describe the issue

Test in contrib-tests.yml should also use --skip-openai instead of pip uninstall -y openai to skip the openai test.

This issue is related to #1088 and #1097

Steps to reproduce

No response

Screenshots and logs

No response

Additional Information

No response
@sonichi sonichi added the dev development experience/productivity label Jan 1, 2024
@sonichi
Copy link
Contributor

sonichi commented Jan 1, 2024

I agree. @maxim-saplin for awareness.

@maxim-saplin
Copy link
Collaborator

I agree. @maxim-saplin for awareness.

Will take care of the PR

@maxim-saplin maxim-saplin mentioned this issue Jan 3, 2024
Merged
1 task
whiskyboy pushed a commit to whiskyboy/autogen that referenced this issue Apr 17, 2024
@sonichi
update notebook with new models (microsoft#1112)
6c343d3 
* rerun notebook with new models

* planning agent

* link to notebook
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
dev development experience/productivity
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Change contrib-tests.yml tests to use --skip-openai
3 participants
@sonichi @maxim-saplin @LinxinS97