Re: [RFC/PATCH 6/8] config: add core.untrackedCache
- From
Christian Couder <christian.couder@gmail.com>
- Date
- Dec 3, 2015, 16:35 UTC
- Message-ID
- <CAP8UFD1cbogcojkY1ywTO=+cRb3mFB-d0W5MG4D_1K2vAx2=aw@mail.gmail.com>
- In-Reply-To
- <5660697D.5090308@web.de>
On Thu, Dec 3, 2015 at 5:10 PM, Torsten Bögershausen <tboegi@web.de> wrote:
Show 7 quoted lines
> [snip all good stuff] > > First of all: > Thanks for explaining it so well > > I now can see the point in having this patch. > (Do the commit messages reflect all this ? I need to re-read)
Maybe not. I will have a look at improving them, but your re-reading is welcome.
> The other question is: Do you have patch on a public repo ?
Yes, here: https://github.com/chriscool/git/commits/uc-notifs8
> And, of course, can we add 1 or 2 test cases ?
Yes I had planned to add tests for this. But it would be nice if I could know first if the last two patches are considered ok even though they are breaking compatibility a bit. In this case I could squash them with previous patches and only write tests for the resulting patches.
> Thanks for pushing this forward.
Thanks for your reviews, Christian.