Provide documentation for title option

Perhaps this is a linux convention but it was not obvious to me that
'y' would use the default.

Change-Id: I5aedcb365828015effa64d41153f6cb9c9318dfc
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/5580992
Commit-Queue: Peter McNeeley <petermcneeley@chromium.org>
Reviewed-by: Yiwei Zhang <yiwzhang@google.com>
changes/92/5580992/3
Peter McNeeley 1 year ago committed by LUCI CQ
parent 84ef52b4df
commit dd0dd62997

@ -1852,8 +1852,8 @@ class Changelist(object):
title = RunGit(['show', '-s', '--format=%s', 'HEAD', '--']).strip() title = RunGit(['show', '-s', '--format=%s', 'HEAD', '--']).strip()
if options.force or options.skip_title: if options.force or options.skip_title:
return title return title
user_title = gclient_utils.AskForData('Title for patchset [%s]: ' % user_title = gclient_utils.AskForData(
title) 'Title for patchset (\'y\' for default) [%s]: ' % title)
# Use the default title if the user confirms the default with a 'y'. # Use the default title if the user confirms the default with a 'y'.
if user_title.lower() == 'y': if user_title.lower() == 'y':

Loading…
Cancel
Save