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

intercept OpenAI requests in worker testruns #2020

Merged
merged 6 commits into from May 16, 2024

Conversation

ksjogo
Copy link
Contributor

@ksjogo ksjogo commented May 8, 2024

What does this PR do?

Adds network levels mocks to allow the passing of worker test runs without an OpenAI API key.
The first iteration is very small, just adding the one necessary call.
As a followup a more comprehensive framework could be constructed.

Type of change

  • New feature (non-breaking change which adds functionality)

Mandatory Tasks

  • Make sure you have self-reviewed the code. A decent size PR without self-review might be rejected.

Copy link

vercel bot commented May 8, 2024

@ksjogo is attempting to deploy a commit to the langfuse Team on Vercel.

A member of the Team first needs to authorize it.

@ksjogo ksjogo marked this pull request as draft May 8, 2024 22:07
@ksjogo ksjogo force-pushed the worker-network-mocks branch 4 times, most recently from bfd8b56 to f288c81 Compare May 16, 2024 09:45
@maxdeichmann maxdeichmann marked this pull request as ready for review May 16, 2024 21:09
@maxdeichmann maxdeichmann enabled auto-merge (squash) May 16, 2024 21:09
@maxdeichmann maxdeichmann merged commit e0930fb into langfuse:main May 16, 2024
10 of 12 checks passed
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