Skip to content

fix: Handle None result in single consumer request queue client #623

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
Merged
merged 1 commit into from
Oct 9, 2025
Merged

fix: Handle None result in single consumer request queue client #623

merged 1 commit into from
Oct 9, 2025

Conversation

janbuchar
Copy link
Contributor

@janbuchar janbuchar commented Oct 9, 2025 •
edited
Loading

@janbuchar janbuchar added the t-tooling Issues with this label are in the ownership of the tooling team. label Oct 9, 2025
@janbuchar janbuchar requested review from Mantisus and vdusek October 9, 2025 08:44
@github-actions github-actions bot added this to the 125th sprint - Tooling team milestone Oct 9, 2025
vdusek
vdusek approved these changes Oct 9, 2025
View reviewed changes
Copy link
Contributor

@vdusek vdusek left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@janbuchar janbuchar merged commit 451284a into master Oct 9, 2025
22 of 24 checks passed
@janbuchar janbuchar deleted the handle-none-in-request-queue-client branch October 9, 2025 11:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@vdusek vdusek vdusek approved these changes

@Mantisus Mantisus Awaiting requested review from Mantisus

Assignees

@janbuchar janbuchar

Labels

t-tooling Issues with this label are in the ownership of the tooling team.

Projects

None yet

Milestone

125th sprint - Tooling team

Development

Successfully merging this pull request may close these issues.

Pydantic validation error in request queue after request retries

2 participants

@janbuchar @vdusek