Skip to content

DEV: add TCEs to MGET command page#2917

Merged
dwdougherty merged 3 commits intomainfrom
DOC-5637
Mar 20, 2026
Merged

DEV: add TCEs to MGET command page#2917
dwdougherty merged 3 commits intomainfrom
DOC-5637

Conversation

@dwdougherty
Copy link
Collaborator

@dwdougherty dwdougherty commented Mar 20, 2026

The next round of code examples, this time for MGET.


Note

Low Risk
Low risk documentation-only change; main risk is example snippets failing to compile/run in the docs test harness due to new language fixtures and cleanup/assert logic.

Overview
Adds a clients-example block to the MGET command page showing expected Redis CLI input/output and wiring it to the shared cmds_string example set.

Introduces new cmds_string client examples for MGET across multiple languages/libraries (Go, C/hiredis, Node redis + ioredis, Python, PHP/Predis, Java/Jedis + Lettuce async/reactive, C#/StackExchange.Redis), including basic setup/cleanup and assertions where applicable.

Written by Cursor Bugbot for commit 821086b. This will update automatically on new commits. Configure here.

@dwdougherty dwdougherty requested review from a team and andy-stark-redis March 20, 2026 15:56
@dwdougherty dwdougherty self-assigned this Mar 20, 2026
@dwdougherty dwdougherty added cmds oss Redis Open Source labels Mar 20, 2026
@github-actions
Copy link
Contributor

github-actions bot commented Mar 20, 2026

DOC-5637

@github-actions
Copy link
Contributor

@jit-ci
Copy link

jit-ci bot commented Mar 20, 2026

🛡️ Jit Security Scan Results

CRITICAL HIGH MEDIUM

✅ No security findings were detected in this PR


Security scan by Jit

Copy link
Contributor

@andy-stark-redis andy-stark-redis left a comment

Choose a reason for hiding this comment

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

All of them look OK (including Rust) and they all seem to have the correct command API mapping, so LGTM. Well done (still amazed that we've even got C examples)!

@dwdougherty
Copy link
Collaborator Author

Thank you, @andy-stark-redis! Now off to wrangle with the buggerer.

@andy-stark-redis
Copy link
Contributor

Thank you, @andy-stark-redis! Now off to wrangle with the buggerer.

Yes, back to buggery with you! :-)

Copy link

@cursor cursor bot left a comment

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, have a team admin enable autofix in the Cursor dashboard.

@dwdougherty dwdougherty merged commit 0c59bcd into main Mar 20, 2026
47 checks passed
@dwdougherty dwdougherty deleted the DOC-5637 branch March 20, 2026 17:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cmds oss Redis Open Source

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants