[GitHub] [pulsar] tisonkun commented on a diff in pull request #18832: [improve][cli] handle help flag properly

[GitBox <gi...@apache.org>]

tisonkun commented on code in PR #18832:
URL: https://github.com/apache/pulsar/pull/18832#discussion_r1044577938


##########
pulsar-client-tools/src/main/java/org/apache/pulsar/admin/cli/CmdBase.java:
##########
@@ -37,12 +37,12 @@ public abstract class CmdBase {
     private PulsarAdmin admin;
     private IUsageFormatter usageFormatter;
 
-    @Parameter(names = { "-h", "--help" }, help = true, hidden = true)
-    private boolean help;
+    @Parameter(names = { "--help", "-h" }, help = true, hidden = true)
+    private boolean help = false;

Review Comment:
   Nope. This `--help` is applied at `pulsar-admin schemas -h` level. That is, one level down.



-- 
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