You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pulsar.apache.org by GitBox <gi...@apache.org> on 2021/11/30 08:07:45 UTC
[GitHub] [pulsar] Anonymitaet opened a new pull request #13044: [Doc] add tips for Pulsar tools
Anonymitaet opened a new pull request #13044:
URL: https://github.com/apache/pulsar/pull/13044
Preview looks good
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@pulsar.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [pulsar] Huanli-Meng commented on a change in pull request #13044: [Doc] add tips for Pulsar tools
Posted by GitBox <gi...@apache.org>.
Huanli-Meng commented on a change in pull request #13044:
URL: https://github.com/apache/pulsar/pull/13044#discussion_r759816995
##########
File path: site2/docs/reference-cli-tools.md
##########
@@ -15,8 +15,12 @@ All Pulsar command-line tools can be run from the `bin` directory of your [insta
* [`bookkeeper`](#bookkeeper)
* [`broker-tool`](#broker-tool)
-> ### Getting help
-> You can get help for any CLI tool, command, or subcommand using the `--help` flag, or `-h` for short. Here's an example:
+> **Important**
+>
+> - This page only shows **some frequently used commands**. For the latest information about `pulsar`, `pulsar-client`, and `pulsar-perf`, including commands, flags, descriptions, and more, see [Pulsar tools](https://pulsar.apache.org/tools/).
Review comment:
I think we can change "and more" to "and more details / information", or "and so on". "And more" sounds like a little bit unformal.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@pulsar.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [pulsar] Anonymitaet commented on pull request #13044: [Doc] add tips for Pulsar tools
Posted by GitBox <gi...@apache.org>.
Anonymitaet commented on pull request #13044:
URL: https://github.com/apache/pulsar/pull/13044#issuecomment-982381052
@urfreespace @tuteng @Huanli-Meng can you help review? Thanks
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@pulsar.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [pulsar] Anonymitaet commented on a change in pull request #13044: [Doc] add tips for Pulsar tools
Posted by GitBox <gi...@apache.org>.
Anonymitaet commented on a change in pull request #13044:
URL: https://github.com/apache/pulsar/pull/13044#discussion_r759916930
##########
File path: site2/docs/reference-cli-tools.md
##########
@@ -15,8 +15,12 @@ All Pulsar command-line tools can be run from the `bin` directory of your [insta
* [`bookkeeper`](#bookkeeper)
* [`broker-tool`](#broker-tool)
-> ### Getting help
-> You can get help for any CLI tool, command, or subcommand using the `--help` flag, or `-h` for short. Here's an example:
+> **Important**
+>
+> - This page only shows **some frequently used commands**. For the latest information about `pulsar`, `pulsar-client`, and `pulsar-perf`, including commands, flags, descriptions, and more, see [Pulsar tools](https://pulsar.apache.org/tools/).
Review comment:
OK
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@pulsar.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [pulsar] Anonymitaet merged pull request #13044: [Doc] add tips for Pulsar tools
Posted by GitBox <gi...@apache.org>.
Anonymitaet merged pull request #13044:
URL: https://github.com/apache/pulsar/pull/13044
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@pulsar.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org