Skip to content

Conversation

@dwdougherty
Copy link
Collaborator

@dwdougherty dwdougherty commented Oct 31, 2025

This is a huge PR. It would be very helpful if reviewers could look at a random sample of say 5-10 commands and compare with the info in the following places:

I will also be looking at all ~500 pages. 😧

Also includes changes to layouts/commands/single.html to add a link at the top of each command page.

@github-actions
Copy link
Contributor

github-actions bot commented Oct 31, 2025

DOC-5890

@github-actions
Copy link
Contributor

Staging links:
https://redis.io/docs/staging/DOC-5890/commands/acl-cat/
https://redis.io/docs/staging/DOC-5890/commands/acl-deluser/
https://redis.io/docs/staging/DOC-5890/commands/acl-dryrun/
https://redis.io/docs/staging/DOC-5890/commands/acl-genpass/
https://redis.io/docs/staging/DOC-5890/commands/acl-getuser/
https://redis.io/docs/staging/DOC-5890/commands/acl-help/
https://redis.io/docs/staging/DOC-5890/commands/acl-list/
https://redis.io/docs/staging/DOC-5890/commands/acl-load/
https://redis.io/docs/staging/DOC-5890/commands/acl-log/
https://redis.io/docs/staging/DOC-5890/commands/acl-save/
https://redis.io/docs/staging/DOC-5890/commands/acl-setuser/
https://redis.io/docs/staging/DOC-5890/commands/acl-users/
https://redis.io/docs/staging/DOC-5890/commands/acl-whoami/
https://redis.io/docs/staging/DOC-5890/commands/append/
https://redis.io/docs/staging/DOC-5890/commands/asking/
https://redis.io/docs/staging/DOC-5890/commands/auth/
https://redis.io/docs/staging/DOC-5890/commands/bf.add/
https://redis.io/docs/staging/DOC-5890/commands/bf.card/
https://redis.io/docs/staging/DOC-5890/commands/bf.exists/
https://redis.io/docs/staging/DOC-5890/commands/bf.info/
https://redis.io/docs/staging/DOC-5890/commands/bf.insert/
https://redis.io/docs/staging/DOC-5890/commands/bf.loadchunk/
https://redis.io/docs/staging/DOC-5890/commands/bf.madd/
https://redis.io/docs/staging/DOC-5890/commands/bf.mexists/
https://redis.io/docs/staging/DOC-5890/commands/bf.reserve/
https://redis.io/docs/staging/DOC-5890/commands/bf.scandump/
https://redis.io/docs/staging/DOC-5890/commands/bgrewriteaof/
https://redis.io/docs/staging/DOC-5890/commands/bgsave/
https://redis.io/docs/staging/DOC-5890/commands/bitcount/
https://redis.io/docs/staging/DOC-5890/commands/bitfield/

Copy link
Contributor

@cmilesb cmilesb left a comment

Choose a reason for hiding this comment

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

I reviewed a few of the Cluster management commands and some of the Connection management commands, and they looked good and correct.

@dwdougherty
Copy link
Collaborator Author

I reviewed a few of the Cluster management commands and some of the Connection management commands, and they looked good and correct.

Thank you, @cmilesb!

Copy link
Contributor

@kaitlynmichael kaitlynmichael left a comment

Choose a reason for hiding this comment

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

I checked about a dozen and they were all correct.

@dwdougherty
Copy link
Collaborator Author

I checked about a dozen and they were all correct.

Thank you, @kaitlynmichael!

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.

I checked the ones below and they all looked OK to me.

BGREWRITEAOF
CONFIG GET
EVAL
FLUSHDB
FT.SYNUPDATE
GEOSEARCHSTORE
HKEYS
JSON.CLEAR
LATENCY HISTOGRAM
LMOVE
MODULE LOAD
QUIT
SETBIT
SMISMEMBER
TOPK.LIST
TOUCH
TS.RANGE
VADD
XPENDING
ZREMRANGEBYSCORE

@dwdougherty
Copy link
Collaborator Author

Thank you, @andy-stark-redis!

@dwdougherty dwdougherty removed the request for review from rrelledge November 3, 2025 21:38
@dwdougherty dwdougherty merged commit c1aec71 into main Nov 4, 2025
7 checks passed
@dwdougherty dwdougherty deleted the DOC-5890 branch November 4, 2025 16:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants