Re: [PATCH 1/3] merge: add config option for verifySignatures

2017-12-12 Thread Junio C Hamano
Kevin Daudt writes: > Hello Hans Jerry, > > Thank you for your contribution. I have soem remarks below. > ... Thanks for a detailed review. I agree with all the things you pointed out, and I see they are reflected in v2 of the series. Very much appreciated; thanks, both.

Re: [PATCH 1/3] merge: add config option for verifySignatures

2017-12-09 Thread Kevin Daudt
Hello Hans Jerry, Thank you for your contribution. I have soem remarks below. On Sat, Dec 09, 2017 at 09:05:28AM +, Hans Jerry Illikainen wrote: > Verify the signature of the tip commit when `merge.verifySignatures` is > true. This can be overridden with `--no-verify-signatures`. > >

[PATCH 1/3] merge: add config option for verifySignatures

2017-12-09 Thread Hans Jerry Illikainen
Verify the signature of the tip commit when `merge.verifySignatures` is true. This can be overridden with `--no-verify-signatures`. Signed-off-by: Hans Jerry Illikainen --- Documentation/merge-config.txt | 7 +++ builtin/merge.c| 2 ++