Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(151)

Issue 5632043: code review 5632043: goauth2: Convert buzz example to profile since buzz is ... (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
14 years, 4 months ago by M-A
Modified:
12 years, 9 months ago
Reviewers:
r, adg, dupoxy
CC:
golang-dev
Visibility:
Public.

Description

goauth2: Convert buzz example to profile since buzz is deprecated

Patch Set 1 #

Patch Set 2 : diff -r 72aedee5f568 https://code.google.com/p/goauth2/ #

Patch Set 3 : diff -r 72aedee5f568 https://code.google.com/p/goauth2/ #

Unified diffs Side-by-side diffs Delta from patch set Stats (+4 lines, -4 lines) Patch
M oauth/example/profile.go View 1 4 chunks +4 lines, -4 lines 0 comments Download

Messages

Total messages: 12
M-A
~/...src/code.google.com/p/goauth2> hg st M oauth/oauth.go A oauth/example/profile.go R oauth/example/buzz.go hg change failed to upload the ...
14 years, 4 months ago (2012-02-05 02:58:37 UTC) #1
adg
There's already another CL that addresses this: http://codereview.appspot.com/5632043/ I like your example code change, though.
14 years, 4 months ago (2012-02-05 12:28:15 UTC) #2
adg
Please take a look at this: http://codereview.appspot.com/5629060/
14 years, 4 months ago (2012-02-05 13:01:11 UTC) #3
M-A
Hello adg@golang.org (cc: golang-dev@googlegroups.com), I'd like you to review this change to https://code.google.com/p/goauth2/
14 years, 4 months ago (2012-02-05 14:10:38 UTC) #4
M-A
On 2012/02/05 14:10:38, M-A wrote: > Hello mailto:adg@golang.org (cc: mailto:golang-dev@googlegroups.com), > > I'd like you ...
14 years, 4 months ago (2012-02-05 14:11:24 UTC) #5
M-A
On 2012/02/05 14:11:24, M-A wrote: > (Sorry I didn't find the command to upload without ...
14 years, 4 months ago (2012-02-05 14:16:04 UTC) #6
dupoxy
somone on this list called minux once told me: " You can use 'hg upload ...
14 years, 4 months ago (2012-02-05 14:16:56 UTC) #7
adg
Hey, have you signed the CLA? http://golang.org/doc/contribute.html#copyright
14 years, 4 months ago (2012-02-05 23:42:56 UTC) #8
adg
Ping?
13 years, 10 months ago (2012-08-08 00:48:33 UTC) #9
r
LGTM
13 years, 10 months ago (2012-08-08 03:08:26 UTC) #10
M-A
On 2012/08/08 00:48:33, adg wrote: > Ping? FTR, I'm pretty sure I had communicated to ...
13 years, 10 months ago (2012-08-18 19:42:42 UTC) #11
adg
13 years, 10 months ago (2012-08-20 02:05:22 UTC) #12
I dimly recall that, but yeah, it's better to put it in the thread so
it's on record.

Anyway, it looks like Anthony Starks' recent changes have made this
one obsolete. Thanks anyway.

Andrew

On 19 August 2012 05:42,  <maruel@chromium.org> wrote:
> On 2012/08/08 00:48:33, adg wrote:
>>
>> Ping?
>
>
> FTR, I'm pretty sure I had communicated to you by IM that I am a Google
> employee so I didn't need to sign the CLA but I can't find it in my
> gtalk history. Sorry I should have replied here instead.
>
> http://codereview.appspot.com/5632043/
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b