[prev in list] [next in list] [prev in thread] [next in thread] 

List:       git
Subject:    Re: [PATCH 01/14] t3905-stash-include-untracked.sh: use the $( ... ) construct for command substitut
From:       Junio C Hamano <gitster () pobox ! com>
Date:       2014-04-30 18:07:17
Message-ID: xmqqiopqg08a.fsf () gitster ! dls ! corp ! google ! com
[Download RAW message or body]

Matthieu Moy <Matthieu.Moy@grenoble-inp.fr> writes:

> Patches 1/14 are
>
> Reviewed-by: Matthieu Moy <Matthieu.Moy@imag.fr>
>
> On a side note, reviewing patches by batches of 14 patches actually
> turns out to be much less convenient for me than reviewing larger
> batches.
>
> If I'm counting correctly, there should be around 100 patches remaining.
> I'd suggest that the next batch contain them all (probably publishing
> the branch somewhere and posting a merge request here would be better to
> avoid sending 100 mails).

I eyeballed these 28 patches so far (but not this round yet) and
each of the batches did fit my attention span for a series while
looking at other topics also in flight.  Tastes and preferences
differ between us, I guess.

I however can, and am very inclined to, trust your review blindly
and apply further patches on this topic with your Reviewed-by:
without even looking at the patches myself.  If "give us the whole"
is more convenient for you, I am perfectly happy with that approach.

Thanks for reviewing.
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
[prev in list] [next in list] [prev in thread] [next in thread]