code review 174072: codereview: add golang-dev@googlegroups.com

codereview: add golang-dev@googlegroups.com
    automatically in "hg mail".
also, avoid "empty list means all modified files in client" bug

[rsc, 2009-12-11 21:34:58]

Hello gri (cc: golang-dev@googlegroups.com),

I'd like you to review the following change.

[gri, 2009-12-11 21:50:01]

LGTM

On Fri, Dec 11, 2009 at 1:34 PM, <rsc@golang.org> wrote:

> Reviewers: gri,
>
> Message:
> Hello gri (cc: golang-dev@googlegroups.com),
>
> I'd like you to review the following change.
>
>
> Description:
> codereview: add golang-dev@googlegroups.com
>    automatically in "hg mail".
> also, avoid "empty list means all modified files in client" bug
>
> Please review this at http://codereview.appspot.com/174072
>
> Affected files:
>  M lib/codereview/codereview.py
>
>
>

[cw, 2009-12-11 22:53:32]

> > codereview: add mailto:golang-dev@googlegroups.com
> >    automatically in "hg mail".
> > also, avoid "empty list means all modified files in client" bug

is this really useful?  it seems at this pointing trying to keep the lists
low-volume and more focused is a good idea

nothing prevents someone from adding golang-dev@googlegroups.com in the cc'
initially if they feel this is something that deserves a wider audience

[rsc, 2009-12-15 21:36:09]

*** Submitted as http://code.google.com/p/go/source/detail?r=885bb5701b4c ***

codereview: add golang-dev@googlegroups.com
    automatically in "hg mail".
also, avoid "empty list means all modified files in client" bug

R=gri, cw
CC=golang-dev
http://codereview.appspot.com/174072