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
cli: update docs url for sql shell #118960
Conversation
It looks like your PR touches production code but doesn't add or edit any test code. Did you consider adding tests to your PR? 🦉 Hoot! I am a Blathers, a bot for CockroachDB. My owner is dev-inf. |
The SQL shell help function redirects the user to use-the-built-in-sql-client.html this page no longer exists. Instead the SQL shell should point to cockroach-sql.html. Epic: None Release note (cli change): Change the SQL shell help URL to point to cockroach-sql.html.
4a29df0
to
1c84dba
Compare
@@ -276,7 +276,7 @@ func (c *cliState) printCliHelp() { | |||
fmt.Fprintf(c.iCtx.stdout, helpMessageFmt, | |||
demoHelpStr, | |||
docs.URL("sql-statements.html"), | |||
docs.URL("use-the-built-in-sql-client.html"), | |||
docs.URL("cockroach-sql.html"), |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not for this PR, but the link without the extension is preferred. The website automatically redirects from the page with .html
to the one without.
* Extend redirect to more versions This link is used in the CLI help text. This redirect will be needed until the help text is updated in cockroachdb/cockroach#118960.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm! the test failures all are unrelated
TFTR! bors r+ |
Since you are already backporting to 23.1, would you consider backporting to 22.2 in case we cut another release? |
Build succeeded: |
The SQL shell help function redirects the user to
use-the-built-in-sql-client.html this page no longer exists. Instead the SQL shell should point to cockroach-sql.html.
Epic: None
Release note (cli change): Change the SQL shell help URL to point to cockroach-sql.html.