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

Re: Reviews on mailing-list

From
DLDavid Lang <david@lang.hm>
Date
Nov 12, 2012, 00:43 UTC
Message-ID
<alpine.DEB.2.02.1211111642030.19687@nftneq.ynat.uz>
In-Reply-To
<CA+ZXwZMh=ON4g32nRHD5cMTX1c7ohhqhXHUNdh57BMvSHG0cYA@mail.gmail.com>
On Sun, 11 Nov 2012, Deniz Türkoglu wrote:
Show 7 quoted lines
> I understand from the feedback that gerrit should get better on making
> it possible to review code via e-mail, as pointed out in Nguyen's
> mail, a flow like Shawn mentioned[1] can be a good solution.
>
> FWIW, I can fetch the change(s) from gerrit I am interested in and
> review it any time I want. I currently have many checked out topics I
> am working on for instance.

That requires that you know before you loose connectivity what changes you want to review.

With e-mail based reviews, you just pull copies of all your mail and it includes any pending reviews along with everything else.

David Lang
Previous: Deniz TürkogluNext: Nguyen Thai Ngoc Duy
Message 13 of 17 in “Reviews on mailing-list”
  1. Deniz TürkogluNov 10, 2012
  2. Felipe ContrerasNov 10, 2012
  3. Thiago FarinaNov 11, 2012
  4. Junio C HamanoNov 11, 2012
  5. suvayu aliNov 11, 2012
  6. Nguyen Thai Ngoc DuyNov 11, 2012
  7. Felipe ContrerasNov 11, 2012
  8. Thiago FarinaNov 11, 2012
  9. Felipe ContrerasNov 11, 2012
  10. Krzysztof MazurNov 11, 2012
  11. David LangNov 11, 2012
  12. Deniz TürkogluNov 12, 2012
  13. David LangNov 12, 2012
  14. Nguyen Thai Ngoc DuyNov 13, 2012
  15. Deniz TürkogluNov 11, 2012
  16. Felipe ContrerasNov 11, 2012
  17. Ramkumar RamachandraNov 11, 2012

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.