On Thu, Dec 3, 2015 at 5:10 PM, Torsten Bögershausen [off-list ref] wrote:
[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.