Hello campoy@google.com (cc: golang-dev@googlegroups.com), I'd like you to review this change to https://code.google.com/p/google-api-go-client/
LGTM Francesc Campoy Flores | Go Developer Relations | campoy@google.com | 415-990-4126 On Tue, May 21, 2013 at 5:11 PM, <campoy@golang.org> wrote: > Reviewers: campoy, > > Message: > Hello campoy@google.com (cc: golang-dev@googlegroups.com), > > I'd like you to review this change to > https://code.google.com/p/**google-api-go-client/<https://code.google.com/p/g... > > > Description: > test > > Please review this at https://codereview.appspot.**com/9631043/<https://codereview.appspot.com/9631... > > Affected files: > M plus/v1/plus-gen.go > > > Index: plus/v1/plus-gen.go > ==============================**==============================**======= > --- a/plus/v1/plus-gen.go > +++ b/plus/v1/plus-gen.go > @@ -1,3 +1,4 @@ > + > // Package plus provides access to the Google+ API. > // > // See https://developers.google.com/**+/api/<https://developers.google.com/+/api/> > > >
NOT LGTM changing auto-generated files by hand (what are you testing?) On Tue, May 21, 2013 at 5:11 PM, <campoy@golang.org> wrote: > Reviewers: campoy, > > Message: > Hello campoy@google.com (cc: golang-dev@googlegroups.com), > > I'd like you to review this change to > https://code.google.com/p/**google-api-go-client/<https://code.google.com/p/g... > > > Description: > test > > Please review this at https://codereview.appspot.**com/9631043/<https://codereview.appspot.com/9631... > > Affected files: > M plus/v1/plus-gen.go > > > Index: plus/v1/plus-gen.go > ==============================**==============================**======= > --- a/plus/v1/plus-gen.go > +++ b/plus/v1/plus-gen.go > @@ -1,3 +1,4 @@ > + > // Package plus provides access to the Google+ API. > // > // See https://developers.google.com/**+/api/<https://developers.google.com/+/api/> > > > -- > > ---You received this message because you are subscribed to the Google > Groups "golang-dev" group. > To unsubscribe from this group and stop receiving emails from it, send an > email to golang-dev+unsubscribe@**googlegroups.com<golang-dev%2Bunsubscribe@googlegrou... > . > For more options, visit https://groups.google.com/**groups/opt_out<https://groups.google.com/groups/o... > . > > >
I'm testing codereview.py, I won't submit this, sorry for the noise Francesc Campoy Flores | Go Developer Relations | campoy@google.com | 415-990-4126 On Tue, May 21, 2013 at 5:17 PM, Brad Fitzpatrick <bradfitz@golang.org>wrote: > NOT LGTM > > changing auto-generated files by hand > > (what are you testing?) > > > > On Tue, May 21, 2013 at 5:11 PM, <campoy@golang.org> wrote: > >> Reviewers: campoy, >> >> Message: >> Hello campoy@google.com (cc: golang-dev@googlegroups.com), >> >> I'd like you to review this change to >> https://code.google.com/p/**google-api-go-client/<https://code.google.com/p/g... >> >> >> Description: >> test >> >> Please review this at https://codereview.appspot.**com/9631043/<https://codereview.appspot.com/9631... >> >> Affected files: >> M plus/v1/plus-gen.go >> >> >> Index: plus/v1/plus-gen.go >> ==============================**==============================**======= >> --- a/plus/v1/plus-gen.go >> +++ b/plus/v1/plus-gen.go >> @@ -1,3 +1,4 @@ >> + >> // Package plus provides access to the Google+ API. >> // >> // See https://developers.google.com/**+/api/<https://developers.google.com/+/api/> >> >> >> >> -- >> >> ---You received this message because you are subscribed to the Google >> Groups "golang-dev" group. >> To unsubscribe from this group and stop receiving emails from it, send an >> email to golang-dev+unsubscribe@**googlegroups.com<golang-dev%2Bunsubscribe@googlegrou... >> . >> For more options, visit https://groups.google.com/**groups/opt_out<https://groups.google.com/groups/o... >> . >> >> >> >
*** Abandoned ***
