From: Weijie Yuan <wy@wyuan.org>
To: Nguyen Duc Thinh <ducthinh100812@gmail.com>
Cc: Jonathan Corbet <corbet@lwn.net>,
Shuah Khan <skhan@linuxfoundation.org>,
Randy Dunlap <rdunlap@infradead.org>,
workflows@vger.kernel.org, linux-doc@vger.kernel.org,
linux-kernel@vger.kernel.org
Subject: Re: [PATCH] Documentation : process : Capitalize Git, Quilt, and Mercurial consistently
Date: Mon, 24 Aug 2026 17:13:09 +0800 [thread overview]
Message-ID: <aowLJUv1VVri3UKw@wyuan.org> (raw)
In-Reply-To: <20260824084235.9396-1-ducthinh100812@gmail.com>
Nit: why are there extra <space> befoer ':'?
On Mon, Aug 24, 2026 at 03:42:35PM +0700, Nguyen Duc Thinh wrote:
> Update various process documentation files to properly capitalize "Git",
> "Quilt", and "Mercurial" as proper nouns when referring to the source
> management tools themselves.
>
> Instances explicitly referring to literal command-line invocations
> (such as "git log" or "git bisect") have been left lowercase.
Yep, Documentation/CodingGuideLines of Git says:
A note on notation:
Use 'git' (all lowercase) when talking about commands i.e. something
the user would type into a shell and use 'Git' (uppercase first letter)
when talking about the version control system and its properties.
(ln~1016)
I would say this is a powerful argument.
> Signed-off-by: Nguyen Duc Thinh <ducthinh100812@gmail.com>
> ---
> Documentation/process/1.Intro.rst | 2 +-
> Documentation/process/2.Process.rst | 18 ++++-----
> Documentation/process/3.Early-stage.rst | 2 +-
> Documentation/process/5.Posting.rst | 10 ++---
> Documentation/process/7.AdvancedTopics.rst | 36 +++++++++---------
> Documentation/process/backporting.rst | 18 ++++-----
> .../process/embargoed-hardware-issues.rst | 2 +-
> .../process/maintainer-pgp-guide.rst | 38 +++++++++----------
> Documentation/process/submitting-patches.rst | 4 +-
> 9 files changed, 65 insertions(+), 65 deletions(-)
Sorry, I wonder if this is all? Is there any place that has been
forgotten? Or you have already matched it all with a script.
[...]
> diff --git a/Documentation/process/submitting-patches.rst b/Documentation/process/submitting-patches.rst
> index 7ae79452e..b6ca49437 100644
> --- a/Documentation/process/submitting-patches.rst
> +++ b/Documentation/process/submitting-patches.rst
> @@ -866,7 +866,7 @@ automatically include the base tree information in your submission by
> using the ``--base`` flag. The easiest and most convenient way to use
> this option is with topical branches::
>
> - $ git checkout -t -b my-topical-branch master
> + $ git switch -t -c my-topical-branch master
> Branch 'my-topical-branch' set up to track local branch 'master'.
> Switched to a new branch 'my-topical-branch'
>
> @@ -882,7 +882,7 @@ notice that it will have the ``base-commit:`` trailer at the very
> bottom, which provides the reviewer and the CI tools enough information
> to properly perform ``git am`` without worrying about conflicts::
>
> - $ git checkout -b patch-review [base-commit-id]
> + $ git switch -c patch-review [base-commit-id]
> Switched to a new branch 'patch-review'
> $ git am patches.mbox
> Applying: First Commit
But you casually changed this part but didn't mention it in the commit
message. (So it is not a logically relevant modification.) I think it's
even better to attach a reason for it, e.g. Git now recommends to use
'git switch' instead of 'git checkout' or so.
Thanks.
prev parent reply other threads:[~2026-08-24 9:13 UTC|newest]
Thread overview: 2+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-08-24 8:42 [PATCH] Documentation : process : Capitalize Git, Quilt, and Mercurial consistently Nguyen Duc Thinh
2026-08-24 9:13 ` Weijie Yuan [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=aowLJUv1VVri3UKw@wyuan.org \
--to=wy@wyuan.org \
--cc=corbet@lwn.net \
--cc=ducthinh100812@gmail.com \
--cc=linux-doc@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=rdunlap@infradead.org \
--cc=skhan@linuxfoundation.org \
--cc=workflows@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox