Skip to content

Add command hyperlinks for server management command files #332

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

Open
wants to merge 57 commits into
base: main
Choose a base branch
from

Conversation

milliehartnt123
Copy link
Contributor

The following files were modified. Some of the files have two sets of commits, with one set being to add the tick marks back to the hyperlink for formatting.

  • acl.md
  • acl-getuser.md
  • acl-list.md
  • acl-log.md
  • acl-setuser.md
  • acl-whoami.md
  • bgrewriteaof.md
  • bgsave.md
  • command.md
  • getkeys.md
  • getkeysandflags.md
  • command-info.md
  • commandlog.md
  • commandlog-get.md
  • commandlog-len.md
  • config.md
  • config-get.md
  • config-resetsat.md
  • config-rewrite.md
  • config-set.md
  • failover.md
  • info.md
  • lastsave.md
  • latency.md
  • latency-graph.md
  • latency-histogram.md
  • latency-latest.md
  • memory.md
  • memory-stats.md
  • module.md
  • module-loadex.md
  • module-unload.md
  • monitor.md
  • save.md
  • shutdown.md
  • sync.md

Added hyperlink for ACL HELP command.

Signed-off-by: milliehartnt123 <[email protected]>
Added hyperlink for ACL SETUSER command.

Signed-off-by: milliehartnt123 <[email protected]>
Added hyperlink for the ACL SAVE command.

Signed-off-by: milliehartnt123 <[email protected]>
Added hyperlinks for the AUTH and HELLO commands.

Signed-off-by: milliehartnt123 <[email protected]>
Added hyperlinks for SET, GET, KEYS, AUTH, ACL CAT, CLIENT KILL, and ACL DELUSER commands.

Signed-off-by: milliehartnt123 <[email protected]>
Added hyperlink for AUTH command.

Signed-off-by: milliehartnt123 <[email protected]>
Added hyperlink for INFO command.

Signed-off-by: milliehartnt123 <[email protected]>
Added hyperlink for LASTSAVE command.

Signed-off-by: milliehartnt123 <[email protected]>
Added hyperlinks to GET, MGET, LATENCY, COMMAND GETKEYS, SWAPDB,  SYNC,  MONITOR, SLOWLOG, SORT, ZUNION, MIGRATE, BRPOPLPUSH, SMOVE, RENAME, MSET,  and CONFIG REWRITE commands. Did some minor editing in order to add hyperlinks.

Signed-off-by: milliehartnt123 <[email protected]>
Added hyperlink for COMMAND GETKEYSANDFLAGS command.

Signed-off-by: milliehartnt123 <[email protected]>
Added hyperlink for COMMAND and COMMAND GETKEYS commands.

Signed-off-by: milliehartnt123 <[email protected]>
Added hyperlink for the COMMAND command.

Signed-off-by: milliehartnt123 <[email protected]>
Added hyperlink for COMMANDLOG HELP command.

Signed-off-by: milliehartnt123 <[email protected]>
Added hyperlinks to COMMANDLOG RESET and CLIENT SETNAME commands.

Signed-off-by: milliehartnt123 <[email protected]>
Added hyperlink for the COMMANDLOG RESET command.

Signed-off-by: milliehartnt123 <[email protected]>
Added hyperlink for CONFIG HELP command.

Signed-off-by: milliehartnt123 <[email protected]>
Added hyperlink for CONFIG SET command and the valkey-cli (topics/cli.md)

Signed-off-by: milliehartnt123 <[email protected]>
Added hyperlinks to the INFO and LATENCY HISTOGRAM commands.

Signed-off-by: milliehartnt123 <[email protected]>
Added hyperlink for CONFIG SET command.

Signed-off-by: milliehartnt123 <[email protected]>
Added hyperlink for CONFIG GET command.

Signed-off-by: milliehartnt123 <[email protected]>
Added hyperlink to CLUSTER FAILOVER, CLIENT PAUSE WRITE , INFO, and REPLICAOF commands where referenced.

Signed-off-by: milliehartnt123 <[email protected]>
Added tick marks to all command hyperlinks
Added hyperlink for ACL CAT command

Signed-off-by: milliehartnt123 <[email protected]>
…perlink for PSYNC FAILOVER

Signed-off-by: milliehartnt123 <[email protected]>
…ting

Reformatted modifier names with tick marks instead of bolded text. Added hyperlinks for commands.

Signed-off-by: milliehartnt123 <[email protected]>
@milliehartnt123
Copy link
Contributor Author

@PingXie @madolson Please review this PR

Copy link
Contributor

@zuiderkwast zuiderkwast left a comment

Choose a reason for hiding this comment

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

Great @milliehartnt123, thank you!

@stockholmux we have been talking about making these links automatic. I think explicit links is pretty good. In this way we can be sure we don't link wrong for some words that look like commands but are not. WDYT?

@@ -6,4 +6,4 @@ The maximum number of entries in the different command log is governed by the `c

Once the command log reaches its maximal size, the oldest entry is removed whenever a new entry is created.

The command log can be cleared with the `COMMANDLOG RESET` command.
The command log can be cleared with the [`COMMANDLOG RESET`](connandlog-reset.md) command.
Copy link
Contributor

Choose a reason for hiding this comment

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

Typo

@zuiderkwast zuiderkwast changed the title Milliehartnt123 issue 294 8 add command hyperlinks for server management command files Add command hyperlinks for server management command files Jul 13, 2025
@stockholmux
Copy link
Member

@zuiderkwast yeah, I would be for linking explicitly instead of some script or regex that might be confusingly wrong.

Is this something we'd manually check for on PR review? E.g. someone submits docs is that going to be noted as feedback?

@zuiderkwast
Copy link
Contributor

Is this something we'd manually check for on PR review? E.g. someone submits docs is that going to be noted as feedback?

Yeah, I suppose we will ask contributors to add links to commands when missing. It will be more natural when it is seen everywhere in existing docs. Should we have some automatic checker?

IMO we don't need to link all occurrences, but the first occurrence on each page would be good to link, at least.

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.

4 participants