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

Issue 6490102: code review 6490102: C: add Francesc Campoy (Google CLA) (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
11 years, 6 months ago by adg
Modified:
11 years, 6 months ago
Reviewers:
fss
CC:
golang-dev, dsymonds, campoy
Visibility:
Public.

Description

C: add Francesc Campoy (Google CLA)

Patch Set 1 #

Patch Set 2 : diff -r 50457a54c26c https://go.googlecode.com/hg #

Patch Set 3 : diff -r 54654f95a754 https://go.googlecode.com/hg #

Unified diffs Side-by-side diffs Delta from patch set Stats (+1 line, -0 lines) Patch
M CONTRIBUTORS View 1 1 chunk +1 line, -0 lines 0 comments Download

Messages

Total messages: 7
adg
Hello golang-dev@googlegroups.com, I'd like you to review this change to https://go.googlecode.com/hg
11 years, 6 months ago (2012-09-11 06:46:15 UTC) #1
dsymonds
Name typo. "Francesc".
11 years, 6 months ago (2012-09-11 06:47:48 UTC) #2
adg
Yeah already caught it. I blame Francisco Souza! PTAL On 11 September 2012 16:47, David ...
11 years, 6 months ago (2012-09-11 06:48:56 UTC) #3
dsymonds
LGTM but check with Francesc that this is how he wants his name rendered.
11 years, 6 months ago (2012-09-11 06:50:11 UTC) #4
campoy
LGTM
11 years, 6 months ago (2012-09-11 07:12:59 UTC) #5
adg
*** Submitted as http://code.google.com/p/go/source/detail?r=3796fdf0d0b8 *** C: add Francesc Campoy (Google CLA) R=golang-dev, dsymonds, campoy CC=golang-dev ...
11 years, 6 months ago (2012-09-11 07:15:27 UTC) #6
fss
11 years, 6 months ago (2012-09-11 20:18:34 UTC) #7
On Tue, Sep 11, 2012 at 3:48 AM, Andrew Gerrand <adg@golang.org> wrote:
> Yeah already caught it. I blame Francisco Souza!

don't blame me :-P


-- 
Francisco Souza
Sign in to reply to this message.

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