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

fix: expose errors occurring before the first yield of stream response #5002

Merged
merged 2 commits into from
Sep 27, 2024
Merged

fix: expose errors occurring before the first yield of stream response #5002

merged 2 commits into from
Sep 27, 2024

Conversation

frostming
Copy link
Contributor

Signed-off-by: Frost Ming me@frostming.com

What does this PR address?

Fixes #(issue)

Before submitting:

@frostming frostming requested a review from a team as a code owner September 27, 2024 03:59
@frostming frostming requested review from parano and removed request for a team September 27, 2024 03:59
@frostming
fix: log endpoint and console url when developing
08935a8 
Signed-off-by: Frost Ming <me@frostming.com>
@frostming frostming merged commit 79c0df7 into bentoml:main Sep 27, 2024
50 of 51 checks passed
@frostming frostming deleted the fix/error-first-yield branch September 27, 2024 09:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@aarnphm aarnphm aarnphm approved these changes

@parano parano Awaiting requested review from parano parano is a code owner automatically assigned from bentoml/bentoml-reviewers

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@frostming @aarnphm