public inbox for [email protected]  

help / color / mirror / Atom feed

From: Kevin Grittner <[email protected]>
To: Marti Raudsepp <[email protected]>
To: pgsql-docs <[email protected]>
Subject: Re: [PATCH] Various documentation typo/grammar fixes
Date: Mon, 25 Aug 2014 12:13:51 -0700
Message-ID: <[email protected]> (raw)
In-Reply-To: <CABRT9RA_kHEZrw9W=C_yYpR8+TTLqVMVKUSU0hrLC213cJ4Hyg@mail.gmail.com>
References: <CABRT9RA_kHEZrw9W=C_yYpR8+TTLqVMVKUSU0hrLC213cJ4Hyg@mail.gmail.com>
List-Unsubscribe: <mailto:[email protected]?body=unsub%20pgsql-docs>

Marti Raudsepp <[email protected]> wrote:

> I ran the Topy typo-fixer (https://github.com/intgr/topy , using rules
> developed for Wikipedia) over the PostgreSQL documentation directory.
> Attached is a patch with hand-picked fixes that are clearly an
> improvement.

For the initial patch, I agree with all except the first change of
"others'" to "other's" -- the start of the sentence uses "All", so
there is clearly more than one "other", so the the apostrophe
belongs after the "s".  The rest of the 0001- patch all look like
improvements to me.

I think most or all of the places that the 0002- patch (i.e. and
e.g.) need a change to look right, but I'm not sure this goes far
enough.  In style guides I've had to use these generally should
follow punctuation (left parenthesis, colon, semi-colon, comma, or
em dash), and be separated from any following text by punctuation
-- usually a comma.

In patch 0003- I agree that the change to "exactly the same" reads
better.  I'm torn on changing the hyphens to spaces.  I probably
wouldn't change them, but I wouldn't squawk if others preferred to
do so.

--
Kevin Grittner
EDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company


-- 
Sent via pgsql-docs mailing list ([email protected])
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-docs

view thread (14+ messages)  latest in thread

reply

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Reply to all the recipients using the --to and --cc options:
  reply via email

  To: [email protected]
  Cc: [email protected], [email protected]
  Subject: Re: [PATCH] Various documentation typo/grammar fixes
  In-Reply-To: <[email protected]>

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

This inbox is served by agora; see mirroring instructions
for how to clone and mirror all data and code used for this inbox