From c177f944d95c460803f8a894fd13d4901c3155fe Mon Sep 17 00:00:00 2001 From: Mike Frysinger Date: Tue, 4 May 2021 08:06:36 -0400 Subject: subcmds: force consistent help text format We're inconsistent with help text as to whether it uses title case and whether it ends in a period. Add a test to enforce a standard, and use the style that Python optparse & argparse use themselves (e.g. with the --help option): always lowercase, and never trailing period. Change-Id: Ic1defae23daeac0ac9116aaf487427f50b34050d Reviewed-on: https://gerrit-review.googlesource.com/c/git-repo/+/305144 Reviewed-by: Raman Tenneti Tested-by: Mike Frysinger --- subcmds/rebase.py | 18 +++++++++--------- 1 file changed, 9 insertions(+), 9 deletions(-) (limited to 'subcmds/rebase.py') diff --git a/subcmds/rebase.py b/subcmds/rebase.py index e0186d4d..9ce4ecb8 100644 --- a/subcmds/rebase.py +++ b/subcmds/rebase.py @@ -46,27 +46,27 @@ branch but need to incorporate new upstream changes "underneath" them. p.add_option('--fail-fast', dest='fail_fast', action='store_true', - help='Stop rebasing after first error is hit') + help='stop rebasing after first error is hit') p.add_option('-f', '--force-rebase', dest='force_rebase', action='store_true', - help='Pass --force-rebase to git rebase') + help='pass --force-rebase to git rebase') p.add_option('--no-ff', dest='ff', default=True, action='store_false', - help='Pass --no-ff to git rebase') + help='pass --no-ff to git rebase') p.add_option('--autosquash', dest='autosquash', action='store_true', - help='Pass --autosquash to git rebase') + help='pass --autosquash to git rebase') p.add_option('--whitespace', dest='whitespace', action='store', metavar='WS', - help='Pass --whitespace to git rebase') + help='pass --whitespace to git rebase') p.add_option('--auto-stash', dest='auto_stash', action='store_true', - help='Stash local modifications before starting') + help='stash local modifications before starting') p.add_option('-m', '--onto-manifest', dest='onto_manifest', action='store_true', - help='Rebase onto the manifest version instead of upstream ' - 'HEAD. This helps to make sure the local tree stays ' - 'consistent if you previously synced to a manifest.') + help='rebase onto the manifest version instead of upstream ' + 'HEAD (this helps to make sure the local tree stays ' + 'consistent if you previously synced to a manifest)') def Execute(self, opt, args): all_projects = self.GetProjects(args) -- cgit v1.2.3-54-g00ecf