Re: [PATCH 01/14] t3905-stash-include-untracked.sh: use the $( ... ) construct for command substitution

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Matthieu Moy <Matthieu.Moy@xxxxxxxxxxxxxxx> writes:

> Patches 1/14 are
>
> Reviewed-by: Matthieu Moy <Matthieu.Moy@xxxxxxx>
>
> 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@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Linux Kernel Development]     [Gcc Help]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [V4L]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]     [Fedora Users]