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

Issue 6777060: code review 6777060: go.talks: add 'go at google' (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
9 years, 11 months ago by r
Modified:
9 years, 11 months ago
Reviewers:
CC:
adg, dchest, golang-dev
Visibility:
Public.

Description

go.talks: add 'go at google' Keynote from Oct 25, 2012 by Rob Pike at ACM SPLASH conference.

Patch Set 1 #

Total comments: 1
Unified diffs Side-by-side diffs Delta from patch set Stats (+832 lines, -0 lines) Patch
A 2012/splash.slide View 1 chunk +832 lines, -0 lines 1 comment Download
A 2012/splash/appenginegophercolor.jpg View Binary file 0 comments Download
A 2012/splash/datacenter.jpg View Binary file 0 comments Download
A 2012/splash/fire.jpg View Binary file 0 comments Download

Messages

Total messages: 6
r
Hello adg@golang.org (cc: golang-dev@googlegroups.com), I'd like you to review this change to https://code.google.com/p/go.talks
9 years, 11 months ago (2012-10-26 16:44:59 UTC) #1
dchest
LGTM https://codereview.appspot.com/6777060/diff/1/2012/splash.slide File 2012/splash.slide (right): https://codereview.appspot.com/6777060/diff/1/2012/splash.slide#newcode779 2012/splash.slide:779: Several big services uses it: s/uses/use/
9 years, 11 months ago (2012-10-26 20:15:29 UTC) #2
r
It was recorded and I'm told it will be made available. I have no idea ...
9 years, 11 months ago (2012-10-26 21:34:36 UTC) #3
adg
LGTM modulo the typo pointed out by dchest
9 years, 11 months ago (2012-10-26 22:23:12 UTC) #4
adg
*** Submitted as http://code.google.com/p/go/source/detail?r=6566ce399759&repo=talks *** go.talks: add 'go at google' Keynote from Oct 25, 2012 ...
9 years, 11 months ago (2012-10-26 23:52:23 UTC) #5
r
9 years, 11 months ago (2012-10-27 16:07:36 UTC) #6
*** Abandoned ***
Sign in to reply to this message.

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