Skip to content

Cleaning up --help-all inconsistencies

Michael J Gruber git 2 versions In review

Get series with b4
b4 am [email protected]

Versions newest first · linked by author and title

  1. v2 Latest10 patches 3 repliesIn review
  2. v17 patches 13 repliesSuperseded

Patches in v2 No applies recorded yet

#Subject and filesSizeRepliesApplied
1/10commit,tag: use same wording for -Fbuiltin/commit.c builtin/tag.c+2 -20 replies
2/10commit,status: describe --porcelain just like pushbuiltin/commit.c+2 -20 replies
3/10clone,init: describe --template using the same wordingbuiltin/clone.c builtin/init-db.c+3 -30 replies
4/10commit,merge,tag: describe -m likewisebuiltin/commit.c builtin/merge.c builtin/tag.c+5 -50 replies
5/10add: describe --patch like checkout, resetbuiltin/add.c+1 -10 replies
6/10commit,status: describe -u likewisebuiltin/commit.c+1 -10 replies
7/10git-tag.txt: list all modes in the descriptionDocumentation/git-tag.txt+6 -50 replies
8/10Make <identifier> lowercase as per CodingGuidelinesparse-options.h+1 -10 replies
9/10Make <identifier> lowercase as per CodingGuidelinesbuiltin/commit.c builtin/config.c builtin/fast-export.c +5 more+25 -250 replies
10/10Make <identifier> lowercase as per CodingGuidelinest/t0040-parse-options.sh test-parse-options.c+2 -20 replies

Reviews trailers per patch

No Reviewed-by, Acked-by or Tested-by trailers for v2 yet.

Read the v2 thread

Cover letter [PATCHv2 00/10] Cleaning up --help-all inconsistencies

v2 incorporates the changes as discussed so far.
It comes with three bonus patches which enforce the use of lowercase <identifiers> (as per CodingGuidelines) in command messages (usage). Besides the guidelines, this is also the prevailing style in code. Documentation is a different issue.

patches 1,3,5 are unchanged wrt v1.

The last 3 are divided according to the grep expressions which I used to identify candidates.

Michael J Gruber (10):
commit,tag: use same wording for -F commit,status: describe --porcelain just like push clone,init: describe --template using the same wording commit,merge,tag: describe -m likewise
add: describe --patch like checkout, reset commit,status: describe -u likewise git-tag.txt: list all modes in the description Make <identifier> lowercase as per CodingGuidelines Make <identifier> lowercase as per CodingGuidelines Make <identifier> lowercase as per CodingGuidelines

 Documentation/git-tag.txt |   11 ++++++-----
 builtin/add.c             |    2 +-
 builtin/clone.c           |    4 ++--
 builtin/commit.c          |   22 +++++++++++-----------
 builtin/config.c          |    2 +-
 builtin/fast-export.c     |    4 ++--
 builtin/fetch.c           |    6 +++---
 builtin/init-db.c         |    2 +-
 builtin/merge.c           |    2 +-
 builtin/notes.c           |   16 ++++++++--------
 builtin/read-tree.c       |    4 ++--
 builtin/tag.c             |    6 +++---
 parse-options.h           |    2 +-
 t/t0040-parse-options.sh  |    2 +-
 test-parse-options.c      |    2 +-
 15 files changed, 44 insertions(+), 43 deletions(-)
Signature · 1 line
1.7.4.1.74.gf39475.dirty