From: Stephen Boyd <bebarino@gmail.com>
To: Michael J Gruber <git@drmicha.warpmail.net>
Cc: git@vger.kernel.org
Subject: Re: [RFC/PATCH 2/3] Documentation: gitrevisions
Date: Wed, 14 Jul 2010 09:49:44 -0700 [thread overview]
Message-ID: <4C3DEAA8.3090509@gmail.com> (raw)
In-Reply-To: <4C3D785D.1050000@drmicha.warpmail.net>
On 07/14/2010 01:42 AM, Michael J Gruber wrote:
>
> I added this text on purpose. The "DESCRIPTION" section is meant to give
> a concise description of the overall picture so that, e.g., you
> understand which section will answer which question without having to
> read all of them. In this case it gives you a short overview of what can
> be referred to by revisions (commit, commit range, general object)
> before the sections go into the details of how to specify them.
>
> Also, as I mentioned in the cover letter, I suggest a rework of the
> actual (included, old) content if that structure is to stay. So, in a
> second step, one could avoid duplications.
Ok, I must have missed that in the cover letter. I hope in the next version we can rework the old content and integrate it into the two paragraphs in this patch. I don't see how keeping the DESCRIPTION section this verbose is helpful. At most it should say:
Many Git commands take revisions as parameters. A revision typically specifies a commit, but it can also specify a blob, a tree, or a set of commits. A set of commits is commonly referred to as a revision range. (or just range?)
And then I wonder if that should just go in the SYNOPSIS section and the DESCRIPTION should be deleted.
next prev parent reply other threads:[~2010-07-14 16:50 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-07-05 16:11 [RFC/PATCH 0/3] Documentation: Improve revision documentation Michael J Gruber
2010-07-05 16:11 ` [RFC/PATCH 1/3] Documentation: split off rev doc into include file Michael J Gruber
2010-07-05 16:11 ` [RFC/PATCH 2/3] Documentation: gitrevisions Michael J Gruber
2010-07-14 6:16 ` Stephen Boyd
2010-07-14 8:42 ` Michael J Gruber
2010-07-14 16:49 ` Stephen Boyd [this message]
2010-07-05 16:11 ` [RFC/PATCH 3/3] Documentation: link to gitrevisions rather than git-rev-parse Michael J Gruber
2010-07-05 17:26 ` [RFC/PATCH 0/3] Documentation: Improve revision documentation Ævar Arnfjörð Bjarmason
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=4C3DEAA8.3090509@gmail.com \
--to=bebarino@gmail.com \
--cc=git@drmicha.warpmail.net \
--cc=git@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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.