diff options
author | Mike Frysinger <vapier@google.com> | 2021-05-04 08:06:36 -0400 |
---|---|---|
committer | Mike Frysinger <vapier@google.com> | 2021-05-04 16:40:53 +0000 |
commit | c177f944d95c460803f8a894fd13d4901c3155fe (patch) | |
tree | 1645c1bcf6e2cd304600ebc1f5193a0edf329429 /subcmds/diff.py | |
parent | aedd1e5ef015fba194681e167edf460c21a1c980 (diff) | |
download | git-repo-c177f944d95c460803f8a894fd13d4901c3155fe.tar.gz |
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 <rtenneti@google.com>
Tested-by: Mike Frysinger <vapier@google.com>
Diffstat (limited to 'subcmds/diff.py')
-rw-r--r-- | subcmds/diff.py | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/subcmds/diff.py b/subcmds/diff.py index 4966bb1a..b400ccfd 100644 --- a/subcmds/diff.py +++ b/subcmds/diff.py | |||
@@ -33,7 +33,7 @@ to the Unix 'patch' command. | |||
33 | def _Options(self, p): | 33 | def _Options(self, p): |
34 | p.add_option('-u', '--absolute', | 34 | p.add_option('-u', '--absolute', |
35 | dest='absolute', action='store_true', | 35 | dest='absolute', action='store_true', |
36 | help='Paths are relative to the repository root') | 36 | help='paths are relative to the repository root') |
37 | 37 | ||
38 | def _ExecuteOne(self, absolute, project): | 38 | def _ExecuteOne(self, absolute, project): |
39 | """Obtains the diff for a specific project. | 39 | """Obtains the diff for a specific project. |