git/list[1] front-page[2] threads[3] people[4] search[5] about
 

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

From
Junio C Hamano <gitster@pobox.com>
Date
Dec 12, 2017, 18:56 UTC
Message-ID
<xmqqh8sv3hde.fsf@gitster.mtv.corp.google.com>
In-Reply-To
<20171209120537.GA6006@alpha.vpn.ikke.info>
Kevin Daudt <me@ikke.info> 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.
Previous: Kevin DaudtNext: Hans Jerry Illikainen
Message 8 of 15 in “merge: add config option for verifySignatures”
  1. 1/3 merge: add config option for verifySignaturesHans Jerry Illikainen, Dec 9, 2017
  2. 2/3 t: add tests for pull --verify-signaturesHans Jerry Illikainen, Dec 9, 2017
  3. Kevin DaudtDec 9, 2017
  4. 3/3 pull: add config option for verifySignaturesHans Jerry Illikainen, Dec 9, 2017
  5. Kevin DaudtDec 9, 2017
  6. Hans Jerry IllikainenDec 10, 2017
  7. Kevin DaudtDec 9, 2017
  8. Junio C HamanoDec 12, 2017
  9. 1/2 merge: add config option for verifySignaturesHans Jerry Illikainen, Dec 10, 2017
  10. 2/2 t: add tests for pull --verify-signaturesHans Jerry Illikainen, Dec 10, 2017
  11. Junio C HamanoDec 12, 2017
  12. Re* [PATCH v2 2/2] t: add tests for pull --verify-signaturesJunio C Hamano, Dec 15, 2017
  13. Hans Jerry IllikainenDec 16, 2017
  14. Junio C HamanoDec 17, 2017
  15. 3/2 t5573, t7612: clean up after unexpected success of 'pull' and 'merge'Junio C Hamano, Dec 19, 2017

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.