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

Jump to bottom

internal/dispatch: return cachingRedispatch #333

Merged
merged 1 commit into from Dec 10, 2021
Merged

internal/dispatch: return cachingRedispatch #333

merged 1 commit into from Dec 10, 2021

Conversation

jzelinskie
Copy link
Member

This was an autocomplete mistake and should've been the value returned.

@jzelinskie
internal/dispatch: return cachingRedispatch
3d0c3e1 
This was an autocomplete mistake and should've been the value returned.
@jzelinskie jzelinskie added kind/bug Something is broken or regressed priority/0 urgent This must be done ASAP area/dispatch Affects dispatching of requests labels Dec 10, 2021
ecordell
ecordell approved these changes Dec 10, 2021
View reviewed changes
Copy link
Contributor

@ecordell ecordell left a comment

Choose a reason for hiding this comment

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

LGTM

@ecordell ecordell merged commit 46fe9d1 into authzed:main Dec 10, 2021
@github-actions github-actions bot locked and limited conversation to collaborators Dec 10, 2021
@jzelinskie jzelinskie deleted the return-dispatch branch December 10, 2021 19:41
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@ecordell ecordell ecordell approved these changes

Assignees
No one assigned
Labels
area/dispatch Affects dispatching of requests kind/bug Something is broken or regressed priority/0 urgent This must be done ASAP
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@jzelinskie @ecordell