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

Re: Reviews on mailing-list

From
Felipe Contreras <felipe.contreras@gmail.com>
Date
Nov 10, 2012, 23:40 UTC
Message-ID
<CAMP44s2AvaAs-dn_eoSFWoLHBQk5QrrV=zYTOpRi2nEywoua4w@mail.gmail.com>
In-Reply-To
<CA+ZXwZOnZZyk_KWmLLLNChWb1R_Av1s5jM4PsLFWKaG8BB+uXQ@mail.gmail.com>
On Sun, Nov 11, 2012 at 12:19 AM, Deniz Türkoglu <deniz@spotify.com> wrote:
Show 11 quoted lines
> This is my first mail to the git mailing list. I have been following
> the list for some time now and I would like to suggest moving the
> reviews out of the mailing list, for example to a gerrit instance, I
> believe it would improve the commits and the mailing list. I have a
> filter on 'PATCH', but I feel I miss some of the discussion, and
> things that I would be interested in.
>
> I have spoken to Shawn Pearce (gerrit project lead, google) and he
> said he is OK with hosting the gerrit instance.
>
> I would like to hear your thoughts on this.

Personally I think reviews on the mailing list is far superior than any other review methods. I've even blogged about it and all the reasons[1]. Gerrit is better than bugzilla, but it still requires a web browser, and logging in.

I love to be able to just hit 'reply' with my favorite MUA, comment inline, and hit send.

Cheers.
[1] http://felipec.wordpress.com/2010/01/19/why-bugzilla-sucks-for-handling-patches/
-- 
Felipe Contreras
Previous: Deniz TürkogluNext: Thiago Farina
Message 2 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.