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.

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

feat(provider/openai): support loading response_format from a file #1711

Conversation

albertlieyingadrian
Copy link
Contributor

Context: #1710

@albertlieyingadrian
Copy link
Contributor Author

albertlieyingadrian commented Sep 18, 2024

Do we have unit test coverage for openAI provider? Wasn't able to find it @mldangelo

Hence, it passed the tests

@mldangelo
Copy link
Member

This looks great thank you.

FYI, unit tests for openai are all in test/providers.test.ts! I plan to split them out at some point in the future.

@mldangelo mldangelo merged commit b6cf309 into promptfoo:main Sep 18, 2024
12 checks passed
@albertlieyingadrian albertlieyingadrian deleted the feat/load-response-format-from-file branch September 18, 2024 18:22
@albertlieyingadrian
Copy link
Contributor Author

Gotcha, thanks! @mldangelo

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants