help: hide default value for default-off flags
authorMartin von Zweigbergk <martinvonz@google.com>
Wed, 19 Dec 2018 09:33:42 -0800
changeset 41020 e8e2a7656e83
parent 41019 fcc0a7ac9ebd
child 41021 55d6d0ff703b
help: hide default value for default-off flags If we no longer show the "[no-]" for default-off flags, it also seems unnecessary to show the "default: off" for them, since that's quite clearly the default. It's extra confusing for action flags like `hg bookmarks --delete`. Differential Revision: https://phab.mercurial-scm.org/D5455
mercurial/help.py
tests/test-extension.t
tests/test-fix.t
tests/test-help.t
tests/test-narrow-trackedcmd.t
tests/test-shelve.t
tests/test-uncommit.t
--- a/mercurial/help.py	Wed Dec 19 09:20:32 2018 -0800
+++ b/mercurial/help.py	Wed Dec 19 09:33:42 2018 -0800
@@ -165,14 +165,14 @@
 
         if isinstance(default, fancyopts.customopt):
             default = default.getdefaultvalue()
-        if (default and not callable(default)) or default is False:
+        if default and not callable(default):
             # default is of unknown type, and in Python 2 we abused
             # the %s-shows-repr property to handle integers etc. To
             # match that behavior on Python 3, we do str(default) and
             # then convert it to bytes.
             defaultstr = pycompat.bytestr(default)
-            if isinstance(default, bool):
-                defaultstr = _("on") if default else _("off")
+            if default is True:
+                defaultstr = _("on")
             desc += _(" (default: %s)") % defaultstr
 
         if isinstance(default, list):
--- a/tests/test-extension.t	Wed Dec 19 09:20:32 2018 -0800
+++ b/tests/test-extension.t	Wed Dec 19 09:33:42 2018 -0800
@@ -716,7 +716,7 @@
       --profile           print command execution profile
       --version           output version information and exit
    -h --help              display help and exit
-      --hidden            consider hidden changesets (default: off)
+      --hidden            consider hidden changesets
       --pager TYPE        when to paginate (boolean, always, auto, or never)
                           (default: auto)
 
@@ -755,7 +755,7 @@
       --profile           print command execution profile
       --version           output version information and exit
    -h --help              display help and exit
-      --hidden            consider hidden changesets (default: off)
+      --hidden            consider hidden changesets
       --pager TYPE        when to paginate (boolean, always, auto, or never)
                           (default: auto)
 
@@ -1043,7 +1043,7 @@
       --profile           print command execution profile
       --version           output version information and exit
    -h --help              display help and exit
-      --hidden            consider hidden changesets (default: off)
+      --hidden            consider hidden changesets
       --pager TYPE        when to paginate (boolean, always, auto, or never)
                           (default: auto)
 
@@ -1080,7 +1080,7 @@
       --profile           print command execution profile
       --version           output version information and exit
    -h --help              display help and exit
-      --hidden            consider hidden changesets (default: off)
+      --hidden            consider hidden changesets
       --pager TYPE        when to paginate (boolean, always, auto, or never)
                           (default: auto)
 
@@ -1155,7 +1155,7 @@
       --profile           print command execution profile
       --version           output version information and exit
    -h --help              display help and exit
-      --hidden            consider hidden changesets (default: off)
+      --hidden            consider hidden changesets
       --pager TYPE        when to paginate (boolean, always, auto, or never)
                           (default: auto)
 
@@ -1191,7 +1191,7 @@
       --profile           print command execution profile
       --version           output version information and exit
    -h --help              display help and exit
-      --hidden            consider hidden changesets (default: off)
+      --hidden            consider hidden changesets
       --pager TYPE        when to paginate (boolean, always, auto, or never)
                           (default: auto)
 
--- a/tests/test-fix.t	Wed Dec 19 09:20:32 2018 -0800
+++ b/tests/test-fix.t	Wed Dec 19 09:33:42 2018 -0800
@@ -104,12 +104,12 @@
   
   options ([+] can be repeated):
   
-      --all          fix all non-public non-obsolete revisions (default: off)
+      --all          fix all non-public non-obsolete revisions
       --base REV [+] revisions to diff against (overrides automatic selection,
                      and applies to every revision being fixed)
    -r --rev REV [+]  revisions to fix
-   -w --working-dir  fix the working directory (default: off)
-      --whole        always fix every line of a file (default: off)
+   -w --working-dir  fix the working directory
+      --whole        always fix every line of a file
   
   (some details hidden, use --verbose to show complete help)
 
--- a/tests/test-help.t	Wed Dec 19 09:20:32 2018 -0800
+++ b/tests/test-help.t	Wed Dec 19 09:33:42 2018 -0800
@@ -440,7 +440,7 @@
       --profile           print command execution profile
       --version           output version information and exit
    -h --help              display help and exit
-      --hidden            consider hidden changesets (default: off)
+      --hidden            consider hidden changesets
       --pager TYPE        when to paginate (boolean, always, auto, or never)
                           (default: auto)
   
@@ -540,7 +540,7 @@
       --profile           print command execution profile
       --version           output version information and exit
    -h --help              display help and exit
-      --hidden            consider hidden changesets (default: off)
+      --hidden            consider hidden changesets
       --pager TYPE        when to paginate (boolean, always, auto, or never)
                           (default: auto)
 
@@ -905,7 +905,7 @@
                                     xxxxxxxxxxxxxxxxxxxxxxx (default: 3)
    -n --                            normal desc
       --newline VALUE               line1 line2
-      --default-off                 enable X (default: off)
+      --default-off                 enable X
       --[no-]default-on             enable Y (default: on)
       --callableopt VALUE           adds foo
       --customopt VALUE             adds bar
@@ -2890,7 +2890,7 @@
   <td>display help and exit</td></tr>
   <tr><td></td>
   <td>--hidden</td>
-  <td>consider hidden changesets (default: off)</td></tr>
+  <td>consider hidden changesets</td></tr>
   <tr><td></td>
   <td>--pager TYPE</td>
   <td>when to paginate (boolean, always, auto, or never) (default: auto)</td></tr>
@@ -3094,7 +3094,7 @@
   <td>display help and exit</td></tr>
   <tr><td></td>
   <td>--hidden</td>
-  <td>consider hidden changesets (default: off)</td></tr>
+  <td>consider hidden changesets</td></tr>
   <tr><td></td>
   <td>--pager TYPE</td>
   <td>when to paginate (boolean, always, auto, or never) (default: auto)</td></tr>
--- a/tests/test-narrow-trackedcmd.t	Wed Dec 19 09:20:32 2018 -0800
+++ b/tests/test-narrow-trackedcmd.t	Wed Dec 19 09:33:42 2018 -0800
@@ -105,9 +105,8 @@
       --import-rules VALUE         import narrowspecs from a file
       --removeexclude VALUE [+]    old paths to no longer exclude
       --clear                      whether to replace the existing narrowspec
-                                   (default: off)
       --force-delete-local-changes forces deletion of local changes when
-                                   narrowing (default: off)
+                                   narrowing
    -e --ssh CMD                    specify ssh command to use
       --remotecmd CMD              specify hg command to run on the remote side
       --insecure                   do not verify server certificate (ignoring
--- a/tests/test-shelve.t	Wed Dec 19 09:20:32 2018 -0800
+++ b/tests/test-shelve.t	Wed Dec 19 09:33:42 2018 -0800
@@ -75,7 +75,7 @@
       --cleanup             delete all shelved changes
       --date DATE           shelve with the specified commit date
    -d --delete              delete the named shelved change(s)
-   -e --edit                invoke editor on commit messages (default: off)
+   -e --edit                invoke editor on commit messages
    -l --list                list current shelves
    -m --message TEXT        use text as shelve message
    -n --name NAME           use the given name for the shelved commit
--- a/tests/test-uncommit.t	Wed Dec 19 09:20:32 2018 -0800
+++ b/tests/test-uncommit.t	Wed Dec 19 09:33:42 2018 -0800
@@ -34,8 +34,7 @@
   
   options ([+] can be repeated):
   
-      --keep                allow an empty commit after uncommiting (default:
-                            off)
+      --keep                allow an empty commit after uncommiting
    -I --include PATTERN [+] include names matching the given patterns
    -X --exclude PATTERN [+] exclude names matching the given patterns