Skip to content

Conversation

cxljs
Copy link
Contributor

@cxljs cxljs commented Aug 31, 2025

closes #3489

@ndyakov
Copy link
Member

ndyakov commented Sep 1, 2025

I think this will be okay to have. Will let @htemelski-redis review.

Copy link
Member

@ndyakov ndyakov left a comment

Choose a reason for hiding this comment

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

Requested tests, not a blocker since this is straightforward getter.

htemelski-redis
htemelski-redis previously approved these changes Sep 1, 2025
Copy link
Member

@ndyakov ndyakov left a comment

Choose a reason for hiding this comment

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

Thank you, @cxljs!
@htemelski-redis you can review and merge if you are satisfied with the change.

Signed-off-by: Xiaolong Chen <fukua95@gmail.com>
Signed-off-by: Xiaolong Chen <fukua95@gmail.com>
@htemelski-redis htemelski-redis merged commit fafec3f into redis:master Sep 1, 2025
20 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.

Expose pipeline command results via Cmds() method
3 participants