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

Re: Reviews on mailing-list

From
Deniz Türkoglu <deniz@spotify.com>
Date
Nov 11, 2012, 01:28 UTC
Message-ID
<CA+ZXwZO8tpGi7_njbFx6w2ZAWoySVb2Bcc+DSupLenKrNAGV_A@mail.gmail.com>
In-Reply-To
<CAMP44s2AvaAs-dn_eoSFWoLHBQk5QrrV=zYTOpRi2nEywoua4w@mail.gmail.com>

On Sat, Nov 10, 2012 at 3:40 PM, Felipe Contreras <felipe.contreras@gmail.com> wrote:

Show 18 quoted lines
> On Sun, Nov 11, 2012 at 12:19 AM, Deniz Türkoglu <deniz@spotify.com> wrote:
>
>> 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 disagree that the current approach is optimal. Bugzilla is a bug-tracker and is not meant to be used for reviews. I believe in using the right tool for the right job. An e-mail should be concise and to the point, in this case only contain the discussion. This will help it to reach a wider audience and be more useful when people stumble upon it through a google search.

cheers, -deniz

Previous: Nguyen Thai Ngoc DuyNext: Felipe Contreras
Message 15 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.