On Thu, 16 Nov 2017 18:17:08 +0000 Jeff Hostetler <git@xxxxxxxxxxxxxxxxx> wrote: > From: Jeff Hostetler <jeffhost@xxxxxxxxxxxxx> > > This part 3 of a 3 part sequence partial clone. It assumes > that part 1 and part 2 are in place. > > This patch series is labeled as V4 to keep it in sync with > the corresponding V4 versions of parts 1 and 2. There was > not a V3 version of this patch series. > > Jonathan and I independently started on this task. This is another > pass at merging those efforts. So there are several places that may > need refactoring and cleanup, but fewer than in the previous submission. > In particular, the test cases should be squashed and new tests added. > > And I think we need more end-to-end tests. I'll work on those next. I think that it would be easier to review if the test for each command was contained in the same patch as (or the patch immediately following) the implementation of the command - for example, as in my modifications [1]. (If you're about to send out v5, that's fine - maybe take this in consideration for v6, if there is one.) [1] https://github.com/jonathantanmy/git/commits/pc20171103