From: Junio C Hamano <gitster@pobox.com>
To: "Jason St. John" <jstjohn@purdue.edu>
Cc: git@vger.kernel.org
Subject: Re: [PATCH 2/2] Fix minor grammatical and other formatting issues in the "git log" man page
Date: Wed, 13 Nov 2013 13:56:24 -0800 [thread overview]
Message-ID: <xmqqy54sc6ev.fsf@gitster.dls.corp.google.com> (raw)
In-Reply-To: <1384323709-2690-2-git-send-email-jstjohn@purdue.edu> (Jason St. John's message of "Wed, 13 Nov 2013 01:21:49 -0500")
"Jason St. John" <jstjohn@purdue.edu> writes:
> Documentation/git-log.txt:
> -- replace single quotes around options/commands with backticks
> -- use single quotes around references to sections
> -- replaced some double quotes with proper AsciiDoc quotes (e.g.
> ``foo'')
> -- use backticks around files and file paths
> -- use title case when referring to section headings
> -- use backticks around option arguments/defaults
>
> Signed-off-by: Jason St. John <jstjohn@purdue.edu>
> ---
> When working on this commit, I noticed a difference in how options and
> option descriptions are separated (e.g. with a blank line or not). At least
> with Vim's syntax highlighting, if there is a blank line between the option
> and its description, the text block is all colored the same; however, if
> there isn't a blank line, then the text block is not specially colored.
>
> Is there an existing convention for how this should be done?
I do not think we have a written rule or convention (and I do not
know if we want one). While reading the text in the source form
(and the point of choosing AsciiDoc was to be able to read the docs
without formatting), I personally have a slight preference to
immediately follow the body text to the label in the labelled list,
and a blank line after the item, i.e.
item label::
This describes the item.
next item label::
This describes the next item.
as it makes it clear that the body belongs to the heading that
precedes it.
But it does help to have a blank between the label and the beginning
of the body when reflowing the body with fill-paragraph, i.e.
item label::
This describes the item.
You say that it is also easier on Vim to have the blank line there,
so perhaps we may want to aim for updating the documentation over
time to consistently do so. I dunno.
next prev parent reply other threads:[~2013-11-13 21:56 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-11-13 6:21 [PATCH 1/2] Rewrite man page explanation of git log's "--log-size" option Jason St. John
2013-11-13 6:21 ` [PATCH 2/2] Fix minor grammatical and other formatting issues in the "git log" man page Jason St. John
2013-11-13 21:56 ` Junio C Hamano [this message]
2013-11-15 1:44 ` Jason St. John
2013-11-15 1:47 ` Jason St. John
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=xmqqy54sc6ev.fsf@gitster.dls.corp.google.com \
--to=gitster@pobox.com \
--cc=git@vger.kernel.org \
--cc=jstjohn@purdue.edu \
/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;
as well as URLs for NNTP newsgroup(s).