This is an archive of the discontinued Mercurial Phabricator instance.

Differential D5432

help: present boolean arguments as "--[no-]foo"
ClosedPublic

Authored by martinvonz on Dec 14 2018, 4:57 PM.

Details

Reviewers
durin42
Group Reviewers
hg-reviewers
Commits
rHGf6187e60f792: help: present boolean arguments as "--[no-]foo"
rHGccc28cc067dc: help: present boolean arguments as "--[no-]foo"
Summary

This should make it much more discoverable (we document it in `hg help
flags`, but most users don't think to look there).

Note that flags that default to None (and not False) will not get this
new presentation. We can change the defaults to False later for flags
where it makes sense (probably almost all boolean flags).

Diff Detail

Repository
rHG Mercurial
Lint
Lint Skipped
Unit
Unit Tests Skipped

Event Timeline

martinvonz created this revision.Dec 14 2018, 4:57 PM
Closed by commit rHGccc28cc067dc: help: present boolean arguments as "--[no-]foo" (authored by martinvonz). · Explain WhyDec 14 2018, 8:36 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Diff 12863

mercurial/help.py

if not verbose and any(w in desc for w in _exclkeywords): if not verbose and any(w in desc for w in _exclkeywords):
continue continue
so = '' so = ''
if shortopt: if shortopt:
so = '-' + shortopt so = '-' + shortopt
lo = '--' + longopt lo = '--' + longopt
if isinstance(default, bool):
lo = '--[no-]' + longopt
if isinstance(default, fancyopts.customopt): if isinstance(default, fancyopts.customopt):
default = default.getdefaultvalue() default = default.getdefaultvalue()
if (default and not callable(default)) or default is False: if (default and not callable(default)) or default is False:
# default is of unknown type, and in Python 2 we abused # default is of unknown type, and in Python 2 we abused
# the %s-shows-repr property to handle integers etc. To # the %s-shows-repr property to handle integers etc. To
# match that behavior on Python 3, we do str(default) and # match that behavior on Python 3, we do str(default) and
# then convert it to bytes. # then convert it to bytes.

tests/test-extension.t

--debugger start debugger --debugger start debugger
--encoding ENCODE set the charset encoding (default: ascii) --encoding ENCODE set the charset encoding (default: ascii)
--encodingmode MODE set the charset encoding mode (default: strict) --encodingmode MODE set the charset encoding mode (default: strict)
--traceback always print a traceback on exception --traceback always print a traceback on exception
--time time how long the command takes --time time how long the command takes
--profile print command execution profile --profile print command execution profile
--version output version information and exit --version output version information and exit
-h --help display help and exit -h --help display help and exit
--hidden consider hidden changesets (default: off) --[no-]hidden consider hidden changesets (default: off)
--pager TYPE when to paginate (boolean, always, auto, or never) --pager TYPE when to paginate (boolean, always, auto, or never)
(default: auto) (default: auto)
--debugger start debugger --debugger start debugger
--encoding ENCODE set the charset encoding (default: ascii) --encoding ENCODE set the charset encoding (default: ascii)
--encodingmode MODE set the charset encoding mode (default: strict) --encodingmode MODE set the charset encoding mode (default: strict)
--traceback always print a traceback on exception --traceback always print a traceback on exception
--time time how long the command takes --time time how long the command takes
--profile print command execution profile --profile print command execution profile
--version output version information and exit --version output version information and exit
-h --help display help and exit -h --help display help and exit
--hidden consider hidden changesets (default: off) --[no-]hidden consider hidden changesets (default: off)
--pager TYPE when to paginate (boolean, always, auto, or never) --pager TYPE when to paginate (boolean, always, auto, or never)
(default: auto) (default: auto)
$ echo 'debugextension = !' >> $HGRCPATH $ echo 'debugextension = !' >> $HGRCPATH
--debugger start debugger --debugger start debugger
--encoding ENCODE set the charset encoding (default: ascii) --encoding ENCODE set the charset encoding (default: ascii)
--encodingmode MODE set the charset encoding mode (default: strict) --encodingmode MODE set the charset encoding mode (default: strict)
--traceback always print a traceback on exception --traceback always print a traceback on exception
--time time how long the command takes --time time how long the command takes
--profile print command execution profile --profile print command execution profile
--version output version information and exit --version output version information and exit
-h --help display help and exit -h --help display help and exit
--hidden consider hidden changesets (default: off) --[no-]hidden consider hidden changesets (default: off)
--pager TYPE when to paginate (boolean, always, auto, or never) --pager TYPE when to paginate (boolean, always, auto, or never)
(default: auto) (default: auto)
Make sure that single '-v' option shows help and built-ins only for 'dodo' command Make sure that single '-v' option shows help and built-ins only for 'dodo' command
$ hg help -v dodo $ hg help -v dodo
hg dodo hg dodo
Does nothing Does nothing
--debugger start debugger --debugger start debugger
--encoding ENCODE set the charset encoding (default: ascii) --encoding ENCODE set the charset encoding (default: ascii)
--encodingmode MODE set the charset encoding mode (default: strict) --encodingmode MODE set the charset encoding mode (default: strict)
--traceback always print a traceback on exception --traceback always print a traceback on exception
--time time how long the command takes --time time how long the command takes
--profile print command execution profile --profile print command execution profile
--version output version information and exit --version output version information and exit
-h --help display help and exit -h --help display help and exit
--hidden consider hidden changesets (default: off) --[no-]hidden consider hidden changesets (default: off)
--pager TYPE when to paginate (boolean, always, auto, or never) --pager TYPE when to paginate (boolean, always, auto, or never)
(default: auto) (default: auto)
In case when extension name doesn't match any of its commands, In case when extension name doesn't match any of its commands,
help message should ask for '-v' to get list of built-in aliases help message should ask for '-v' to get list of built-in aliases
along with extension help along with extension help
$ cat > $TESTTMP/d/dudu.py <<EOF $ cat > $TESTTMP/d/dudu.py <<EOF
> """ > """
--debugger start debugger --debugger start debugger
--encoding ENCODE set the charset encoding (default: ascii) --encoding ENCODE set the charset encoding (default: ascii)
--encodingmode MODE set the charset encoding mode (default: strict) --encodingmode MODE set the charset encoding mode (default: strict)
--traceback always print a traceback on exception --traceback always print a traceback on exception
--time time how long the command takes --time time how long the command takes
--profile print command execution profile --profile print command execution profile
--version output version information and exit --version output version information and exit
-h --help display help and exit -h --help display help and exit
--hidden consider hidden changesets (default: off) --[no-]hidden consider hidden changesets (default: off)
--pager TYPE when to paginate (boolean, always, auto, or never) --pager TYPE when to paginate (boolean, always, auto, or never)
(default: auto) (default: auto)
$ hg help -v -e dudu $ hg help -v -e dudu
dudu extension - dudu extension -
This is an awesome 'dudu' extension. It does something and also writes 'Beep This is an awesome 'dudu' extension. It does something and also writes 'Beep
beep' beep'
--debugger start debugger --debugger start debugger
--encoding ENCODE set the charset encoding (default: ascii) --encoding ENCODE set the charset encoding (default: ascii)
--encodingmode MODE set the charset encoding mode (default: strict) --encodingmode MODE set the charset encoding mode (default: strict)
--traceback always print a traceback on exception --traceback always print a traceback on exception
--time time how long the command takes --time time how long the command takes
--profile print command execution profile --profile print command execution profile
--version output version information and exit --version output version information and exit
-h --help display help and exit -h --help display help and exit
--hidden consider hidden changesets (default: off) --[no-]hidden consider hidden changesets (default: off)
--pager TYPE when to paginate (boolean, always, auto, or never) --pager TYPE when to paginate (boolean, always, auto, or never)
(default: auto) (default: auto)
Disabled extension commands: Disabled extension commands:
$ ORGHGRCPATH=$HGRCPATH $ ORGHGRCPATH=$HGRCPATH
$ HGRCPATH= $ HGRCPATH=
$ export HGRCPATH $ export HGRCPATH

tests/test-fix.t

revisions are not forgotten in later ones. The --base flag can be used to revisions are not forgotten in later ones. The --base flag can be used to
override this default behavior, though it is not usually desirable to do override this default behavior, though it is not usually desirable to do
so. so.
(use 'hg help -e fix' to show help for the fix extension) (use 'hg help -e fix' to show help for the fix extension)
options ([+] can be repeated): options ([+] can be repeated):
--all fix all non-public non-obsolete revisions (default: off) --[no-]all fix all non-public non-obsolete revisions (default:
--base REV [+] revisions to diff against (overrides automatic selection, off)
and applies to every revision being fixed) --base REV [+] revisions to diff against (overrides automatic
selection, and applies to every revision being fixed)
-r --rev REV [+] revisions to fix -r --rev REV [+] revisions to fix
-w --working-dir fix the working directory (default: off) -w --[no-]working-dir fix the working directory (default: off)
--whole always fix every line of a file (default: off) --[no-]whole always fix every line of a file (default: off)
(some details hidden, use --verbose to show complete help) (some details hidden, use --verbose to show complete help)
$ hg help -e fix $ hg help -e fix
fix extension - rewrite file content in changesets or working copy fix extension - rewrite file content in changesets or working copy
(EXPERIMENTAL) (EXPERIMENTAL)
Provides a command that runs configured tools on the contents of modified Provides a command that runs configured tools on the contents of modified

tests/test-help.t

--debugger start debugger --debugger start debugger
--encoding ENCODE set the charset encoding (default: ascii) --encoding ENCODE set the charset encoding (default: ascii)
--encodingmode MODE set the charset encoding mode (default: strict) --encodingmode MODE set the charset encoding mode (default: strict)
--traceback always print a traceback on exception --traceback always print a traceback on exception
--time time how long the command takes --time time how long the command takes
--profile print command execution profile --profile print command execution profile
--version output version information and exit --version output version information and exit
-h --help display help and exit -h --help display help and exit
--hidden consider hidden changesets (default: off) --[no-]hidden consider hidden changesets (default: off)
--pager TYPE when to paginate (boolean, always, auto, or never) --pager TYPE when to paginate (boolean, always, auto, or never)
(default: auto) (default: auto)
(use 'hg help' for the full list of commands) (use 'hg help' for the full list of commands)
$ hg add -h $ hg add -h
hg add [OPTION]... [FILE]... hg add [OPTION]... [FILE]...
--debugger start debugger --debugger start debugger
--encoding ENCODE set the charset encoding (default: ascii) --encoding ENCODE set the charset encoding (default: ascii)
--encodingmode MODE set the charset encoding mode (default: strict) --encodingmode MODE set the charset encoding mode (default: strict)
--traceback always print a traceback on exception --traceback always print a traceback on exception
--time time how long the command takes --time time how long the command takes
--profile print command execution profile --profile print command execution profile
--version output version information and exit --version output version information and exit
-h --help display help and exit -h --help display help and exit
--hidden consider hidden changesets (default: off) --[no-]hidden consider hidden changesets (default: off)
--pager TYPE when to paginate (boolean, always, auto, or never) --pager TYPE when to paginate (boolean, always, auto, or never)
(default: auto) (default: auto)
Test the textwidth config option Test the textwidth config option
$ hg root -h --config ui.textwidth=50 $ hg root -h --config ui.textwidth=50
hg root hg root
options: options:
--longdesc VALUE --longdesc VALUE
xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx
xxxxxxxxxxxxxxxxxxxxxxx (default: 3) xxxxxxxxxxxxxxxxxxxxxxx (default: 3)
-n -- normal desc -n -- normal desc
--newline VALUE line1 line2 --newline VALUE line1 line2
--default-off enable X (default: off) --[no-]default-off enable X (default: off)
--default-on enable Y (default: on) --[no-]default-on enable Y (default: on)
--callableopt VALUE adds foo --callableopt VALUE adds foo
--customopt VALUE adds bar --customopt VALUE adds bar
--customopt-withdefault VALUE adds bar (default: foo) --customopt-withdefault VALUE adds bar (default: foo)
(some details hidden, use --verbose to show complete help) (some details hidden, use --verbose to show complete help)
Test that default list of commands includes extension commands that have help, Test that default list of commands includes extension commands that have help,
but not those that don't, except in verbose mode, when a keyword is passed, or but not those that don't, except in verbose mode, when a keyword is passed, or
<td>print command execution profile</td></tr> <td>print command execution profile</td></tr>
<tr><td></td> <tr><td></td>
<td>--version</td> <td>--version</td>
<td>output version information and exit</td></tr> <td>output version information and exit</td></tr>
<tr><td>-h</td> <tr><td>-h</td>
<td>--help</td> <td>--help</td>
<td>display help and exit</td></tr> <td>display help and exit</td></tr>
<tr><td></td> <tr><td></td>
<td>--hidden</td> <td>--[no-]hidden</td>
<td>consider hidden changesets (default: off)</td></tr> <td>consider hidden changesets (default: off)</td></tr>
<tr><td></td> <tr><td></td>
<td>--pager TYPE</td> <td>--pager TYPE</td>
<td>when to paginate (boolean, always, auto, or never) (default: auto)</td></tr> <td>when to paginate (boolean, always, auto, or never) (default: auto)</td></tr>
</table> </table>
</div> </div>
</div> </div>
<td>print command execution profile</td></tr> <td>print command execution profile</td></tr>
<tr><td></td> <tr><td></td>
<td>--version</td> <td>--version</td>
<td>output version information and exit</td></tr> <td>output version information and exit</td></tr>
<tr><td>-h</td> <tr><td>-h</td>
<td>--help</td> <td>--help</td>
<td>display help and exit</td></tr> <td>display help and exit</td></tr>
<tr><td></td> <tr><td></td>
<td>--hidden</td> <td>--[no-]hidden</td>
<td>consider hidden changesets (default: off)</td></tr> <td>consider hidden changesets (default: off)</td></tr>
<tr><td></td> <tr><td></td>
<td>--pager TYPE</td> <td>--pager TYPE</td>
<td>when to paginate (boolean, always, auto, or never) (default: auto)</td></tr> <td>when to paginate (boolean, always, auto, or never) (default: auto)</td></tr>
</table> </table>
</div> </div>
</div> </div>

tests/test-narrow-trackedcmd.t

$ hg tracked --import-rules $ hg tracked --import-rules
hg tracked: option --import-rules requires argument hg tracked: option --import-rules requires argument
hg tracked [OPTIONS]... [REMOTE] hg tracked [OPTIONS]... [REMOTE]
show or change the current narrowspec show or change the current narrowspec
options ([+] can be repeated): options ([+] can be repeated):
--addinclude VALUE [+] new paths to include --addinclude VALUE [+] new paths to include
--removeinclude VALUE [+] old paths to no longer include --removeinclude VALUE [+] old paths to no longer include
--addexclude VALUE [+] new paths to exclude --addexclude VALUE [+] new paths to exclude
--import-rules VALUE import narrowspecs from a file --import-rules VALUE import narrowspecs from a file
--removeexclude VALUE [+] old paths to no longer exclude --removeexclude VALUE [+] old paths to no longer exclude
--clear whether to replace the existing narrowspec --[no-]clear whether to replace the existing
(default: off) narrowspec (default: off)
--force-delete-local-changes forces deletion of local changes when --[no-]force-delete-local-changes forces deletion of local changes when
narrowing (default: off) narrowing (default: off)
-e --ssh CMD specify ssh command to use -e --ssh CMD specify ssh command to use
--remotecmd CMD specify hg command to run on the remote side --remotecmd CMD specify hg command to run on the remote
--insecure do not verify server certificate (ignoring side
web.cacerts config) --insecure do not verify server certificate
(ignoring web.cacerts config)
(use 'hg tracked -h' to show more help) (use 'hg tracked -h' to show more help)
[255] [255]
$ hg tracked --import-rules doesnotexist $ hg tracked --import-rules doesnotexist
abort: cannot read narrowspecs from '$TESTTMP/narrow/doesnotexist': $ENOENT$ abort: cannot read narrowspecs from '$TESTTMP/narrow/doesnotexist': $ENOENT$
[255] [255]
$ cat > specs <<EOF $ cat > specs <<EOF

tests/test-shelve.t

options ([+] can be repeated): options ([+] can be repeated):
-A --addremove mark new/missing files as added/removed before -A --addremove mark new/missing files as added/removed before
shelving shelving
-u --unknown store unknown files in the shelve -u --unknown store unknown files in the shelve
--cleanup delete all shelved changes --cleanup delete all shelved changes
--date DATE shelve with the specified commit date --date DATE shelve with the specified commit date
-d --delete delete the named shelved change(s) -d --delete delete the named shelved change(s)
-e --edit invoke editor on commit messages (default: off) -e --[no-]edit invoke editor on commit messages (default: off)
-l --list list current shelves -l --list list current shelves
-m --message TEXT use text as shelve message -m --message TEXT use text as shelve message
-n --name NAME use the given name for the shelved commit -n --name NAME use the given name for the shelved commit
-p --patch output patches for changes (provide the names of the -p --patch output patches for changes (provide the names of the
shelved changes as positional arguments) shelved changes as positional arguments)
-i --interactive interactive mode, only works while creating a shelve -i --interactive interactive mode, only works while creating a shelve
--stat output diffstat-style summary of changes (provide --stat output diffstat-style summary of changes (provide
the names of the shelved changes as positional the names of the shelved changes as positional

tests/test-uncommit.t

If no files are specified, the commit will be pruned, unless --keep is If no files are specified, the commit will be pruned, unless --keep is
given. given.
(use 'hg help -e uncommit' to show help for the uncommit extension) (use 'hg help -e uncommit' to show help for the uncommit extension)
options ([+] can be repeated): options ([+] can be repeated):
--keep allow an empty commit after uncommiting (default: --[no-]keep allow an empty commit after uncommiting (default:
off) off)
-I --include PATTERN [+] include names matching the given patterns -I --include PATTERN [+] include names matching the given patterns
-X --exclude PATTERN [+] exclude names matching the given patterns -X --exclude PATTERN [+] exclude names matching the given patterns
(some details hidden, use --verbose to show complete help) (some details hidden, use --verbose to show complete help)
Uncommit with no commits should fail Uncommit with no commits should fail