Re: [PATCH] squash! completion: fill COMPREPLY directly when completing refs
- From
Junio C Hamano <gitster@pobox.com>
- Date
- Feb 13, 2017, 20:24 UTC
- Message-ID
- <xmqqshnhdeka.fsf@gitster.mtv.corp.google.com>
- In-Reply-To
- <CAM0VKj=Pai0fL7KtMdv1sg3N2KA1aBafGQ_XzXWKUsBmo62ZoA@mail.gmail.com>
SZEDER Gábor <szeder.dev@gmail.com> writes:
Show 14 quoted lines
> On Fri, Feb 10, 2017 at 10:44 PM, Junio C Hamano <gitster@pobox.com> wrote: > >> Should I expect a reroll to come, or is this the only fix-up to the >> series that begins at <20170203025405.8242-1-szeder.dev@gmail.com>? >> >> No hurries. > > Yes, definitely. > > I found a minor bug in the middle of the series, and haven't quite > made up my mind about it and its fix yet. Perhaps in a day or three. > > Regarding the 'strip' -> 'lstrip' 'for-each-ref' format modifier > rename that broke pu: I should keep using 'strip', right?
Right. I view the removal of 'strip' as an accident when 'lstrip' was introduced, not an intended change.