From: Karthik Nayak <karthik.188@gmail.com>
Date: Tue, 23 Jun 2015 11:05:38 GMT
Subject: Re: [PATCH v4 07/19] for-each-ref: add '--merged' and '--no-merged' options
Message-ID: <CAOLa=ZRUuRM8jofM6As_=F5Lbpb2LBq=X4pTbmDfufntYeWnmg@mail.gmail.com>
In-Reply-To: <CAPig+cQLHX2spg00Zn1J=PMg_cuy2ARuWPOQQXLT1yLoQZ_SFg@mail.gmail.com>

On Tue, Jun 23, 2015 at 4:11 AM, Eric Sunshine <sunshine@sunshineco.com> wrote:
>
> According to the documentation you added to the OPTIONS section, the
> object following --merged and --no-merged is optional. Therefore,
> shouldn't this be s/<object>/[<object>]/ ?
>
> Also, in the OPTIONS section, you spelled it "commit" rather than "object".
>

>
> Ditto: s/<object>/[<object>]/
>

Noted and changed, this would apply to --contains also :)

-- 
Regards,
Karthik Nayak